makerdao / spells-mainnet

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

PE-1013-20220615: Urgent Weekly Executive 2022-06-15 #252

Closed godsflaw closed 2 years ago

godsflaw commented 2 years ago

Urgent Weekly Executive 2022-06-15

Contribution Checklist

Checklist

julienmartinlevrai commented 2 years ago

Spell looks good except for the copy hash.

We could remove the spell action and just use the custom DssExec. It would be kind of cool and would save gas, but is not necessary.

godsflaw commented 2 years ago
Running 15 tests for src/DssSpell.t.sol:DssSpellTest
[PASS] testAAVEDirectBarChange() (gas: 721281)
[PASS] testCastCost() (gas: 390844)
[PASS] testFailTooEarly() (gas: 12550)
[PASS] testFailTooLate() (gas: 12549)
[PASS] testFailWrongDay() (gas: 12594)
[PASS] testFail_notScheduled() (gas: 14139)
[PASS] testOnTime() (gas: 387233)
[PASS] testSpellIsCast_GENERAL() (gas: 22187701)
[PASS] test_auth() (gas: 9223336852494766489)
[PASS] test_auth_in_sources() (gas: 9223336852484704559)
[PASS] test_bytecode_matches() (gas: 1130857)
[PASS] test_chainlog_values() (gas: 5448349)
[PASS] test_chainlog_version_bump() (gas: 2813999)
[PASS] test_nextCastTime() (gas: 368090)
[PASS] test_use_eta() (gas: 366900)
Test result: ok. 15 passed; 0 failed; finished in 8.26s
godsflaw commented 2 years ago

Deployed spell: 0x76b686379234C90b2d9d4A9412A359783273f6b2 etherscan: https://etherscan.io/address/0x76b686379234C90b2d9d4A9412A359783273f6b2#code

Running 15 tests for src/DssSpell.t.sol:DssSpellTest
[PASS] testAAVEDirectBarChange() (gas: 721281)
[PASS] testCastCost() (gas: 390844)
[PASS] testFailTooEarly() (gas: 12550)
[PASS] testFailTooLate() (gas: 12549)
[PASS] testFailWrongDay() (gas: 12594)
[PASS] testFail_notScheduled() (gas: 14139)
[PASS] testOnTime() (gas: 387233)
[PASS] testSpellIsCast_GENERAL() (gas: 23305420)
[PASS] test_auth() (gas: 9223336852494766489)
[PASS] test_auth_in_sources() (gas: 9223336852484704559)
[PASS] test_bytecode_matches() (gas: 1130857)
[PASS] test_chainlog_values() (gas: 5448349)
[PASS] test_chainlog_version_bump() (gas: 2813999)
[PASS] test_nextCastTime() (gas: 368090)
[PASS] test_use_eta() (gas: 366900)
Test result: ok. 15 passed; 0 failed; finished in 8.60s