Skip to content

Commit cd1d8b1

Browse files
authored
Merge pull request #68 from euler-xyz/missing-test-coverage
Missing test coverage
2 parents 17b085d + ed485c3 commit cd1d8b1

File tree

2 files changed

+29
-0
lines changed

2 files changed

+29
-0
lines changed

test/Basic.t.sol

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
pragma solidity ^0.8.24;
33

44
import {IEVault, IEulerSwap, EulerSwapTestBase, EulerSwap, TestERC20} from "./EulerSwapTestBase.t.sol";
5+
import {QuoteLib} from "../src/libraries/QuoteLib.sol";
56

67
contract Basic is EulerSwapTestBase {
78
EulerSwap public eulerSwap;
@@ -40,6 +41,14 @@ contract Basic is EulerSwapTestBase {
4041
assertEq(assetTST2.balanceOf(address(this)), amountOut);
4142
}
4243

44+
function test_badTokenAddrs() public {
45+
vm.expectRevert(QuoteLib.UnsupportedPair.selector);
46+
periphery.quoteExactInput(address(eulerSwap), address(assetTST), address(1234), 0);
47+
48+
vm.expectRevert(QuoteLib.UnsupportedPair.selector);
49+
periphery.quoteExactInput(address(eulerSwap), address(1234), address(assetTST), 0);
50+
}
51+
4352
function test_altPrice() public {
4453
uint256 price = 0.5e18;
4554
uint256 px = price;

test/HookSwaps.t.sol

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -126,6 +126,26 @@ contract HookSwapsTest is EulerSwapTestBase {
126126
vm.stopPrank();
127127
}
128128

129+
function test_hookPermissions() public view {
130+
Hooks.Permissions memory perms = eulerSwap.getHookPermissions();
131+
132+
assertTrue(perms.beforeInitialize);
133+
assertTrue(perms.beforeAddLiquidity);
134+
assertTrue(perms.beforeSwap);
135+
assertTrue(perms.beforeSwapReturnDelta);
136+
137+
assertFalse(perms.afterInitialize);
138+
assertFalse(perms.afterAddLiquidity);
139+
assertFalse(perms.beforeRemoveLiquidity);
140+
assertFalse(perms.afterRemoveLiquidity);
141+
assertFalse(perms.afterSwap);
142+
assertFalse(perms.beforeDonate);
143+
assertFalse(perms.afterDonate);
144+
assertFalse(perms.afterSwapReturnDelta);
145+
assertFalse(perms.afterAddLiquidityReturnDelta);
146+
assertFalse(perms.afterRemoveLiquidityReturnDelta);
147+
}
148+
129149
/// @dev adding liquidity as a concentrated liquidity position will revert
130150
function test_revertAddConcentratedLiquidity() public {
131151
assetTST.mint(anyone, 10000e18);

0 commit comments

Comments
 (0)