test: stabilize websocket tests #1272
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tabaktoni
added a commit
that referenced
this pull request
Mar 6, 2025
* feat: ws init setup * feat: init ws and test * feat: subscribe, unsubscribe, waitForDisconnection, test flow for newHeads working fine * feat: subscribeNewHeads done and tested * feat: test standard endpoint, update wait methods with statuses * feat: transaction status and pending transactions * fix: missing file * feat: onUnsubscribe, make bulk test works * feat: beta types-js release and import, on methods data type definitions added * feat: docs, extended tests, events map, types * feat: manage subscriptions by map * test: onUnsubscribe, docs, cleanup * fix: cleanup * fix: tipo and test * Update __tests__/WebSocketChannel.test.ts Co-authored-by: Ivan Pavičić <[email protected]> * Update src/channel/ws_0_8.ts Co-authored-by: Ivan Pavičić <[email protected]> * Update src/channel/ws_0_8.ts Co-authored-by: Ivan Pavičić <[email protected]> * fix: a SUBSCRIPTION_RESULT, bump types beta.2, debug tests * chore: starknet-types to starknet-types-08 * docs: websocket channel docs init * docs: websocket channel docs init * fix: block_id and txStatus event name * feat: add types-beta-4, add T check on socket methods * test: stabilize websocket tests (#1272) * feat: onReorg * fix: update packages * chore: updated json lock --------- Co-authored-by: Ivan Pavičić <[email protected]> Co-authored-by: Petar Penović <[email protected]>
github-actions bot
pushed a commit
that referenced
this pull request
Mar 10, 2025
# [7.0.0-beta.2](v7.0.0-beta.1...v7.0.0-beta.2) (2025-03-10) ### Bug Fixes * contract withOptions Fix/1252 ([#1255](#1255)) ([b59952e](b59952e)) * update ws impl, add TEST_WS_URL, update token constants ([411eeec](411eeec)) ### Features * the WebSockets ([#1251](#1251)) ([fabca27](fabca27)), closes [#1272](#1272) * v7 fee, new methods, tests ([#1337](#1337)) ([00743de](00743de))
github-actions bot
pushed a commit
that referenced
this pull request
Mar 28, 2025
# [7.0.0](v6.24.1...v7.0.0) (2025-03-28) * Implement RPC 0.8 and V3 transactions ([#1328](#1328)) ([316ae27](316ae27)) ### Bug Fixes * contract withOptions Fix/1252 ([#1255](#1255)) ([b59952e](b59952e)) * update ws impl, add TEST_WS_URL, update token constants ([411eeec](411eeec)) ### Features * drop fetch-cookie dependency ([9a57daf](9a57daf)) * drop isomorphic-fetch dependency ([9da0083](9da0083)) * prune deprecated functionalities ([cbd18f5](cbd18f5)) * the WebSockets ([#1251](#1251)) ([fabca27](fabca27)), closes [#1272](#1272) * Update Beta to latest Develop ([#1312](#1312)) ([2958051](2958051)), closes [#949](#949) [#970](#970) * v7 fee, new methods, tests ([#1337](#1337)) ([00743de](00743de)) ### BREAKING CHANGES * Removed multiple functionalities that have been marked as deprecated * Library defaults to RPC 0.8 with the corresponding API changes, dropped RPC 0.6 support
PhilippeR26
added a commit
to PhilippeR26/starknet.js
that referenced
this pull request
May 5, 2025
docs: update guide build: ledger app v2.3.0 chore: upgrade name to 2.3.1 and polish doc docs: update guide chore: upgrade name to 2.3.1 and polish doc feat: prune deprecated functionalities BREAKING CHANGE: Removed multiple functionalities that have been marked as deprecated chore(release): 7.0.0-beta.4 [skip ci] * repair snip-12 enum type nested dependency ([starknet-io#1289](starknet-io#1289)) ([1cd4219](starknet-io@1cd4219)) * drop fetch-cookie dependency ([9a57daf](starknet-io@9a57daf)) * drop isomorphic-fetch dependency ([9da0083](starknet-io@9da0083)) * prune deprecated functionalities ([cbd18f5](starknet-io@cbd18f5)) * Removed multiple functionalities that have been marked as deprecated chore(release): 7.0.0 [skip ci] * Implement RPC 0.8 and V3 transactions ([starknet-io#1328](starknet-io#1328)) ([316ae27](starknet-io@316ae27)) * contract withOptions Fix/1252 ([starknet-io#1255](starknet-io#1255)) ([b59952e](starknet-io@b59952e)) * update ws impl, add TEST_WS_URL, update token constants ([411eeec](starknet-io@411eeec)) * drop fetch-cookie dependency ([9a57daf](starknet-io@9a57daf)) * drop isomorphic-fetch dependency ([9da0083](starknet-io@9da0083)) * prune deprecated functionalities ([cbd18f5](starknet-io@cbd18f5)) * the WebSockets ([starknet-io#1251](starknet-io#1251)) ([fabca27](starknet-io@fabca27)), closes [starknet-io#1272](starknet-io#1272) * Update Beta to latest Develop ([starknet-io#1312](starknet-io#1312)) ([2958051](starknet-io@2958051)), closes [starknet-io#949](starknet-io#949) [starknet-io#970](starknet-io#970) * v7 fee, new methods, tests ([starknet-io#1337](starknet-io#1337)) ([00743de](starknet-io@00743de)) * Removed multiple functionalities that have been marked as deprecated * Library defaults to RPC 0.8 with the corresponding API changes, dropped RPC 0.6 support fix: preserve value for numeric arguments within address padding utility chore(release): 7.0.1 [skip ci] * preserve value for numeric arguments within address padding utility ([abc7958](starknet-io@abc7958)) docs: generate documentation version docs: expand v7 migration guide (starknet-io#1368) --------- Co-authored-by: Petar Penovic <[email protected]> fix: make the provider create method generic to enable it when mixins are applied (starknet-io#1370) fix: ensure enum replacement objects are reachable for type resolution (starknet-io#1375) docs: update guides for v7 (starknet-io#1376) --------- Co-authored-by: Petar Penovic <[email protected]> feat: update to rpc 0.8.1 chore(release): 7.1.0 [skip ci] * ensure enum replacement objects are reachable for type resolution ([starknet-io#1375](starknet-io#1375)) ([eb8b10d](starknet-io@eb8b10d)) * make the provider create method generic to enable it when mixins are applied ([starknet-io#1370](starknet-io#1370)) ([1f48dc6](starknet-io@1f48dc6)) * update to rpc 0.8.1 ([83e17b8](starknet-io@83e17b8)) style: remove comment
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation and Resolution
This PR contains small tweaks to stabilize the tests:
WebSocketChannelinitializations into setup hooks ensures they are bound to their respectivedescribescopes and properly followed by the teardown hooks, without the change they are treated as if they were in the same higher scope causing one of them to be orphaned in case of errorsisowsdependency with a mock allows the tests fromutils/stark.browser.test.tsto run