Skip to content

Commit

Permalink
refactor(single-auto-line-wipe): remove Wipe event parameters
Browse files Browse the repository at this point in the history
  • Loading branch information
amusingaxl committed Jun 25, 2024
1 parent 2eeec5b commit cfafa6b
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
6 changes: 3 additions & 3 deletions src/auto-line-wipe/SingleAutoLineWipeSpell.sol
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ contract SingleAutoLineWipeSpell is DssEmergencySpell {
LineMomLike public immutable lineMom = LineMomLike(_log.getAddress("LINE_MOM"));
bytes32 public immutable ilk;

event Wipe(bytes32 indexed ilk, uint256 prevLine);
event Wipe();

constructor(bytes32 _ilk) {
ilk = _ilk;
Expand All @@ -44,8 +44,8 @@ contract SingleAutoLineWipeSpell is DssEmergencySpell {
}

function _emergencyActions() internal override {
uint256 prevLine = lineMom.wipe(ilk);
emit Wipe(ilk, prevLine);
lineMom.wipe(ilk);
emit Wipe();
}

/**
Expand Down
5 changes: 2 additions & 3 deletions src/auto-line-wipe/SingleAutoLineWipeSpell.t.integration.sol
Original file line number Diff line number Diff line change
Expand Up @@ -62,8 +62,7 @@ contract SingleAutoLineWipeSpellTest is DssTest {
assertFalse(spell.done(), "before: spell already done");

vm.expectEmit(true, true, true, false);
// Ignore prevLine for now
emit Wipe(ilk, 0);
emit Wipe();
spell.schedule();

(uint256 maxLine, uint256 gap,,,) = autoLine.ilks(ilk);
Expand All @@ -89,5 +88,5 @@ contract SingleAutoLineWipeSpellTest is DssTest {
assertFalse(spell.done(), "after: spell done unexpectedly");
}

event Wipe(bytes32 indexed autoLine, uint256 prevLine);
event Wipe();
}

0 comments on commit cfafa6b

Please sign in to comment.