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
* @param targets address of the executor contract
*/
function batchSetExecutor(address[] memory targets)
function setExecutors(address[] memory targets)
external
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
);
function testSetExecutorValidRole() public {
function testSetExecutorsValidRole() public {
// Set single executor
address[] memory executors = new address[](1);
executors[0] = DUMMY;
vm.startPrank(EXECUTOR_SETTER);
tychoRouter.batchSetExecutor(executors);
tychoRouter.setExecutors(executors);
vm.stopPrank();
assert(tychoRouter.executors(DUMMY) == true);
}
@@ -35,7 +35,7 @@ contract TychoRouterTest is TychoRouterTestSetup {
executors[0] = DUMMY;
executors[1] = DUMMY2;
vm.startPrank(EXECUTOR_SETTER);
tychoRouter.batchSetExecutor(executors);
tychoRouter.setExecutors(executors);
vm.stopPrank();
assert(tychoRouter.executors(DUMMY) == true);
assert(tychoRouter.executors(DUMMY2) == true);
@@ -45,7 +45,7 @@ contract TychoRouterTest is TychoRouterTestSetup {
vm.startPrank(EXECUTOR_SETTER);
address[] memory executors = new address[](1);
executors[0] = DUMMY;
tychoRouter.batchSetExecutor(executors);
tychoRouter.setExecutors(executors);
tychoRouter.removeExecutor(DUMMY);
vm.stopPrank();
assert(tychoRouter.executors(DUMMY) == false);
@@ -56,11 +56,11 @@ contract TychoRouterTest is TychoRouterTestSetup {
tychoRouter.removeExecutor(BOB);
}
function testSetExecutorMissingSetterRole() public {
function testSetExecutorsMissingSetterRole() public {
vm.expectRevert();
address[] memory executors = new address[](1);
executors[0] = DUMMY;
tychoRouter.batchSetExecutor(executors);
tychoRouter.setExecutors(executors);
}
function testSetVerifierValidRole() public {

2
foundry/test/TychoRouterTestSetup.sol
View File

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