Comment skipped tests because appmapp sees them as failed

[hopsoft]

No description provided.

[github-actions]

AppMap runtime code review

Summary	Status
Failed tests	✅ All tests passed
API changes	0️⃣ No API changes
Security flaws	✅ None detected
Performance problems	✅ None detected
Code anti-patterns	🚨 4 new
New AppMaps	0️⃣ No new AppMaps
Removed AppMaps	✖️ 4 removed minitest tests

Code anti-patterns

🚨 New problems detected (4)

Data update performed in GET or HEAD request

Description

Data update performed in HTTP request GET /basic_command: INSERT INTO "users" ("session_id", "count", "created_at", "updated_at") VALUES (?, ?, ?, ?) RETURNING "id"

Field	Value
Rule	update-in-get-request
Impact domain	Maintainability

View in AppMap

• Full AppMap »
• Sequence diagram diff »

Stack trace

• vendor/bundle/ruby/3.0.0/gems/activesupport-7.1.3.2/lib/active_support/callbacks.rb:597

Data update performed in GET or HEAD request

Description

Data update performed in HTTP request GET /basic_command: UPDATE "users" SET "count" = COALESCE("count", 0) - ? WHERE "users"."id" = ?

Field	Value
Rule	update-in-get-request
Impact domain	Maintainability

View in AppMap

• Full AppMap »
• Sequence diagram diff »

Stack trace

• vendor/bundle/ruby/3.0.0/gems/activesupport-7.1.3.2/lib/active_support/callbacks.rb:597

Data update performed in GET or HEAD request

Description

Data update performed in HTTP request GET /tests: INSERT INTO "users" ("session_id", "count", "created_at", "updated_at") VALUES (?, ?, ?, ?) RETURNING "id"

Field	Value
Rule	update-in-get-request
Impact domain	Maintainability

View in AppMap

• Full AppMap »
• Sequence diagram diff »

Stack trace

• vendor/bundle/ruby/3.0.0/gems/activesupport-7.1.3.2/lib/active_support/callbacks.rb:597

Data update performed in GET or HEAD request

Description

Data update performed in HTTP request GET /basic_command: UPDATE "users" SET "count" = COALESCE("count", 0) + ? WHERE "users"."id" = ?

Field	Value
Rule	update-in-get-request
Impact domain	Maintainability

View in AppMap

• Full AppMap »
• Sequence diagram diff »

Stack trace

• vendor/bundle/ruby/3.0.0/gems/activesupport-7.1.3.2/lib/active_support/callbacks.rb:597

✖️ Removed AppMaps

[minitest] Reset frame increment once then reset and accept confirm

[minitest] Reset frame increment once then reset and dismiss confirm

[minitest] Reset no frame increment once then reset and accept confirm

[minitest] Reset no frame increment once then reset and dismiss confirm