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

Hanging promise in web3-eth integration tests #5079

Closed
spacesailor24 opened this issue May 26, 2022 · 9 comments
Closed

Hanging promise in web3-eth integration tests #5079

spacesailor24 opened this issue May 26, 2022 · 9 comments
Assignees
Labels
4.x 4.0 related Bug Addressing a bug Investigate

Comments

@spacesailor24
Copy link
Contributor

--forceExit flag should be removed from the test:integration npm script in web3-eth package, but can't because there seems to be hanging promises in:

@spacesailor24 spacesailor24 added the 4.x 4.0 related label May 26, 2022
@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. If you believe this was a mistake, please comment.

@github-actions github-actions bot added the Stale Has not received enough activity label Aug 16, 2022
@jdevcs jdevcs removed the Stale Has not received enough activity label Aug 25, 2022
@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. If you believe this was a mistake, please comment.

@github-actions github-actions bot added the Stale Has not received enough activity label Oct 25, 2022
@mconnelly8 mconnelly8 removed the Stale Has not received enough activity label Nov 1, 2022
@github-actions
Copy link

github-actions bot commented Jan 1, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. If you believe this was a mistake, please comment.

@github-actions github-actions bot added the Stale Has not received enough activity label Jan 1, 2023
@nikoulai nikoulai removed the Stale Has not received enough activity label Jan 3, 2023
@github-actions
Copy link

github-actions bot commented Mar 5, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. If you believe this was a mistake, please comment.

@github-actions github-actions bot added the Stale Has not received enough activity label Mar 5, 2023
@mconnelly8 mconnelly8 reopened this Apr 5, 2023
@mconnelly8 mconnelly8 removed the Stale Has not received enough activity label Apr 5, 2023
@jdevcs jdevcs self-assigned this Jul 16, 2024
@jdevcs jdevcs mentioned this issue Jul 18, 2024
17 tasks
@jdevcs
Copy link
Contributor

jdevcs commented Jul 22, 2024

defaults.test.ts - Websocket

  • System Test Utils ( number of functions )
  • transactionConfirmationBlocks
  • transactionConfirmationBlocks implementation
  • should fallback to polling if provider support on but newBlockHeaders does not arrive in blockHeaderTimeout seconds
  • defaultTransactionType

@jdevcs
Copy link
Contributor

jdevcs commented Jul 22, 2024

bug:
setMaxListenerWarningThreshold(); is not called for setting MaxListenerWarningThreshold in event emitter if config is set via setConfig(), so maxListenersWarningThreshold doesn't match with config and eventemitter Threshold values

@jdevcs
Copy link
Contributor

jdevcs commented Jul 22, 2024

defaults.transactionBlockTimeout.test.ts - Websocket

  • should fail if transaction was not mined within transactionBlockTimeout blocks
  • should fail if transaction was not mined within transactionBlockTimeout blocks - when subscription is used

@jdevcs
Copy link
Contributor

jdevcs commented Jul 25, 2024

list of tests in get_revert_reason.test.ts - Websocket

@jdevcs
Copy link
Contributor

jdevcs commented Jul 25, 2024

Tests in:

  • Defaults
  • defaults.transactionBlockTimeout
  • getRevertReason
  • WatchTransaction

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x 4.0 related Bug Addressing a bug Investigate
Projects
None yet
Development

No branches or pull requests

4 participants