Skip to content

Commit

Permalink
Bug! Fix indexed event argument checking
Browse files Browse the repository at this point in the history
  • Loading branch information
duncancmt committed Jan 15, 2024
1 parent 59dbdc9 commit 9420cf3
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/deployer/Deployer.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ contract DeployerTest is Test {

function testFeeCollector() public {
assertEq(deployer.feeCollector(1), address(0));
vm.expectEmit(true, false, false, false, address(deployer));
vm.expectEmit(true, true, false, false, address(deployer));
emit FeeCollectorChanged(1, address(this));
assertTrue(deployer.setFeeCollector(1, address(this)));
assertEq(deployer.feeCollector(1), address(this));
Expand All @@ -102,7 +102,7 @@ contract DeployerTest is Test {
deployer.authorize(1, address(this), uint96(block.timestamp + 1 days));
deployer.setFeeCollector(1, auth);
address predicted = AddressDerivation.deriveContract(address(deployer), 1);
vm.expectEmit(true, true, false, false, address(deployer));
vm.expectEmit(true, true, true, false, address(deployer));
emit Deployed(1, 1, predicted);
vm.expectEmit(true, true, true, false, address(deployer));
emit Transfer(address(0), predicted, 1);
Expand Down Expand Up @@ -174,7 +174,7 @@ contract DeployerTest is Test {

vm.expectEmit(true, true, true, false, address(deployer));
emit Transfer(AddressDerivation.deriveContract(address(deployer), 1), address(0), 1);
vm.expectEmit(true, true, false, false, address(deployer));
vm.expectEmit(true, true, true, false, address(deployer));
emit Unsafe(1, 1, instance);
assertTrue(deployer.setUnsafe(1, nonce));
vm.expectRevert(abi.encodeWithSignature("NoToken(uint256)", 1));
Expand Down

0 comments on commit 9420cf3

Please sign in to comment.