test: add ArbSys precompile tests#162
Merged
AnkushinDaniil merged 1 commit intoAug 25, 2025
Merged
Conversation
hudem1
reviewed
Aug 23, 2025
Merged
e73ff60 to
980dd8a
Compare
wurdum
approved these changes
Aug 23, 2025
hudem1
reviewed
Aug 24, 2025
hudem1
reviewed
Aug 24, 2025
hudem1
reviewed
Aug 24, 2025
| } | ||
|
|
||
| [Test] | ||
| public void SendTxToL1_BurnsCorrectAmountOfGas() |
hudem1
reviewed
Aug 25, 2025
hudem1
approved these changes
Aug 25, 2025
bc9b122 to
cefcf69
Compare
1bb2183
into
hudem1/feat/precompile/arb-sys
3 of 5 checks passed
hudem1
pushed a commit
that referenced
this pull request
Aug 26, 2025
hudem1
added a commit
that referenced
this pull request
Aug 26, 2025
* feat: First version of incomplete implementation * feat: Implementation finished * fix: Make BurnBalance public * fix: InverseAddressAliasOffset fix * refactor: Move ArbSysMetadata code to ArbSys * clean: Remove console logs * fix: Use correct types for ArbSysL2ToL1Tx and ArbSysL2ToL1Transaction structs * fix: The value passed to WithdrawEth and SendTxToL1 is the solidity msg.value * fix: DecodeEvent now does not assume non-indexed params as last * quick fix: use BigInteger instead of UInt256 as value can be negative * fix: Return SendRoot from DigestMessage instead of Hash256.Zero * refactor(send-hash): Reuse ArbosStorage.CalculateHash instead of creating a new same one * chore: Use explicit types * refactor: Rename KeccakHashWithCost to ComputeKeccakHash * refactor: Add explicit comment * chore: Add context when throwing exception * chore: Ternary if convention style * test: ArbSysParserTests (#163) * Tests & few changes * feat: Use latest version of feature/arbitrum-setup for NMC submodule with StateStack * refactor: Use existing helper * test: add `ArbSys` precompile tests (#162) * fix: Revert unintentional changes when merging test PR * fix: Forgotten merge conflict changes * fix: Code format * fix: Remove local-only changes * fix: Code format --------- Co-authored-by: Stavros Vlachakis <89769224+svlachakis@users.noreply.github.com> Co-authored-by: Daniil Ankushin <ankushin.daniil42@gmail.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 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.
Fixes #168
Note
The base is
hudem1/feat/precompile/arb-sysImportant
These PRs need to be merged in order for the tests to pass:
#154
#9180
#9190