Integration tests #71
Merged
Conversation
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
lesa-telos
requested review from
poplexity,
aamirpashaa,
guilledk and
coa-telos
lesa-telos
force-pushed
the
integration-tests
branch
from
b0950d4 to
51ef536
Compare
guilledk
approved these changes
Nov 21, 2024
aamirpashaa
added a commit
that referenced
this pull request
Dec 30, 2024
Adding feature for telos Changes for cleaner logging of compare data Update container version and don't gobble errors on send transaction Bumping cargo versions, specifically updating revm version Change how we construct the chainspec Fix for Packer trait mismatch with dependency versions, still not working for accountstate table lookup Add code override based on original_size and separate override code test Remove bad import Starting of compare_account More strict on cache loading so we don't think we are writing to storage when the account was missing build.sh working Attempt at getting rebase to compile, still broken Fixing missing feature flag Update revm to get latest patch for deposit to address 0 Pin consensus dep to right rev Setting values for TelosArgs and changing sleep to tokio Bumping cargo to update translator for test Changes for integration test Remove duplicate test in args Cleanup from rebase, temp disable trx forwarding to finish the rebase Clean up error handling on sending transactions to Telos Only run test on pull_request Refactor and fix for storage comparision logic Remove access list so we send a proper LegacyTransaction Use correct branch of consensus client for now Updating readme with squash notes Add an exception for addresses that are not actually changed on revm in comparision Moving to use latest master of antelope-client Update consensus client dependency Removing .github files from upstream Adding the start of a storage compare script Adding to live test Initialize state diff comparator Add more checks to two-way storage compare Add back in search by code hash for case where we hit the de-duplicator eth_gasPrice modification Updating to latest consensus client Add exception for key-value removal by setting value to 0 Integration test fixes Adding TryFrom for ExEx support and adding key to logging of account storage diff Load account into cache before we attempt to read/write it's storage Use KECCAK_EMPTY for account override as well as code override Cleanup compare function Trying to get test working Add exception for addresses created using openwallet in comparator Reset first_new_address to None in fill_tx_env Remove unwrap Fixing integration test Adding "removed" field for state deltas Update to latest testcontainer with low gas limit on erc20 transactions Improve approve test. Fix getTransactionByhash Passing evm to compare function, better handling of duplicate tevm diffs Updating cargo.lock and adding CI workflow Add retry to telos send_transaction Use OnceCell instead of RwLock for telos client Pass extra fields to execute function Pinning consensus client version for integration test State bypass & other fixes for release 1.0.0 Bring amirs rebase fixes in execute.rs Telos EVM Runtime Changes Update wait for shutdown in integration test. Refactor compare function logic Better logging on compare panic, fixes for integration test Fix override_code function for the case telos account has empty array as code field, and enable compare Dont call compare function due to bugs on it and new two-way-compare script Enabling live comparison for account table only Add double approve test. Remove parent block gas price and revision read in execute_state_transitions Replace the pending block with the latest block in all RPC functions A little more on compare_account function Initial changes to store TelosBlockExtension Replace pending block with latest block in all RPC functions Improvements to integration test Updating consensus version Add tests to trigger panics on compare_state_diffs function Adding more feature flags Add two-way storage compare Remove unused mut and add block number to compare panic Ensuring telos feature flag is enabled Add Telos mainnet/testnet chainspecs/genesis to the supported chains Refactor code compraison by size Try setting code to None instead of Some(Bytecode::default()) Almost working storage_compare Adding some fixes for integration test Remove the check for gas limit elasticity Updating revm version for push0 fix Modify start script to allow more configuration Adding telos feature and cleanup imports Add telos engine api extra fields as crate and add it to the intermediate function parameters Fix test suite HashMap::default and finish clean impl of MockDBError in order to maintain traits in compare function Update consesus client dependency Add state override mechanism for most cases, need to finish storage Reverting unnecesary changes Fixing integration test so it doesn't drop stuff Point to specific consensus revision. Updating consensus client version Add recipts vector to extra telos fields struct Adding start/stop/build scripts and example config Fixing dependency Changes to make all tests pass except the one commented out due to hitting unreachable code panic Update consensus client and improve error handling in test Adding missing feature flag Update antelope client for send transaction resposne parsing fix Disable cargo check for now Update test. Fix warnings Exit integration test in case of failure Enable telos feature for extra fields Remove hardcoded revision/gas_price values Adding support for using TelosTxEnv for call/gas estimation If the account is not in reth_db then set status to Created|Touched Working storage of gas price/revision number enough for integration test to pass Update revm to get latest patch for pre-charge of address 0 Fix warnings on node tests Pass around telos_extra_fields on side chain append functions until execution api, increase batch_size on integration test Telos crate/binary, e2e test WIP: Attempting to maintain our changes to the Header table without forking alloy Update integration test. Bumping version of antelope-rs Modifications to storage_compare Disambiguate error messages and fix Option/Result for cached account loading Working/passing block num test Updating cargo lock for latest consensus/translator changes Add test for eip1559 and eip2930 transaction types. Set correct original_value while overriding storage Migrating to use a TelosEthApi impl to forward transactions to Telos zero, adding live tests Adding support for experimental engine Integration tests (#71) Minimize telos chain config changes Further WIP on rebase, compiles and runs but the Telos gas/revision values are sometimes default/zero Fill missing telos args Fixing compile issues for test Add block number to print output Update TELOS_README.md Update alloy-primitives, to fix zero sig value issueAll Telos Changes (Squashed commit) Adding feature for telos Changes for cleaner logging of compare data Update container version and don't gobble errors on send transaction Bumping cargo versions, specifically updating revm version Change how we construct the chainspec Fix for Packer trait mismatch with dependency versions, still not working for accountstate table lookup Add code override based on original_size and separate override code test Remove bad import Starting of compare_account More strict on cache loading so we don't think we are writing to storage when the account was missing build.sh working Attempt at getting rebase to compile, still broken Fixing missing feature flag Update revm to get latest patch for deposit to address 0 Pin consensus dep to right rev Setting values for TelosArgs and changing sleep to tokio Bumping cargo to update translator for test Changes for integration test Remove duplicate test in args Cleanup from rebase, temp disable trx forwarding to finish the rebase Clean up error handling on sending transactions to Telos Only run test on pull_request Refactor and fix for storage comparision logic Remove access list so we send a proper LegacyTransaction Use correct branch of consensus client for now Updating readme with squash notes Add an exception for addresses that are not actually changed on revm in comparision Moving to use latest master of antelope-client Update consensus client dependency Removing .github files from upstream Adding the start of a storage compare script Adding to live test Initialize state diff comparator Add more checks to two-way storage compare Add back in search by code hash for case where we hit the de-duplicator eth_gasPrice modification Updating to latest consensus client Add exception for key-value removal by setting value to 0 Integration test fixes Adding TryFrom for ExEx support and adding key to logging of account storage diff Load account into cache before we attempt to read/write it's storage Use KECCAK_EMPTY for account override as well as code override Cleanup compare function Trying to get test working Add exception for addresses created using openwallet in comparator Reset first_new_address to None in fill_tx_env Remove unwrap Fixing integration test Adding "removed" field for state deltas Update to latest testcontainer with low gas limit on erc20 transactions Improve approve test. Fix getTransactionByhash Passing evm to compare function, better handling of duplicate tevm diffs Updating cargo.lock and adding CI workflow Add retry to telos send_transaction Use OnceCell instead of RwLock for telos client Pass extra fields to execute function Pinning consensus client version for integration test State bypass & other fixes for release 1.0.0 Bring amirs rebase fixes in execute.rs Telos EVM Runtime Changes Update wait for shutdown in integration test. Refactor compare function logic Better logging on compare panic, fixes for integration test Fix override_code function for the case telos account has empty array as code field, and enable compare Dont call compare function due to bugs on it and new two-way-compare script Enabling live comparison for account table only Add double approve test. Remove parent block gas price and revision read in execute_state_transitions Replace the pending block with the latest block in all RPC functions A little more on compare_account function Initial changes to store TelosBlockExtension Replace pending block with latest block in all RPC functions Improvements to integration test Updating consensus version Add tests to trigger panics on compare_state_diffs function Adding more feature flags Add two-way storage compare Remove unused mut and add block number to compare panic Ensuring telos feature flag is enabled Add Telos mainnet/testnet chainspecs/genesis to the supported chains Refactor code compraison by size Try setting code to None instead of Some(Bytecode::default()) Almost working storage_compare Adding some fixes for integration test Remove the check for gas limit elasticity Updating revm version for push0 fix Modify start script to allow more configuration Adding telos feature and cleanup imports Add telos engine api extra fields as crate and add it to the intermediate function parameters Fix test suite HashMap::default and finish clean impl of MockDBError in order to maintain traits in compare function Update consesus client dependency Add state override mechanism for most cases, need to finish storage Reverting unnecesary changes Fixing integration test so it doesn't drop stuff Point to specific consensus revision. Updating consensus client version Add recipts vector to extra telos fields struct Adding start/stop/build scripts and example config Fixing dependency Changes to make all tests pass except the one commented out due to hitting unreachable code panic Update consensus client and improve error handling in test Adding missing feature flag Update antelope client for send transaction resposne parsing fix Disable cargo check for now Update test. Fix warnings Exit integration test in case of failure Enable telos feature for extra fields Remove hardcoded revision/gas_price values Adding support for using TelosTxEnv for call/gas estimation If the account is not in reth_db then set status to Created|Touched Working storage of gas price/revision number enough for integration test to pass Update revm to get latest patch for pre-charge of address 0 Fix warnings on node tests Pass around telos_extra_fields on side chain append functions until execution api, increase batch_size on integration test Telos crate/binary, e2e test WIP: Attempting to maintain our changes to the Header table without forking alloy Update integration test. Bumping version of antelope-rs Modifications to storage_compare Disambiguate error messages and fix Option/Result for cached account loading Working/passing block num test Updating cargo lock for latest consensus/translator changes Add test for eip1559 and eip2930 transaction types. Set correct original_value while overriding storage Migrating to use a TelosEthApi impl to forward transactions to Telos zero, adding live tests Adding support for experimental engine Integration tests (#71) Minimize telos chain config changes Further WIP on rebase, compiles and runs but the Telos gas/revision values are sometimes default/zero Fill missing telos args Fixing compile issues for test Add block number to print output Update TELOS_README.md Update alloy-primitives, to fix zero sig value issueAll Telos Changes (Squashed commit) Adding feature for telos Changes for cleaner logging of compare data Update container version and don't gobble errors on send transaction Bumping cargo versions, specifically updating revm version Change how we construct the chainspec Fix for Packer trait mismatch with dependency versions, still not working for accountstate table lookup Add code override based on original_size and separate override code test Remove bad import Starting of compare_account More strict on cache loading so we don't think we are writing to storage when the account was missing build.sh working Attempt at getting rebase to compile, still broken Fixing missing feature flag Update revm to get latest patch for deposit to address 0 Pin consensus dep to right rev Setting values for TelosArgs and changing sleep to tokio Bumping cargo to update translator for test Changes for integration test Remove duplicate test in args Cleanup from rebase, temp disable trx forwarding to finish the rebase Clean up error handling on sending transactions to Telos Only run test on pull_request Refactor and fix for storage comparision logic Remove access list so we send a proper LegacyTransaction Use correct branch of consensus client for now Updating readme with squash notes Add an exception for addresses that are not actually changed on revm in comparision Moving to use latest master of antelope-client Update consensus client dependency Removing .github files from upstream Adding the start of a storage compare script Adding to live test Initialize state diff comparator Add more checks to two-way storage compare Add back in search by code hash for case where we hit the de-duplicator eth_gasPrice modification Updating to latest consensus client Add exception for key-value removal by setting value to 0 Integration test fixes Adding TryFrom for ExEx support and adding key to logging of account storage diff Load account into cache before we attempt to read/write it's storage Use KECCAK_EMPTY for account override as well as code override Cleanup compare function Trying to get test working Add exception for addresses created using openwallet in comparator Reset first_new_address to None in fill_tx_env Remove unwrap Fixing integration test Adding "removed" field for state deltas Update to latest testcontainer with low gas limit on erc20 transactions Improve approve test. Fix getTransactionByhash Passing evm to compare function, better handling of duplicate tevm diffs Updating cargo.lock and adding CI workflow Add retry to telos send_transaction Use OnceCell instead of RwLock for telos client Pass extra fields to execute function Pinning consensus client version for integration test State bypass & other fixes for release 1.0.0 Bring amirs rebase fixes in execute.rs Telos EVM Runtime Changes Update wait for shutdown in integration test. Refactor compare function logic Better logging on compare panic, fixes for integration test Fix override_code function for the case telos account has empty array as code field, and enable compare Dont call compare function due to bugs on it and new two-way-compare script Enabling live comparison for account table only Add double approve test. Remove parent block gas price and revision read in execute_state_transitions Replace the pending block with the latest block in all RPC functions A little more on compare_account function Initial changes to store TelosBlockExtension Replace pending block with latest block in all RPC functions Improvements to integration test Updating consensus version Add tests to trigger panics on compare_state_diffs function Adding more feature flags Add two-way storage compare Remove unused mut and add block number to compare panic Ensuring telos feature flag is enabled Add Telos mainnet/testnet chainspecs/genesis to the supported chains Refactor code compraison by size Try setting code to None instead of Some(Bytecode::default()) Almost working storage_compare Adding some fixes for integration test Remove the check for gas limit elasticity Updating revm version for push0 fix Modify start script to allow more configuration Adding telos feature and cleanup imports Add telos engine api extra fields as crate and add it to the intermediate function parameters Fix test suite HashMap::default and finish clean impl of MockDBError in order to maintain traits in compare function Update consesus client dependency Add state override mechanism for most cases, need to finish storage Reverting unnecesary changes Fixing integration test so it doesn't drop stuff Point to specific consensus revision. Updating consensus client version Add recipts vector to extra telos fields struct Adding start/stop/build scripts and example config Fixing dependency Changes to make all tests pass except the one commented out due to hitting unreachable code panic Update consensus client and improve error handling in test Adding missing feature flag Update antelope client for send transaction resposne parsing fix Disable cargo check for now Update test. Fix warnings Exit integration test in case of failure Enable telos feature for extra fields Remove hardcoded revision/gas_price values Adding support for using TelosTxEnv for call/gas estimation If the account is not in reth_db then set status to Created|Touched Working storage of gas price/revision number enough for integration test to pass Update revm to get latest patch for pre-charge of address 0 Fix warnings on node tests Pass around telos_extra_fields on side chain append functions until execution api, increase batch_size on integration test Telos crate/binary, e2e test WIP: Attempting to maintain our changes to the Header table without forking alloy Update integration test. Bumping version of antelope-rs Modifications to storage_compare Disambiguate error messages and fix Option/Result for cached account loading Working/passing block num test Updating cargo lock for latest consensus/translator changes Add test for eip1559 and eip2930 transaction types. Set correct original_value while overriding storage Migrating to use a TelosEthApi impl to forward transactions to Telos zero, adding live tests Adding support for experimental engine Integration tests (#71) Minimize telos chain config changes Further WIP on rebase, compiles and runs but the Telos gas/revision values are sometimes default/zero Fill missing telos args Fixing compile issues for test Add block number to print output Update TELOS_README.md Update alloy-primitives, to fix zero sig value issue
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 #issue_number
Add tests for transactions that are not supported and calling approve method multiple times on erc20 contract.
Description
Test scenarios
Checklist: