makerdao / spells-mainnet

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

PE-751: 2021-11-19 Executive #203

Closed naszam closed 2 years ago

naszam commented 2 years ago

Description

Contribution Checklist

Checklist

gbalabasquer commented 2 years ago

@naszam new max mats need to be set in the tests for LRC-A, BAT-A and ZRX-A

gbalabasquer commented 2 years ago

Spell is ready to be deployed

gbalabasquer commented 2 years ago
gbalabasquer commented 2 years ago

@naszam I don't think I'll get the tests passing locally before having to be off for a while. So if you get confirmation the tests are passing I give my approval to share the spell address.

naszam commented 2 years ago

Test PASS (with deployed spell address at block n. 13646843):

dapp-test: rpc block: latest
Running 22 tests for src/DssSpell.t.sol:DssSpellTest
[PASS] testBATIncreasedMatLerpOffboarding() (gas: 5324172)
[PASS] testFail_notScheduled() (gas: 12347)
[PASS] testLRCIncreasedMatLerpOffboarding() (gas: 5324238)
[PASS] test_auth_in_sources() (gas: 281473904573849)
[PASS] test_use_eta() (gas: 334015)
[PASS] testZRXIncreasedMatLerpOffboarding() (gas: 5324370)
[PASS] testOnTime() (gas: 5268536)
[PASS] test_nextCastTime() (gas: 402313)
[PASS] testFailTooEarly() (gas: 373791)
[PASS] testCOMPLerpOffboardings() (gas: 5317821)
[PASS] test_bytecode_matches() (gas: 2801218)
[PASS] testAAVELerpOffboardings() (gas: 5317909)
[PASS] test_auth() (gas: 281473909822607)
[PASS] testSpellIsCast_GENERAL() (gas: 16263988)
[PASS] testFailWrongDay() (gas: 373855)
[PASS] testCollateralIntegrations() (gas: 6179488)
[PASS] testCastCost() (gas: 5270452)
[PASS] testUNIV2LINKETHIncreasedMatLerpOffboarding() (gas: 5324854)
[PASS] testNewChainlogValues() (gas: 5279316)
[PASS] testNewIlkRegistryValues() (gas: 5289530)
[PASS] testBALLerpOffboardings() (gas: 5318217)
[PASS] testFailTooLate() (gas: 374140)
gbalabasquer commented 2 years ago

Tests passed locally:

Running 22 tests for src/DssSpell.t.sol:DssSpellTest
[PASS] testBATIncreasedMatLerpOffboarding() (gas: 5324172)
[PASS] testFail_notScheduled() (gas: 12347)
[PASS] testLRCIncreasedMatLerpOffboarding() (gas: 5324238)
[PASS] test_auth_in_sources() (gas: 281473904573849)
[PASS] test_use_eta() (gas: 334015)
[PASS] testZRXIncreasedMatLerpOffboarding() (gas: 5324370)
[PASS] testOnTime() (gas: 5268536)
[PASS] test_nextCastTime() (gas: 402313)
[PASS] testFailTooEarly() (gas: 373791)
[PASS] testCOMPLerpOffboardings() (gas: 5317821)
[PASS] test_bytecode_matches() (gas: 2801218)
[PASS] testAAVELerpOffboardings() (gas: 5317909)
[PASS] test_auth() (gas: 281473909822607)
[PASS] testSpellIsCast_GENERAL() (gas: 16263988)
[PASS] testFailWrongDay() (gas: 373855)
[PASS] testCollateralIntegrations() (gas: 6179488)
[PASS] testCastCost() (gas: 5270452)
[PASS] testUNIV2LINKETHIncreasedMatLerpOffboarding() (gas: 5324854)
[PASS] testNewChainlogValues() (gas: 5279316)
[PASS] testNewIlkRegistryValues() (gas: 5289530)
[PASS] testBALLerpOffboardings() (gas: 5318217)
[PASS] testFailTooLate() (gas: 374140)

Please archive and will give the approval. But so far the spell can be sent to GovAlpha.