makerdao / spells-mainnet

Staging repo for MakerDAO weekly executive spells
GNU Affero General Public License v3.0
120 stars 47 forks source link

PE-1045 #269

Closed hexonaut closed 2 years ago

hexonaut commented 2 years ago

Description

PLEASE NOTE THE TESTS ONLY WORK IN FOUNDRY / FORGE.

Contribution Checklist

Checklist

hexonaut commented 2 years ago

This is ready for review except for hash.

hexonaut commented 2 years ago

Ok this is ready for final review for deploy.

talbaneth commented 2 years ago

Unless we want to wait for goerli manual testing this LGTM to publish and archive.

Spell verified, license ok, compiler and flags ok, code matches repo, correct execlib address used. tests pass:

Running 1 test for src/test/starknet.t.sol:StarknetTests [PASS] testStarknet() (gas: 2888546) Test result: ok. 1 passed; 0 failed; finished in 11.56s

Running 19 tests for src/DssSpell.t.sol:DssSpellTest [PASS] testCastCost() (gas: 2682535) [PASS] testCureTeleport() (gas: 4432575) [PASS] testFailTooEarly() (gas: 411191) [PASS] testFailTooLate() (gas: 411336) [PASS] testFailWrongDay() (gas: 411222) [PASS] testFail_notScheduled() (gas: 14147) [PASS] testNewChainlogValues() (gas: 2849386) [PASS] testNewIlkRegistryValues() (gas: 2706013) [PASS] testOnTime() (gas: 2678749) [PASS] testRWA009_SPELL_DRAW() (gas: 2702842) [PASS] testSpellIsCast_GENERAL() (gas: 32758648) [PASS] testTeleportFW() (gas: 4707111) [PASS] test_auth() (gas: 9223336852500860092) [PASS] test_auth_in_sources() (gas: 9223336852485195825) [PASS] test_bytecode_matches() (gas: 3846848) [PASS] test_chainlog_values() (gas: 8517564) [PASS] test_chainlog_version_bump() (gas: 4759365) [PASS] test_nextCastTime() (gas: 420135) [PASS] test_use_eta() (gas: 347239) Test result: ok. 19 passed; 0 failed; finished in 37.49s

hexonaut commented 2 years ago

Archive matches. Good to merge.