-
Notifications
You must be signed in to change notification settings - Fork 11.4k
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
[fastx adapter] fix failing expected value test #21
Comments
Not needed after 6d48c5c |
brson
pushed a commit
to brson/sui
that referenced
this issue
Apr 30, 2024
Tests can now add compute_unit_limit as a config. Followups - Remove `DEFAULT_EXECUTION_BOUND_SOLANA_STDLIB_TEST` added in MystenLabs#18 - Parse GAS_BUDGET_HEAP_SIZE - Parse GAS_BUDGET_MAX_CALL_DEPTH Fixes: MystenLabs#20
sadhansood
added a commit
that referenced
this issue
Mar 7, 2025
#21440) ## Description cherry pick #21412 ## Test plan How did you test the new or updated feature? --- ## Release notes Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required. For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates. - [ ] Protocol: - [ ] Nodes (Validators and Full nodes): - [ ] gRPC: - [ ] JSON-RPC: - [ ] GraphQL: - [ ] CLI: - [ ] Rust SDK: Co-authored-by: phoenix <51927076+phoenix-o@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This expected value test works if you run it directly:
, but not if you uncomment the
test::run_one
to drive the test via thecargo test
hook here:Looks like a crash inside of
diem
code.The text was updated successfully, but these errors were encountered: