-
Notifications
You must be signed in to change notification settings - Fork 15
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
Test: Ethereum Settlement Client #81
Merged
Merged
Changes from 32 commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
84a3819
update: settlement-client: ethereum test cases for conversion fns
heemankv 0c6d868
update: removed .expect from slice_u8_to_u256
heemankv f77093b
update: Eth Settlement client: tests for conversions
heemankv c23a4a6
update: Eth Settlement client : prepare_sidecar
heemankv 5e2ea5e
update: settlement_client: eth: prepare_sidecar tests
heemankv f435b1d
chore: optimisations
heemankv 229e50d
update: working test case
heemankv aa1bef2
update: working test #2
apoorvsadana 3158c48
update: added cfg test for update_state_with_blobs
heemankv db1a675
update: cleaner cfg(test) implemented code for update_state_and_blob_…
heemankv d562e4c
chore: liniting fixes
heemankv 69cc0b7
update: linting fixes
heemankv 46fc944
docs: changelog
heemankv c6bfdce
update: Nonce prefetch for state_update
heemankv f3fc826
update: creation of input_data works
heemankv d28b98b
update: using correct input bytes
heemankv a03f9db
chore: lint fix
heemankv 06d97cd
update: test normal transaction
heemankv 5aa7b10
update: dummy contract and impersonation tests ready
heemankv e7023af
update: test cases for settlement client
heemankv 391d693
chore: lint fixes
heemankv 856fa1f
chore: fix lints
heemankv a03387f
Merge branch 'main' into test/settlement-client
heemankv 6b69fdd
update: Changes for PR review
heemankv bbecc38
chore: fixing test cases for eth settlement client
heemankv 01294a0
update: tests fix
heemankv 1178e22
chore: lint fix
heemankv acf9738
Merge branch 'main' into test/settlement-client
heemankv 60fac37
chore: lint fix
heemankv 3a125ad
update: path fix
heemankv e2a0bed
update: testing anvil install on gh
heemankv 92aa145
update: fixing path
heemankv 6c03f7b
update: added Blast rpc for eth
heemankv a3c0863
update: Coverage CI fixes
heemankv d0ddfde
update: correct Pr checks
heemankv 4de6e8b
update: PR reviews fixes
heemankv ebe967e
update: PR reviews fixes
heemankv ef35e52
update: adding rationale for update_state_blob_with_impersonation_wor…
heemankv a8f4bb1
update: cleaner test integration on update_state_with_blobs
heemankv 30e8584
update: removing EthProvider
heemankv ef5a562
Reworking Settlement Client Changes (#89)
apoorvsadana 6b7a656
Merge branch 'main' into test/settlement-client
heemankv 8e9c13d
Merge branch 'main' into test/settlement-client
heemankv b93372e
update PR reviews fixed
heemankv f9d3682
update PR reviews fixed
heemankv File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you okay with this being public? If this is not a problem, document why this is set in Github.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ideally, we can remove this but it's genuinely no harm since this is Private_key provided by Anvil, will add a comment to explain the same