Skip to content

Conversation

@UdeshyaDhungana
Copy link
Contributor

@UdeshyaDhungana UdeshyaDhungana commented Nov 7, 2025

  • Added better error messages for frequently occuring errors regarding string type mismatch

Before
Screenshot 2025-11-07 at 13 52 49

After
Screenshot 2025-11-07 at 13 56 32


Note

Improve RESP string type-mismatch diagnostics and add a ping-pong test scenario that reproduces bulk vs simple string errors.

  • Assertions (RESP):
    • internal/resp_assertions/bulk_string_assertion.go: Add explicit error when a simple string matches expected bulk string.
    • internal/resp_assertions/simple_string_assertion.go: Add explicit error when a bulk string matches expected simple string.
    • internal/resp_assertions/bulk_string_present_in_array_assertion.go: Add diagnostic when matching simple string is found instead of bulk string in arrays.
  • Tests:
    • internal/stages_test.go: Add ping_pong_string_type_mismatch case for stage rg2.
    • New scenario internal/test_helpers/scenarios/ping-pong/string_type_mismatch/ with fixture output to simulate returning $4\r\nPONG\r\n and validate improved messaging.

Written by Cursor Bugbot for commit 7aa2ff6. This will update automatically on new commits. Configure here.

@UdeshyaDhungana UdeshyaDhungana merged commit 173b984 into main Nov 13, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants