Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new system test for diagnostic events #40

Closed
tsachiherman opened this issue Mar 6, 2023 · 0 comments · Fixed by #53
Closed

Add new system test for diagnostic events #40

tsachiherman opened this issue Mar 6, 2023 · 0 comments · Fixed by #53
Assignees

Comments

@tsachiherman
Copy link
Contributor

tsachiherman commented Mar 6, 2023

What ?

What problem does your feature solve?

lack of automated e2e coverage of the soroban platform retrieval of diagnostic contract events at the dapp use case level

What would you like to see?

new automated test for e2e case:

as a dapp developer, I invoke a contract function that triggers a diagnostic event to be emitted by soroban, I receive the event through a filtered request for events.

test should be packaged in docker and as a go test and asserts/verifies events on both cli(soroban-tools) and nodejs(soroban-js-client).

Need to identify what filter parameters for diagnostic events the test should exercise:

  • 'by contract id'
  • 'by diagnostic event topic'
  • 'by function name'

note * - there is ticket #2 , which is for testing of just pure contract events, no diagnostic.

What alternatives are there?

manually perform the same test steps with cli

Epic

stellar/stellar-cli#471

@tsachiherman tsachiherman changed the title Add new system test for debug events Add new system test for diagnostic events Mar 15, 2023
@mollykarcher mollykarcher moved this from Backlog to Next Sprint Proposal in Platform Scrum Mar 20, 2023
@sreuland sreuland assigned sreuland and unassigned sreuland Mar 28, 2023
@sreuland sreuland moved this from Next Sprint Proposal to Current Sprint in Platform Scrum Mar 28, 2023
@sreuland sreuland self-assigned this Mar 28, 2023
@sreuland sreuland moved this from Current Sprint to In Progress in Platform Scrum Mar 28, 2023
sreuland added a commit to sreuland/system-test that referenced this issue Apr 14, 2023
sreuland added a commit to sreuland/system-test that referenced this issue Apr 14, 2023
sreuland added a commit to sreuland/system-test that referenced this issue Apr 14, 2023
sreuland added a commit to sreuland/system-test that referenced this issue Apr 14, 2023
sreuland added a commit to sreuland/system-test that referenced this issue Apr 14, 2023
…e scenario, lowers the duration of test runtimes
@sreuland sreuland moved this from In Progress to Needs Review in Platform Scrum Apr 14, 2023
@github-project-automation github-project-automation bot moved this from Needs Review to Done in Platform Scrum Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants