-
Notifications
You must be signed in to change notification settings - Fork 79
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add fork tests on snapshot #2187
Open
shahthepro
wants to merge
11
commits into
master
Choose a base branch
from
shah/snapshot-fork-test
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
11 commits
Select commit
Hold shift + click to select a range
0043343
basic setup for the strategy contract
sparrowDom b1e9311
Merge remote-tracking branch 'origin/master' into sparrowDom/strategyAMO
sparrowDom bbff69c
create proof of concept to simulate required amount to swap consideri…
sparrowDom 3b7bd5f
add strategist and simplifty the amo code
sparrowDom 18d3e11
for testing purposes create a gauge in the fixture
sparrowDom 1594be5
deposit and withdraw from gauges now works
sparrowDom 91ac555
add a setter function for pool weth share variance allowed
sparrowDom 092ddff
Merge branch 'master' into sparrowDom/strategyAMO
shahthepro 04bc306
fix typo
sparrowDom ce6db27
refactor stuff from initialize to the constructor
sparrowDom f802d17
Add fork tests on snapshot
shahthepro File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -289,6 +289,49 @@ jobs: | |
./contracts/coverage.json | ||
./contracts/coverage/**/* | ||
retention-days: 1 | ||
|
||
contracts-base-snapshot-forktest: | ||
name: "Base Fork Tests (Fixed Snapshot)" | ||
runs-on: ubuntu-latest | ||
continue-on-error: true | ||
env: | ||
HARDHAT_CACHE_DIR: ./cache | ||
PROVIDER_URL: ${{ secrets.PROVIDER_URL }} | ||
BASE_PROVIDER_URL: ${{ secrets.BASE_PROVIDER_URL }} | ||
TEST_ON_SNAPSHOT: "true" | ||
# Hardcoded number to be changed later | ||
BASE_BLOCK_NUMBER: 18346074 | ||
steps: | ||
- uses: actions/checkout@v4 | ||
|
||
- name: Use Node.js | ||
uses: actions/setup-node@v4 | ||
with: | ||
node-version: "20.x" | ||
cache: "yarn" | ||
cache-dependency-path: contracts/yarn.lock | ||
|
||
- uses: actions/cache@v3 | ||
id: hardhat-cache | ||
with: | ||
path: contracts/cache | ||
key: ${{ runner.os }}-hardhat-${{ hashFiles('contracts/cache/*.json') }} | ||
restore-keys: | | ||
${{ runner.os }}-hardhat-cache | ||
|
||
- run: yarn install --frozen-lockfile | ||
working-directory: ./contracts | ||
|
||
- run: yarn run test:coverage:base-fork | ||
working-directory: ./contracts | ||
|
||
- uses: actions/upload-artifact@v3 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The latest version is actions/upload-artifact@v4 |
||
with: | ||
name: fork-test-base-coverage-${{ github.sha }} | ||
path: | | ||
./contracts/coverage.json | ||
./contracts/coverage/**/* | ||
retention-days: 1 | ||
|
||
coverage-uploader: | ||
name: "Upload Coverage Reports" | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,65 @@ | ||
// SPDX-License-Identifier: MIT | ||
pragma solidity >=0.8.0; | ||
|
||
interface ICLGauge { | ||
/// @notice Returns the claimable rewards for a given account and tokenId | ||
/// @dev Throws if account is not the position owner | ||
/// @dev pool.updateRewardsGrowthGlobal() needs to be called first, to return the correct claimable rewards | ||
/// @param account The address of the user | ||
/// @param tokenId The tokenId of the position | ||
/// @return The amount of claimable reward | ||
function earned(address account, uint256 tokenId) external view returns (uint256); | ||
|
||
/// @notice Retrieve rewards for all tokens owned by an account | ||
/// @dev Throws if not called by the voter | ||
/// @param account The account of the user | ||
function getReward(address account) external; | ||
|
||
/// @notice Retrieve rewards for a tokenId | ||
/// @dev Throws if not called by the position owner | ||
/// @param tokenId The tokenId of the position | ||
function getReward(uint256 tokenId) external; | ||
|
||
/// @notice Notifies gauge of gauge rewards. | ||
/// @param amount Amount of gauge rewards (emissions) to notify. Must be greater than 604_800. | ||
function notifyRewardAmount(uint256 amount) external; | ||
|
||
/// @dev Notifies gauge of gauge rewards without distributing its fees. | ||
/// Assumes gauge reward tokens is 18 decimals. | ||
/// If not 18 decimals, rewardRate may have rounding issues. | ||
/// @param amount Amount of gauge rewards (emissions) to notify. Must be greater than 604_800. | ||
function notifyRewardWithoutClaim(uint256 amount) external; | ||
|
||
/// @notice Used to deposit a CL position into the gauge | ||
/// @notice Allows the user to receive emissions instead of fees | ||
/// @param tokenId The tokenId of the position | ||
function deposit(uint256 tokenId) external; | ||
|
||
/// @notice Used to withdraw a CL position from the gauge | ||
/// @notice Allows the user to receive fees instead of emissions | ||
/// @notice Outstanding emissions will be collected on withdrawal | ||
/// @param tokenId The tokenId of the position | ||
function withdraw(uint256 tokenId) external; | ||
|
||
// /// @notice Fetch all tokenIds staked by a given account | ||
// /// @param depositor The address of the user | ||
// /// @return The tokenIds of the staked positions | ||
// function stakedValues(address depositor) external view returns (uint256[] memory); | ||
|
||
// /// @notice Fetch a staked tokenId by index | ||
// /// @param depositor The address of the user | ||
// /// @param index The index of the staked tokenId | ||
// /// @return The tokenId of the staked position | ||
// function stakedByIndex(address depositor, uint256 index) external view returns (uint256); | ||
|
||
// /// @notice Check whether a position is staked in the gauge by a certain user | ||
// /// @param depositor The address of the user | ||
// /// @param tokenId The tokenId of the position | ||
// /// @return Whether the position is staked in the gauge | ||
// function stakedContains(address depositor, uint256 tokenId) external view returns (bool); | ||
|
||
// /// @notice The amount of positions staked in the gauge by a certain user | ||
// /// @param depositor The address of the user | ||
// /// @return The amount of positions staked in the gauge | ||
// function stakedLength(address depositor) external view returns (uint256); | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
pragma solidity >=0.5.0; | ||
|
||
/// @title The interface for a CL Pool | ||
/// @notice A CL pool facilitates swapping and automated market making between any two assets that strictly conform | ||
/// to the ERC20 specification | ||
/// @dev The pool interface is broken up into many smaller pieces | ||
interface ICLPool { | ||
function slot0() | ||
external | ||
view | ||
returns ( | ||
uint160 sqrtPriceX96, | ||
int24 tick, | ||
uint16 observationIndex, | ||
uint16 observationCardinality, | ||
uint16 observationCardinalityNext, | ||
bool unlocked | ||
); | ||
} |
149 changes: 149 additions & 0 deletions
149
contracts/contracts/interfaces/aerodrome/INonfungiblePositionManager.sol
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,149 @@ | ||
// SPDX-License-Identifier: GPL-2.0-or-later | ||
pragma solidity >=0.7.5; | ||
pragma abicoder v2; | ||
|
||
/// @title Non-fungible token for positions | ||
/// @notice Wraps CL positions in a non-fungible token interface which allows for them to be transferred | ||
/// and authorized. | ||
interface INonfungiblePositionManager | ||
{ | ||
/// @notice Returns the position information associated with a given token ID. | ||
/// @dev Throws if the token ID is not valid. | ||
/// @param tokenId The ID of the token that represents the position | ||
/// @return nonce The nonce for permits | ||
/// @return operator The address that is approved for spending | ||
/// @return token0 The address of the token0 for a specific pool | ||
/// @return token1 The address of the token1 for a specific pool | ||
/// @return tickSpacing The tick spacing associated with the pool | ||
/// @return tickLower The lower end of the tick range for the position | ||
/// @return tickUpper The higher end of the tick range for the position | ||
/// @return liquidity The liquidity of the position | ||
/// @return feeGrowthInside0LastX128 The fee growth of token0 as of the last action on the individual position | ||
/// @return feeGrowthInside1LastX128 The fee growth of token1 as of the last action on the individual position | ||
/// @return tokensOwed0 The uncollected amount of token0 owed to the position as of the last computation | ||
/// @return tokensOwed1 The uncollected amount of token1 owed to the position as of the last computation | ||
function positions(uint256 tokenId) | ||
external | ||
view | ||
returns ( | ||
uint96 nonce, | ||
address operator, | ||
address token0, | ||
address token1, | ||
int24 tickSpacing, | ||
int24 tickLower, | ||
int24 tickUpper, | ||
uint128 liquidity, | ||
uint256 feeGrowthInside0LastX128, | ||
uint256 feeGrowthInside1LastX128, | ||
uint128 tokensOwed0, | ||
uint128 tokensOwed1 | ||
); | ||
|
||
struct MintParams { | ||
address token0; | ||
address token1; | ||
int24 tickSpacing; | ||
int24 tickLower; | ||
int24 tickUpper; | ||
uint256 amount0Desired; | ||
uint256 amount1Desired; | ||
uint256 amount0Min; | ||
uint256 amount1Min; | ||
address recipient; | ||
uint256 deadline; | ||
uint160 sqrtPriceX96; | ||
} | ||
|
||
/// @notice Creates a new position wrapped in a NFT | ||
/// @dev Call this when the pool does exist and is initialized. Note that if the pool is created but not initialized | ||
/// a method does not exist, i.e. the pool is assumed to be initialized. | ||
/// @param params The params necessary to mint a position, encoded as `MintParams` in calldata | ||
/// @return tokenId The ID of the token that represents the minted position | ||
/// @return liquidity The amount of liquidity for this position | ||
/// @return amount0 The amount of token0 | ||
/// @return amount1 The amount of token1 | ||
function mint(MintParams calldata params) | ||
external | ||
payable | ||
returns (uint256 tokenId, uint128 liquidity, uint256 amount0, uint256 amount1); | ||
|
||
struct IncreaseLiquidityParams { | ||
uint256 tokenId; | ||
uint256 amount0Desired; | ||
uint256 amount1Desired; | ||
uint256 amount0Min; | ||
uint256 amount1Min; | ||
uint256 deadline; | ||
} | ||
|
||
/// @notice Increases the amount of liquidity in a position, with tokens paid by the `msg.sender` | ||
/// @param params tokenId The ID of the token for which liquidity is being increased, | ||
/// amount0Desired The desired amount of token0 to be spent, | ||
/// amount1Desired The desired amount of token1 to be spent, | ||
/// amount0Min The minimum amount of token0 to spend, which serves as a slippage check, | ||
/// amount1Min The minimum amount of token1 to spend, which serves as a slippage check, | ||
/// deadline The time by which the transaction must be included to effect the change | ||
/// @return liquidity The new liquidity amount as a result of the increase | ||
/// @return amount0 The amount of token0 to acheive resulting liquidity | ||
/// @return amount1 The amount of token1 to acheive resulting liquidity | ||
function increaseLiquidity(IncreaseLiquidityParams calldata params) | ||
external | ||
payable | ||
returns (uint128 liquidity, uint256 amount0, uint256 amount1); | ||
|
||
struct DecreaseLiquidityParams { | ||
uint256 tokenId; | ||
uint128 liquidity; | ||
uint256 amount0Min; | ||
uint256 amount1Min; | ||
uint256 deadline; | ||
} | ||
|
||
/// @notice Decreases the amount of liquidity in a position and accounts it to the position | ||
/// @param params tokenId The ID of the token for which liquidity is being decreased, | ||
/// amount The amount by which liquidity will be decreased, | ||
/// amount0Min The minimum amount of token0 that should be accounted for the burned liquidity, | ||
/// amount1Min The minimum amount of token1 that should be accounted for the burned liquidity, | ||
/// deadline The time by which the transaction must be included to effect the change | ||
/// @return amount0 The amount of token0 accounted to the position's tokens owed | ||
/// @return amount1 The amount of token1 accounted to the position's tokens owed | ||
/// @dev The use of this function can cause a loss to users of the NonfungiblePositionManager | ||
/// @dev for tokens that have very high decimals. | ||
/// @dev The amount of tokens necessary for the loss is: 3.4028237e+38. | ||
/// @dev This is equivalent to 1e20 value with 18 decimals. | ||
function decreaseLiquidity(DecreaseLiquidityParams calldata params) | ||
external | ||
payable | ||
returns (uint256 amount0, uint256 amount1); | ||
|
||
struct CollectParams { | ||
uint256 tokenId; | ||
address recipient; | ||
uint128 amount0Max; | ||
uint128 amount1Max; | ||
} | ||
|
||
/// @notice Collects up to a maximum amount of fees owed to a specific position to the recipient | ||
/// @notice Used to update staked positions before deposit and withdraw | ||
/// @param params tokenId The ID of the NFT for which tokens are being collected, | ||
/// recipient The account that should receive the tokens, | ||
/// amount0Max The maximum amount of token0 to collect, | ||
/// amount1Max The maximum amount of token1 to collect | ||
/// @return amount0 The amount of fees collected in token0 | ||
/// @return amount1 The amount of fees collected in token1 | ||
function collect(CollectParams calldata params) external payable returns (uint256 amount0, uint256 amount1); | ||
|
||
/// @notice Burns a token ID, which deletes it from the NFT contract. The token must have 0 liquidity and all tokens | ||
/// must be collected first. | ||
/// @param tokenId The ID of the token that is being burned | ||
function burn(uint256 tokenId) external payable; | ||
|
||
/// @notice Sets a new Token Descriptor | ||
/// @param _tokenDescriptor Address of the new Token Descriptor to be chosen | ||
function setTokenDescriptor(address _tokenDescriptor) external; | ||
|
||
/// @notice Sets a new Owner address | ||
/// @param _owner Address of the new Owner to be chosen | ||
function setOwner(address _owner) external; | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,88 @@ | ||
// SPDX-License-Identifier: GPL-2.0-or-later | ||
pragma solidity >=0.7.5; | ||
pragma abicoder v2; | ||
|
||
/// @title QuoterV2 Interface | ||
/// @notice Supports quoting the calculated amounts from exact input or exact output swaps. | ||
/// @notice For each pool also tells you the number of initialized ticks crossed and the sqrt price of the pool after the swap. | ||
/// @dev These functions are not marked view because they rely on calling non-view functions and reverting | ||
/// to compute the result. They are also not gas efficient and should not be called on-chain. | ||
interface IQuoterV2 { | ||
/// @notice Returns the amount out received for a given exact input swap without executing the swap | ||
/// @param path The path of the swap, i.e. each token pair and the pool tick spacing | ||
/// @param amountIn The amount of the first token to swap | ||
/// @return amountOut The amount of the last token that would be received | ||
/// @return sqrtPriceX96AfterList List of the sqrt price after the swap for each pool in the path | ||
/// @return initializedTicksCrossedList List of the initialized ticks that the swap crossed for each pool in the path | ||
/// @return gasEstimate The estimate of the gas that the swap consumes | ||
function quoteExactInput(bytes memory path, uint256 amountIn) | ||
external | ||
returns ( | ||
uint256 amountOut, | ||
uint160[] memory sqrtPriceX96AfterList, | ||
uint32[] memory initializedTicksCrossedList, | ||
uint256 gasEstimate | ||
); | ||
|
||
struct QuoteExactInputSingleParams { | ||
address tokenIn; | ||
address tokenOut; | ||
uint256 amountIn; | ||
int24 tickSpacing; | ||
uint160 sqrtPriceLimitX96; | ||
} | ||
|
||
/// @notice Returns the amount out received for a given exact input but for a swap of a single pool | ||
/// @param params The params for the quote, encoded as `QuoteExactInputSingleParams` | ||
/// tokenIn The token being swapped in | ||
/// tokenOut The token being swapped out | ||
/// tickSpacing The tick spacing of the token pool to consider for the pair | ||
/// amountIn The desired input amount | ||
/// sqrtPriceLimitX96 The price limit of the pool that cannot be exceeded by the swap | ||
/// @return amountOut The amount of `tokenOut` that would be received | ||
/// @return sqrtPriceX96After The sqrt price of the pool after the swap | ||
/// @return initializedTicksCrossed The number of initialized ticks that the swap crossed | ||
/// @return gasEstimate The estimate of the gas that the swap consumes | ||
function quoteExactInputSingle(QuoteExactInputSingleParams memory params) | ||
external | ||
returns (uint256 amountOut, uint160 sqrtPriceX96After, uint32 initializedTicksCrossed, uint256 gasEstimate); | ||
|
||
/// @notice Returns the amount in required for a given exact output swap without executing the swap | ||
/// @param path The path of the swap, i.e. each token pair and the pool tick spacing. Path must be provided in reverse order | ||
/// @param amountOut The amount of the last token to receive | ||
/// @return amountIn The amount of first token required to be paid | ||
/// @return sqrtPriceX96AfterList List of the sqrt price after the swap for each pool in the path | ||
/// @return initializedTicksCrossedList List of the initialized ticks that the swap crossed for each pool in the path | ||
/// @return gasEstimate The estimate of the gas that the swap consumes | ||
function quoteExactOutput(bytes memory path, uint256 amountOut) | ||
external | ||
returns ( | ||
uint256 amountIn, | ||
uint160[] memory sqrtPriceX96AfterList, | ||
uint32[] memory initializedTicksCrossedList, | ||
uint256 gasEstimate | ||
); | ||
|
||
struct QuoteExactOutputSingleParams { | ||
address tokenIn; | ||
address tokenOut; | ||
uint256 amount; | ||
int24 tickSpacing; | ||
uint160 sqrtPriceLimitX96; | ||
} | ||
|
||
/// @notice Returns the amount in required to receive the given exact output amount but for a swap of a single pool | ||
/// @param params The params for the quote, encoded as `QuoteExactOutputSingleParams` | ||
/// tokenIn The token being swapped in | ||
/// tokenOut The token being swapped out | ||
/// tickSpacing The tick spacing of the token pool to consider for the pair | ||
/// amountOut The desired output amount | ||
/// sqrtPriceLimitX96 The price limit of the pool that cannot be exceeded by the swap | ||
/// @return amountIn The amount required as the input for the swap in order to receive `amountOut` | ||
/// @return sqrtPriceX96After The sqrt price of the pool after the swap | ||
/// @return initializedTicksCrossed The number of initialized ticks that the swap crossed | ||
/// @return gasEstimate The estimate of the gas that the swap consumes | ||
function quoteExactOutputSingle(QuoteExactOutputSingleParams memory params) | ||
external | ||
returns (uint256 amountIn, uint160 sqrtPriceX96After, uint32 initializedTicksCrossed, uint256 gasEstimate); | ||
} |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the latest version is actions/cache@v4