Revert "Emit event for contract registration (#1518)" #1528
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
* evm wip * Create EVM module and add types/keeper * lint * rebase * tests * add tests * address comments
* evm wip * Create EVM module and add types/keeper * add tests * EVM signature verification * refactor
* [EVM] Add gas check ante handler * add tests and remove unnecessary configs/versions
* [EVM] implement balance related methods for state interface * add tests
* [EVM] Add gas check ante handler * [EVM] implement balance related methods for state interface * Add account state to EVM state interface implementation * add Snapshot and rollback * wip * [EVM] add remaining state interface function * rebase
* Add account state to EVM state interface implementation * add Snapshot and rollback * wip * [EVM] add remaining state interface function * [EVM] access list * add unit tests * improve code coverage --------- Co-authored-by: Jeremy Wei <[email protected]>
* [EVM] implement message server that processes EVM transactions * add tests
* [EVM] implement message server that processes EVM transactions * [EVM] add bank precompile * upgrade to Go 1.20 * rename coinbase * tests & linter
* [EVM] implement message server that processes EVM transactions * rename coinbase * [EVM] add RPC/WS server helpers * tests * fix
* [EVM] implement message server that processes EVM transactions * rename coinbase * [EVM] add RPC/WS server helpers * tests * wip * [EVM] add getBlockByHash endpoint * rebase
[EVM] getBlockByNumber
[EVM] add endpoint to get block transaction count
* [EVM] add get transaction receipt endpoint * refactor tests
[EVM] add getTransaction endpoints
[EVM] refactor gas logic
* [EVM] add endpoint to return fee history * bump to 1.21
* Implement get code, storage, balance endpoints * revert logging change * also using bank keeper to check for balances * appease linter
* simulate api * add tests * linter
* [EVM] add subscription utility * finished new filter * finished tests for new filter * finished uninstall filter * query tendermint for events * in progress * got first couple of tests done * finished tests for get logs * get filter logs and get filter changes * refactor * get filter logs accepts multiple addresses * use multiple addresses and use cursor map * remove bad test * remove prints and todos * minor fix * fix lint errors and failing test * poc to pass filtercriteria into JSON RPC * made topics 2d array and adjusted regex * GetLogs tests work * Finished other functions * remove unncessary inputs from NewFilterAPI * add expiration/timeouts for filters * removed error from UninstallFilter * minor fix * appease linter --------- Co-authored-by: codchen <[email protected]>
* [EVM] return gas info in message handler correctly * add comments
* integration * fix tests
* [EVM] add transaction type to associate account * tests
* [EVM] add transaction type to associate account * [EVM] convert balance decimal points * [EVM] add ACL dependencies for EVM transactions * Revert "[EVM] add ACL dependencies for EVM transactions" This reverts commit 5376a87. * address comments
* added new block filter * wip * finished tests * minor fix * fix lint * minor fix * fix test
evm Basic Ante during delivery
Update Precompile Input Method + Evm Balance
Add More Precompile Validation
* Precompiles Raise Error over Panic * Update test * update test
* not following eip-4788 * add run_blocktests script * add initialize_local_chain_no_run script * add prints * add more prints * made min fee per gas 0 * Revert "add more prints" This reverts commit 3c940e1. * Revert "add prints" This reverts commit fd28389. * 2 new failing tests * use initialize local chain with NO_RUN flag * Revert "Revert "add more prints"" This reverts commit 4c7603a. * Revert "Revert "add prints"" This reverts commit 98ea147. * increment height in getHash and ignore withdraw addrs * update failed tests * add eth blocktests * adjust git clone * Revert "Revert "Revert "add more prints""" This reverts commit 1e0aede. * Revert "Revert "Revert "add prints""" This reverts commit 3764122. * Split eth blocktests in CI * Modify ethtests path * refactor output * minor fix * add debug statement * Add more stuff * wrap filesnames with quotes * github workflow split tests * fix ethtests script * fix ethtests script * fix ethtests script * fix ethtests script * adjust for invalid block tests * a bunch of invalid block tests seem to work * run invalid block tests in ci * add logging * sort block tests * reduce prints * reduce prints * now getting real errors * cleanup * cleanup * not run blockWithAllTransactionTypes * Revert "not run blockWithAllTransactionTypes" - this should work This reverts commit 51aa2b1. * not run blockWithAllTransactionTypes --------- Co-authored-by: Uday Patil <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## seiv2 #1528 +/- ##
==========================================
- Coverage 60.21% 60.15% -0.07%
==========================================
Files 363 363
Lines 25797 25765 -32
==========================================
- Hits 15533 15498 -35
- Misses 9277 9278 +1
- Partials 987 989 +2
|
codchen
force-pushed
the
try-revert-emit
branch
4 times, most recently
from
afb1633 to
7ef488e
Compare
This reverts commit e2440de.
codchen
force-pushed
the
try-revert-emit
branch
from
7ef488e to
355bd5b
Compare
udpatil
force-pushed
the
seiv2
branch
2 times, most recently
from
aa0aca9 to
4eea060
Compare
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.
This reverts commit e2440de.
Describe your changes and provide context
Testing performed to validate your change