Skip to content

test(e2e): add extra scenarios to e2e tests#761

Open
javiertoledo wants to merge 1 commit intomainfrom
add-extra-scenarios-to-e2e-tests
Open

test(e2e): add extra scenarios to e2e tests#761
javiertoledo wants to merge 1 commit intomainfrom
add-extra-scenarios-to-e2e-tests

Conversation

@javiertoledo
Copy link
Member

Summary

  • Add 7 new e2e test scenarios covering previously untested areas:
    • Non-existent account query: verifies null response with no errors
    • Multi-account isolation: deposits to a second account and confirms both accounts maintain independent balances
    • ListAccountBalances: exercises the paginated list query with items and count
    • eventsByEntity: queries the event store for a specific entity's event history
    • eventsByType: queries all events of a given type across accounts
    • Invalid GraphQL query: verifies structured error response for unknown fields
  • Remove obsolete PARITY.md (bash-based e2e tests no longer exist)

Test plan

  • E2E suite passes in Docker CI (wf_test-e2e.yml)
  • New tests execute after the existing bank deposit flow
  • No regressions in existing tests

🤖 Generated with Claude Code

Cover aggregate isolation, ListAccountBalances, eventsByEntity/eventsByType
queries, non-existent account lookup, and invalid GraphQL error response.
Remove obsolete PARITY.md (bash-based tests no longer exist).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant