From 24e95b1206d403a9c7b2c82b12bd14c3fc7ee6c4 Mon Sep 17 00:00:00 2001 From: royvardhan Date: Thu, 30 Jan 2025 21:01:28 +0530 Subject: [PATCH] fix: rm redundant test --- foundry/test/Constants.sol | 2 ++ foundry/test/TychoRouter.t.sol | 12 +++++------- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/foundry/test/Constants.sol b/foundry/test/Constants.sol index 8cad6fb..c1b5d57 100644 --- a/foundry/test/Constants.sol +++ b/foundry/test/Constants.sol @@ -17,6 +17,7 @@ contract Constants is Test { // Dummy contracts address DUMMY = makeAddr("dummy"); address DUMMY2 = makeAddr("dummy2"); + address DUMMY3 = makeAddr("dummy3"); address PAUSER = makeAddr("pauser"); address UNPAUSER = makeAddr("unpauser"); @@ -41,5 +42,6 @@ contract Constants is Test { // Deploy minimal bytecode vm.etch(DUMMY, minimalBytecode); vm.etch(DUMMY2, minimalBytecode); + vm.etch(DUMMY3, minimalBytecode); } } diff --git a/foundry/test/TychoRouter.t.sol b/foundry/test/TychoRouter.t.sol index 5fa72ac..5d8ebea 100644 --- a/foundry/test/TychoRouter.t.sol +++ b/foundry/test/TychoRouter.t.sol @@ -27,18 +27,16 @@ contract TychoRouterTest is TychoRouterTestSetup { tychoRouter.setExecutors(executors); vm.stopPrank(); assert(tychoRouter.executors(DUMMY) == true); - } - function testSetExecutorsValidRole() public { // Set multiple executors - address[] memory executors = new address[](2); - executors[0] = DUMMY; - executors[1] = DUMMY2; + address[] memory executors2 = new address[](2); + executors2[0] = DUMMY2; + executors2[1] = DUMMY3; vm.startPrank(EXECUTOR_SETTER); - tychoRouter.setExecutors(executors); + tychoRouter.setExecutors(executors2); vm.stopPrank(); - assert(tychoRouter.executors(DUMMY) == true); assert(tychoRouter.executors(DUMMY2) == true); + assert(tychoRouter.executors(DUMMY3) == true); } function testRemoveExecutorValidRole() public {