Liquidity.Party/tycho-execution
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

feat: rename batchSetExecutor to setExecutors

Browse Source
This commit is contained in:
royvardhan
2025-01-30 19:29:24 +05:30
parent 719a15c0e5
commit c653062027
3 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
foundry/src/TychoRouter.sol
View File

@@ -237,7 +237,7 @@ contract TychoRouter is
* @dev Entrypoint to add or replace an approved executor contract address * @dev Entrypoint to add or replace an approved executor contract address
* @param targets address of the executor contract * @param targets address of the executor contract
*/ */
function batchSetExecutor(address[] memory targets) function setExecutors(address[] memory targets)
external external
onlyRole(EXECUTOR_SETTER_ROLE) onlyRole(EXECUTOR_SETTER_ROLE)
{ {

12
foundry/test/TychoRouter.t.sol
View File

@@ -19,12 +19,12 @@ contract TychoRouterTest is TychoRouterTestSetup {
address indexed token, uint256 amount, address indexed receiver address indexed token, uint256 amount, address indexed receiver
); );
function testSetExecutorValidRole() public { function testSetExecutorsValidRole() public {
// Set single executor // Set single executor
address[] memory executors = new address[](1); address[] memory executors = new address[](1);
executors[0] = DUMMY; executors[0] = DUMMY;
vm.startPrank(EXECUTOR_SETTER); vm.startPrank(EXECUTOR_SETTER);
tychoRouter.batchSetExecutor(executors); tychoRouter.setExecutors(executors);
vm.stopPrank(); vm.stopPrank();
assert(tychoRouter.executors(DUMMY) == true); assert(tychoRouter.executors(DUMMY) == true);
} }
@@ -35,7 +35,7 @@ contract TychoRouterTest is TychoRouterTestSetup {
executors[0] = DUMMY; executors[0] = DUMMY;
executors[1] = DUMMY2; executors[1] = DUMMY2;
vm.startPrank(EXECUTOR_SETTER); vm.startPrank(EXECUTOR_SETTER);
tychoRouter.batchSetExecutor(executors); tychoRouter.setExecutors(executors);
vm.stopPrank(); vm.stopPrank();
assert(tychoRouter.executors(DUMMY) == true); assert(tychoRouter.executors(DUMMY) == true);
assert(tychoRouter.executors(DUMMY2) == true); assert(tychoRouter.executors(DUMMY2) == true);
@@ -45,7 +45,7 @@ contract TychoRouterTest is TychoRouterTestSetup {
vm.startPrank(EXECUTOR_SETTER); vm.startPrank(EXECUTOR_SETTER);
address[] memory executors = new address[](1); address[] memory executors = new address[](1);
executors[0] = DUMMY; executors[0] = DUMMY;
tychoRouter.batchSetExecutor(executors); tychoRouter.setExecutors(executors);
tychoRouter.removeExecutor(DUMMY); tychoRouter.removeExecutor(DUMMY);
vm.stopPrank(); vm.stopPrank();
assert(tychoRouter.executors(DUMMY) == false); assert(tychoRouter.executors(DUMMY) == false);
@@ -56,11 +56,11 @@ contract TychoRouterTest is TychoRouterTestSetup {
tychoRouter.removeExecutor(BOB); tychoRouter.removeExecutor(BOB);
} }
function testSetExecutorMissingSetterRole() public { function testSetExecutorsMissingSetterRole() public {
vm.expectRevert(); vm.expectRevert();
address[] memory executors = new address[](1); address[] memory executors = new address[](1);
executors[0] = DUMMY; executors[0] = DUMMY;
tychoRouter.batchSetExecutor(executors); tychoRouter.setExecutors(executors);
} }
function testSetVerifierValidRole() public { function testSetVerifierValidRole() public {

2
foundry/test/TychoRouterTestSetup.sol
View File

@@ -58,7 +58,7 @@ contract TychoRouterTestSetup is Test, Constants {
vm.startPrank(EXECUTOR_SETTER); vm.startPrank(EXECUTOR_SETTER);
address[] memory executors = new address[](1); address[] memory executors = new address[](1);
executors[0] = address(usv2Executor); executors[0] = address(usv2Executor);
tychoRouter.batchSetExecutor(executors); tychoRouter.setExecutors(executors);
vm.stopPrank(); vm.stopPrank();
vm.startPrank(BOB); vm.startPrank(BOB);