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

test(CL Fees): updateFeeGrowthGlobal method #4086

Closed
Tracked by #3074
p0mvn opened this issue Jan 24, 2023 · 2 comments
Closed
Tracked by #3074

test(CL Fees): updateFeeGrowthGlobal method #4086

p0mvn opened this issue Jan 24, 2023 · 2 comments
Assignees
@p0mvn
Labels
F: concentrated-liquidity Tracking the development of concentrated liquidity feature to improve filtering on the project board T:tests
Milestone
CL 8: Fees, Pool ...

Comments

@p0mvn
Copy link
Member

p0mvn commented Jan 24, 2023

Background

The following method needs to be tested:

osmosis/x/concentrated-liquidity/swaps.go

Line 33 in df3853f

// TODO: test

Suggested Design

Add table driven test to maximize test coverage

Acceptance Criteria

  • test added
  • coverage maximized
@p0mvn p0mvn added the F: concentrated-liquidity Tracking the development of concentrated liquidity feature to improve filtering on the project board label Jan 24, 2023
@github-project-automation github-project-automation bot moved this to Needs Review 🔍 in Osmosis Chain Development Jan 24, 2023
@p0mvn
Copy link
Member Author

p0mvn commented Jan 24, 2023

It might be that this issue ends up being obsolete after implementing swap fees for cal out given in. Making it for tracking purposes

@p0mvn
Copy link
Member Author

p0mvn commented Jan 24, 2023

Revisit after #4088 is complete

This was referenced Jan 24, 2023
Closed
Merged
@p0mvn p0mvn self-assigned this Feb 1, 2023
@p0mvn p0mvn mentioned this issue Feb 1, 2023
Merged
@p0mvn p0mvn closed this as completed Feb 27, 2023
@github-project-automation github-project-automation bot moved this from Needs Triage 🔍 to Done ✅ in Osmosis Chain Development Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@p0mvn p0mvn

Labels
F: concentrated-liquidity Tracking the development of concentrated liquidity feature to improve filtering on the project board T:tests
Projects
Osmosis Chain Development
Archived in project
Milestone
CL 8: Fees, Pool Manager Module Final...
Development

No branches or pull requests
2 participants
@daniel-farina @p0mvn