Skip to content

Commit

Permalink
fix: mintShare uint8 param
Browse files Browse the repository at this point in the history
  • Loading branch information
0xashu committed May 21, 2024
1 parent 4ab097f commit 70167fc
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
2 changes: 0 additions & 2 deletions test/BaseTest.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,4 @@ contract BaseTest is Test {
sharesFactory.transferOwnership(owner);
aaveYieldAggregator.transferOwnership(owner);
}

function testSuccess() public { }
}
6 changes: 1 addition & 5 deletions test/unit/SharesFactory.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ contract SharesFactoryTests is BaseTest {
assertEq(curveType, defaultCurveType);

vm.expectRevert(bytes("Invalid curveType"));
sharesFactory.mintShare(999);
sharesFactory.mintShare(99);
}

function test_minAndBuyShare() public {
Expand Down Expand Up @@ -462,10 +462,6 @@ contract SharesFactoryTests is BaseTest {
uint256 yieldBalance = yieldAggregator.yieldBalanceOf(address(sharesFactory));
uint256 yieldMaxClaimable = yieldAggregator.yieldMaxClaimable(depositedETHAmount);
uint256 yieldBuffer = 1e12;
// uint256 yieldBuffer = yieldAggregator.yieldBuffer();
console.log("depositedETHAmount: ", depositedETHAmount);
console.log("yieldBalance: ", yieldBalance);
console.log("yieldMaxClaimable: ", yieldMaxClaimable);
return (depositedETHAmount, yieldBalance, yieldMaxClaimable, yieldBuffer);
}
}

0 comments on commit 70167fc

Please sign in to comment.