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.

Sign up for GitHub

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

Jump to bottom

Create tests for TransactionNotifier #291

Closed
Tracked by #298
ethan-tbd opened this issue Sep 5, 2024 · 2 comments · Fixed by #317
Closed
Tracked by #298

Create tests for TransactionNotifier #291

ethan-tbd opened this issue Sep 5, 2024 · 2 comments · Fixed by #317
Assignees
@victoreronmosele
Labels
hacktoberfest medium

Comments

@ethan-tbd
Copy link
Contributor

ethan-tbd commented Sep 5, 2024 •
edited
Loading

  • using the existing test suite as an example, create tests to stress-test TransactionNotifier
  • the TransactionNotifier widget is responsible for:
    • polling PFIs for a specific exchange id until an exchange is considered closed or until it is closed by the caller which started the polling
@taniashiba taniashiba mentioned this issue Sep 23, 2024
Open
13 tasks
@victoreronmosele
Copy link
Contributor

.take

@github-actions GitHub Actions
Copy link

Thanks for taking this issue! Let us know if you have any questions!

@victoreronmosele victoreronmosele mentioned this issue Oct 16, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@victoreronmosele victoreronmosele

Labels
hacktoberfest medium
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test: add tests for TransactionNotifier victoreronmosele/didpay
2 participants
@victoreronmosele @ethan-tbd