-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Add tests for convertInternalLogsToOvmLogs #21
Conversation
Other changes: * Add a seperate function for getting ovm transaction metadata * Move CallingWithEOA into the EOA function and add an argument
…m/TestEventParsing
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.
🎉
packages/ovm/package.json
Outdated
@@ -10,7 +10,7 @@ | |||
"scripts": { | |||
"all": "yarn clean && yarn build && yarn test && yarn fix && yarn lint", | |||
"test:govm": "waffle waffle-config.json && mocha --require source-map-support/register --require ts-node/register 'test/govm/*.spec.ts' --timeout 60000 --exit", | |||
"test": "waffle waffle-config.json && mocha --require source-map-support/register --require ts-node/register 'test/contracts/*.spec.ts' --timeout 8000 --exit", | |||
"test": "waffle waffle-config.json && mocha --require source-map-support/register --require ts-node/register 'test/contracts/*.spec.ts' 'test/app/*.spec.ts' --timeout 8000 --exit", |
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.
Can we use test/**/*.spec.ts
instead of writing both out?
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.
Unfortunately we can't match on everything in test
because that would match test/govm
which we want to run separately with test:govm
. We might be able to remove govm
tests in a separate PR if we decide to table govm
for the time being.
I did at least combine app and contracts into one though 👍
@@ -41,7 +41,9 @@ contract ExecutionManager is FullStateManager { | |||
address _codeContractAddress, | |||
bytes32 _codeContractHash | |||
); | |||
event CallingWithEOA(); | |||
event CallingWithEOA( | |||
address _ovmFromAddress |
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.
👍
* pkg: import l2geth * chore: add l2geth tests submodule * ci: unit test l2geth * chore: skip Android tests * chore: cleanup ci
* fix jumpi inputs ordering * fix comment
* pkg: import l2geth * chore: add l2geth tests submodule * ci: unit test l2geth * chore: skip Android tests * chore: cleanup ci
…-stopped feat: add API admin_sequencerStopped
* op-batcher: Add metrics for pending L2 transaction data size (ethereum-optimism#5797) * feat(op-node): Finalize Mainnet Rollup Config [release branch] (ethereum-optimism#5905) * copy over develop chainsgo * stage rollup config changes * final rollup config values * fix(op-batcher): solve race condition of BatchSubmitter publishTxToL1 and handleReceipt access state concurrently (#5) * chore: update readme, add testnet assets (ethereum-optimism#9) * chore: update readme, add testnet assets * doc: clarify readme * ci: add the ci code used to package and release docker images (ethereum-optimism#11) * ci: add the ci code used to package and release docker images (#7) * ci: add the ci code used to package and release docker images Co-authored-by: Welkin <welkin.b@nodereal.com> * fix: add latest tag for docker image (ethereum-optimism#9) Co-authored-by: Welkin <welkin.b@nodereal.com> * try to use cache for docker build (ethereum-optimism#10) Co-authored-by: Welkin <welkin.b@nodereal.com> --------- Co-authored-by: Welkin <welkin.b@nodereal.com> * feat(ci): add ci workflow * fix * fix * update * update * skip fail test temporarily * add batcher/proposer * add e2e * skip fail e2e case temporary * add op-node-lint * fix lint * add batcher/proposer lint * test junit-report * add junit report for all * adjust parallel to 2 for avoiding fail * add needs for job serial execution * use testname format to simplify result --------- Co-authored-by: Joshua Gutow <jgutow@oplabs.co> Co-authored-by: refcell.eth <abigger87@gmail.com> Co-authored-by: bnoieh <135800952+bnoieh@users.noreply.github.com> Co-authored-by: Owen <owen.h@nodereal.io> Co-authored-by: Welkin <welkin.b@nodereal.com>
Other changes:
CallingWithEOA
event into theCallEOA
function and add an argument to simplify event parsingMetadata
Fixes
Contributing Agreement