-
Notifications
You must be signed in to change notification settings - Fork 83
fix: enabled Jumbo Transaction testing #3830
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
Conversation
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) ✅ license/snyk check is complete. No issues have been found. (View Details) |
Test Results 4 files - 16 533 suites +277 1m 1s ⏱️ - 17m 5s Results for commit dba175f. ± Comparison against base commit 27bcc1c. This pull request removes 666 and adds 1835 tests. Note that renamed tests count towards both.
This pull request removes 5 skipped tests and adds 1 skipped test. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
7d9b0b9
to
021e8fc
Compare
bc409bc
to
0dc7cb3
Compare
Closing due to known bugs #3769 (comment). Teams are working on resolving it so will re-visit the PR when things are more stable. |
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
- Upgraded local-node from version 2.32.5 to 2.37.1, which also involved upgrading @hashgraph/sdk from 2.49.2 to 2.66.0. - The SDK upgrade introduced a large number of changes. With @hashgraph/sdk@2.66.0, the servicesNode.createHTS method used in batch3 failed with the following error: StatusError: transaction transaction_id failed precheck with status INVALID_RENEWAL_PERIOD against node account id 0.0.3. - To prevent test failures, reverted @hashgraph/sdk back to 2.49.2. - A follow-up ticket will be created to resolve the SDK version discrepancy, as it is outside the scope of this PR. Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
- Upgraded local-node from version 2.32.5 to 2.37.1, which includes upgrading the Mirror Node from 0.118.0-rc1 to 0.131.0. This version gap introduces a large number of changes. - The test “should NOT allow eth_call to process IHRC719.isAssociated()” is temporarily skipped and will be addressed in a follow-up PR. Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
This reverts commit 777c8df. Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
0dc7cb3
to
dba175f
Compare
Description:
This PR upgrades local-node to version 2.37.1, which supports Consensus Node @^0.62.2—the latest stable version with Jumbo Transaction support.
It also upgrades the overridden ethers.js to v6 to avoid conflicts with child dependencies introduced by local-node@2.37.0.
Related issue(s):
Fixes #3769
Notes for reviewer:
Checklist