feat[integration-tests]: add basic stress tests#971
Conversation
🦋 Changeset detectedLatest commit: f1dc8b7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
| const value = ethers.utils.keccak256( | ||
| ethers.BigNumber.from(i).toHexString() | ||
| ) | ||
| const transaction = await env.l1Messenger.sendMessage( |
There was a problem hiding this comment.
Perhaps we want to Promise.all these messages?
|
Can you add a changeset because the integration tests are published in a docker image? |
Codecov Report
@@ Coverage Diff @@
## develop #971 +/- ##
========================================
Coverage 82.26% 82.26%
========================================
Files 49 49
Lines 1900 1900
Branches 303 303
========================================
Hits 1563 1563
Misses 337 337 Continue to review full report at Codecov.
|
4fcaf56 to
2839249
Compare
2839249 to
76a1c34
Compare
| { | ||
| gasPrice: 0, | ||
| } |
There was a problem hiding this comment.
Is it possible to run these tests with nonzero gas price? Perhaps this could be an extension issue to improve these tests.
There was a problem hiding this comment.
Not yet :-/ definitely could be an extension issue though.
integration-tests/package.json
Outdated
| "test:integration": "hardhat --network optimism test", | ||
| "test:integration:live": "IS_LIVE_NETWORK=true hardhat --network optimism-live test", | ||
| "test:sync": "hardhat --network optimism test sync-tests/*.spec.ts --no-compile", | ||
| "test:load": "hardhat --network optimism test load-tests/*.spec.ts --no-compile", |
There was a problem hiding this comment.
Maybe test:concurrency? We should be careful to not overload the term "load"
There was a problem hiding this comment.
Instead just made this file part of the standard integration tests.
76a1c34 to
34e3bfc
Compare
ab20587 to
69be710
Compare
69be710 to
f1dc8b7
Compare
Description
Adds a series of new load tests. So far I've got:
I'll likely break this out into a series of helper functions so we can compose these different actions into more interesting test cases.
Metadata