Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Restructuring LiquidityApproximation and E2ESwap tests #1080

Open
wants to merge 104 commits into
base: main
Choose a base branch
from
Open

Restructuring LiquidityApproximation and E2ESwap tests #1080

wants to merge 104 commits into from

Conversation

joaobrunoah
Copy link
Contributor

@joaobrunoah joaobrunoah commented Oct 25, 2024 •
edited
Loading

Description

This PR restructures LiquidityApproximation and E2ESwap tests, so any future pool will easily extend these tests.

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Dependency changes
  • Code refactor / cleanup
  • Optimization: [ ] gas / [ ] bytecode
  • Documentation or wording changes
  • Other

Checklist:

  • The diff is legible and has no extraneous changes
  • Complex code has been commented, including external interfaces
  • Tests have 100% code coverage
  • The base branch is either main, or there's a description of how to merge

joaobrunoah and others added 30 commits October 9, 2024 19:13
@joaobrunoah
Gyro 2-CLP
83f678f
@joaobrunoah
Merge branch 'main' into gyro-2clp
1224305
@joaobrunoah
Gyro Liquidity Approx Tests
ddbbe5b
@joaobrunoah
E2e tests
7a20c2b
@joaobrunoah
Fix invariant calculation on Gyro Pools
e3e7a5a
@joaobrunoah
Remove unused variable
e05dcf2
@joaobrunoah
Fix README
c99779d
@joaobrunoah
Fix E2EBatchSwap tests
d5b491c
@joaobrunoah
Merge branch 'main' into gyro-2clp
ea82f17
@joaobrunoah
Comment Gyro Math
93c9846
@joaobrunoah
Fix mutability of test functions
d86085d
@joaobrunoah
Fix comments of factory
963e16f
@joaobrunoah
First version of E-CLP pool
28e213d
@joaobrunoah
Implement computeBalance
51202d8
@joaobrunoah
Implement LiquidityApprox test for E-CLP
6182916
@joaobrunoah
Fix LiquidityApproximation tests
50cb3fe
@joaobrunoah
Create EclpPoolDeployer, to reuse the same pool creation for multiple…
801be0c 
… tests
@joaobrunoah
Fix E2E tests
a1a95eb
@joaobrunoah
Fix comments
6f0dd15
@joaobrunoah
Implement deployer to 2CLP
f5fabda
@joaobrunoah
Merge branch 'main' into gyro-2clp
c0f3ef3
@joaobrunoah
Merge branch 'gyro-2clp' into gyro-eclp-new
758df85
@joaobrunoah
Fix naming
fc2b634
@joaobrunoah
Fix comments
fc6e4a5
@joaobrunoah
Lint
b5330d5
@joaobrunoah
Implement CI of Gyro
7288a9f
@joaobrunoah @jubeira
Update pkg/pool-gyro/contracts/Gyro2CLPPool.sol
7bffe4e 
Co-authored-by: Juan Ignacio Ubeira <[email protected]>
@joaobrunoah
Merge remote-tracking branch 'origin/gyro-2clp' into gyro-2clp
5034e4c
@joaobrunoah
Fix comment
937dca7
@joaobrunoah
Implement _mulDownOrUp
7e18709
@joaobrunoah joaobrunoah changed the base branch from main to gyro-eclp-new December 9, 2024 13:18
Base automatically changed from gyro-eclp-new to main December 17, 2024 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jubeira jubeira Awaiting requested review from jubeira

@elshan-eth elshan-eth Awaiting requested review from elshan-eth

@EndymionJkb EndymionJkb Awaiting requested review from EndymionJkb

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@joaobrunoah @EndymionJkb @sschuldenzucker