Skip to content
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

Ndev 2981 trace call overrides params #327

Open
wants to merge 43 commits into
base: develop
Choose a base branch
from

Conversation

romanova-natasha
Copy link

@romanova-natasha romanova-natasha commented May 8, 2024

  1. eth_call with overriding is not supported yet (task for it NDEV-2400)
  2. for blockOverrides we support only time and number parameters

add case to check stateOverrides nonce
add dbg prints and run only one test
mv tests to new file, add cases for stateOverride check
add cases for stateOverride check
rm dbg
link bugs to tests and skip these cases
@romanova-natasha romanova-natasha marked this pull request as draft May 8, 2024 20:18
add blockOverride case using debug_traceCall request
add case with blockOverrides for debug_traceCall method
add comment to test_stateOverrides_eth_call_override_code case
minor refactoring
…omment to cases with useless error message, skip one case due to bug 3010

add negative cases to override params suite, minor refactoring, add comment to cases with useless error message, skip one case due to bug 3010
add cases for blockOverrides with time parameter
add negative cases for stateDiff param
add cases for stateOverride nonce, balance
add cases for stateOverride code parameter
enable stateDiff cases which were fixed
…lance for executing tx

fix stateOverride balance tests, add a negative case: insufficient balance for executing tx
…for stateOverrides lower nonce

add blockOverrides time and number case, minor refactoring, add case for stateOverrides lower nonce
… stateOverrides all params without storage, small refactoring

add case: stateOverrides and blockOverrides in one requests, add case stateOverrides all params without storage, small refactoring
refactoring of debug_methods suite according to the recent changes
add stateOverride cases for state param
@romanova-natasha romanova-natasha marked this pull request as ready for review May 14, 2024 07:42
integration/tests/tracer/test_override_params.py Outdated Show resolved Hide resolved
integration/tests/tracer/test_override_params.py Outdated Show resolved Hide resolved
integration/tests/tracer/test_override_params.py Outdated Show resolved Hide resolved
integration/tests/tracer/test_override_params.py Outdated Show resolved Hide resolved
integration/tests/tracer/test_override_params.py Outdated Show resolved Hide resolved
integration/tests/tracer/test_override_params.py Outdated Show resolved Hide resolved
integration/tests/tracer/test_override_params.py Outdated Show resolved Hide resolved
integration/tests/tracer/test_override_params.py Outdated Show resolved Hide resolved
…rrided code for contract and externally-owned accounts

fix tests stateOverrides with code param: add check of trace with overrided code for contract and externally-owned accounts
…s overrided

add contract function with retrieving sender balance, check balance is overrided
fix: add make_raw_tx where it is nedeed
add function: fill params for prestateTracer call
mv storage contract to separate class
fix review
fix review: do not wait transaction
add case for stateDiff: check storage with 2 indixes
rm code field from state and stateDiff requests
fix state/stateDiff param format: should be hex string 32 bytes
fix state/stateDiff param format: should be hex string 32 bytes
…equests

fix blockOverrides: add is_tx_block flag to control block number in requests
fix stateDiff cases
rm dbg info
fix state cases
rm wait for get transaction method
…l-overrides-param

# Conflicts:
#	integration/tests/tracer/test_tracer_debug_methods.py
rm wait transaction method
Copy link
Collaborator

@kristinaNikolaevaa kristinaNikolaevaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CAN BE MERGED ONLY WITH neonlabsorg/neon-evm#425

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants