Add regression test #24
Open
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
|
please add a test for all of the existing mainnetConfig.toml files so we can regression test all of these templates. |
* remove: name and description from task config Signed-off-by: Elliot <[email protected]> add example task template config files Signed-off-by: Elliot <[email protected]> add fps example templates Signed-off-by: Elliot <[email protected]> remove name and description, set task nonce to current multisig nonce Signed-off-by: Elliot <[email protected]> fmt and add TODO to switch to manual nonce later Signed-off-by: Elliot <[email protected]> templates Signed-off-by: Elliot <[email protected]> remove safeNonce from mock network config Signed-off-by: Elliot <[email protected]> * checkpoint Signed-off-by: Elliot <[email protected]> * remove isNested flag from toml config file Signed-off-by: Elliot <[email protected]> * fmt Signed-off-by: Elliot <[email protected]> * fix is nested safe logic * fix task01 * add Task02 * refactor _getDataToSign function Signed-off-by: Elliot <[email protected]> * add: gas template docs Signed-off-by: Elliot <[email protected]> * add: template examples to run Signed-off-by: Elliot <[email protected]> * add: error handle parsing l2 chain configs Signed-off-by: Elliot <[email protected]> * fmt Signed-off-by: Elliot <[email protected]> * remove unused variable name Signed-off-by: Elliot <[email protected]> * error handle reading in and parsing task config file Signed-off-by: Elliot <[email protected]> * add breakdown of calldata difference for signers Signed-off-by: Elliot <[email protected]> * remove setting scalars to simplify example task Signed-off-by: Elliot <[email protected]> * add duplicate call structure of task 020 in existing tooling Signed-off-by: Elliot <[email protected]> * remove setting gas scalars from gas config template Signed-off-by: Elliot <[email protected]> * note on commenting out failing check and link to diffchecker Signed-off-by: Elliot <[email protected]> * add: task config tests Signed-off-by: Elliot <[email protected]> * add forge build before running template Signed-off-by: Elliot <[email protected]> * update docs to only include gas limits and not gas scalars Signed-off-by: Elliot <[email protected]> * remove template ci runs Signed-off-by: Elliot <[email protected]> * Update src/fps/FPS.md Co-authored-by: blaine <[email protected]> * Update src/fps/FPS.md Co-authored-by: blaine <[email protected]> * Update src/fps/FPS.md * fps -> new task simulation tooling Signed-off-by: Elliot <[email protected]> * remove unused template files Signed-off-by: Elliot <[email protected]> * naming: fps.md -> readme.md Signed-off-by: Elliot <[email protected]> * naming: move away from fps Signed-off-by: Elliot <[email protected]> * use op types, remove inlined interfaces Signed-off-by: Elliot <[email protected]> * update readme for running tasks Signed-off-by: Elliot <[email protected]> * update: cleanup, make MultisigTask a generic template Signed-off-by: Elliot <[email protected]> * fix test Signed-off-by: Elliot <[email protected]> * delete: task config Signed-off-by: Elliot <[email protected]> * update based on PR feedback, remove taskConfig.toml fields, and safeNonce and isNestedSafe Signed-off-by: Elliot <[email protected]> * remove unused mock network configs Signed-off-by: Elliot <[email protected]> * remove opmainnetConfig.toml Signed-off-by: Elliot <[email protected]> * add op set gas limit config Signed-off-by: Elliot <[email protected]> * update command to use new run function Signed-off-by: Elliot <[email protected]> * update command to only have mainnet config toml file Signed-off-by: Elliot <[email protected]> * naming: networkConfigFilePath -> taskConfigFilePath Signed-off-by: Elliot <[email protected]> * fmt Signed-off-by: Elliot <[email protected]> * natspec, taskStorageWrites -> _taskStorageWrites Signed-off-by: Elliot <[email protected]> * fix test Signed-off-by: Elliot <[email protected]> * add additional docs for address registry Signed-off-by: Elliot <[email protected]> * fix: nested multisig issue Signed-off-by: Elliot <[email protected]> * add op gas config command Signed-off-by: Elliot <[email protected]> * add task tests * Revert "add task tests" This reverts commit 70ed74f. * add tests * fix: nonce for nested safes use their own nonce and not parent Signed-off-by: Elliot <[email protected]> * update natspec proposal -> task Signed-off-by: Elliot <[email protected]> * fix failing single multisig tests * cleanup debug logs * fix address registry failing test * add test for nested hash to approve * rename public function * remove build warnings * pr feedback * Update src/fps/task/MultisigTask.sol Co-authored-by: Elliot <[email protected]> * pr feedback * fix: virtual keyword to allow template variants * address pr feedback: multicall import Signed-off-by: Elliot <[email protected]> * delete testnetConfig.toml Signed-off-by: Elliot <[email protected]> * add: natspec comment on taskConfigFilePath var Signed-off-by: Elliot <[email protected]> * address feedback: add comments Signed-off-by: Elliot <[email protected]> --------- Signed-off-by: Elliot <[email protected]> Co-authored-by: Elliot <[email protected]> Co-authored-by: Elliot <[email protected]> Co-authored-by: blaine <[email protected]>
* execute sep-028 * fix: remove CI task. --------- Co-authored-by: Blaine Malone <[email protected]>
…sm#513) * fix: Better validation of storage layout with bit packing * fix: images dir directly in task. * fix: attesting in docs to additional values.
prateek105
force-pushed
the
feat/regression-tests
branch
from
6e9c50f to
130bc50
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.
No description provided.