-
Notifications
You must be signed in to change notification settings - Fork 2.2k
chore: set seed for flaky tests, use etherscan api key #12255
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
Conversation
b1d8405 to
2ff1c87
Compare
cdb1ca0 to
9bd3ab5
Compare
550a89e to
3fff500
Compare
3fff500 to
d3ae9bb
Compare
| [SEQUENCE] | ||
| invariant_fork_handler_state() ([RUNS]) | ||
| [STATS] |
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.
suspect some cases where [STATS] redact doesn't work as expected, I trimmed the output only to relevant
| ... | ||
| Ran 1 test for test/InvariantScrapeValues.t.sol:FindFromReturnValueTest | ||
| [FAIL: value from return found] | ||
| [SEQUENCE] |
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.
same as above, left only relevant
| SVM_TARGET_PLATFORM: ${{ matrix.svm_target_platform }} | ||
| HTTP_ARCHIVE_URLS: ${{ secrets.HTTP_ARCHIVE_URLS }} | ||
| WS_ARCHIVE_URLS: ${{ secrets.WS_ARCHIVE_URLS }} | ||
| ETHERSCAN_KEY: ${{ secrets.ETHERSCAN_API_KEY }} |
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.
if set env var to ETHERSCAN_API_KEY then it's picked up by testdata test wich fails expected config (and other inline tests), hence used different ETHERSCAN_KEY key
Motivation
Solution
PR Checklist