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

Add close position test for rust sdk #660

Merged
merged 53 commits into from
Jan 19, 2025
Merged

Add close position test for rust sdk #660

merged 53 commits into from
Jan 19, 2025

Conversation

pauldragonfly
Copy link
Contributor

Add close position test for rust sdk

Key feature :

  • Add multiple combo test for close position

pauldragonfly and others added 30 commits December 18, 2024 03:07
@pauldragonfly
Add position test
542ad5f
@pauldragonfly
Merge branch 'main' of github.com:orca-so/whirlpools into paul/rust-s…
6c64a13 
…dk-add-position-test
@pauldragonfly
Initial commit for position test
faf362a
@pauldragonfly
Update rpc
d268223
@pauldragonfly
Update format
901ed91
@pauldragonfly
Add decrease liquidity test
476885d
@pauldragonfly
fmt
208a7ca
@pauldragonfly
Address comments
7a8934d
@pauldragonfly
Remove print
217552a 
Remove unused config function
@pauldragonfly
Merge branch 'paul/rust-sdk-add-position-test' into paul/add-increase…
8a60fc2 
…-liquidity-test-rust-sdk
@pauldragonfly
Update comments
ecefd33
@pauldragonfly
Fix fmt
1cad741
@pauldragonfly
Merge branch 'paul/add-increase-liquidity-test-rust-sdk' into paul/ad…
aa15c51 
…d-decrese-liquidity-test-rust-sdk
@pauldragonfly
Cleanup unused variables and wrong test
74fe0b3
@pauldragonfly
Cleanup spacing
45c753e
@pauldragonfly
Update comments
50ef034
@pauldragonfly
Cleanup repo
34ee972
@pauldragonfly
Merge branch 'main' of github.com:orca-so/whirlpools into paul/rust-s…
a88ca69 
…dk-add-position-test
@pauldragonfly
Merge branch 'paul/rust-sdk-add-position-test' of github.com:orca-so/…
0f884eb 
…whirlpools into paul/add-increase-liquidity-test-rust-sdk
@pauldragonfly
Merge branch 'paul/add-increase-liquidity-test-rust-sdk' of github.co…
f296ab5 
…m:orca-so/whirlpools into paul/add-decrese-liquidity-test-rust-sdk
@pauldragonfly
Cleanup
e422a86
@wjthieme
Merge branch 'main' into paul/rust-sdk-add-position-test
3c095da
@pauldragonfly
Update increase liquidity test to actually check tokens
a32d25e
@pauldragonfly
Merge branch 'paul/rust-sdk-add-position-test' of github.com:orca-so/…
b6ee336 
…whirlpools into paul/add-increase-liquidity-test-rust-sdk
@pauldragonfly
Update test to check te and bundle
77947f6
@pauldragonfly
Merge branch 'paul/rust-sdk-add-position-test' of github.com:orca-so/…
0cc91b2 
…whirlpools into paul/rust-sdk-add-position-test
@pauldragonfly
Update dependabot
1440df3
@pauldragonfly
Fix dependabot space
24abe07
@pauldragonfly
revert dependabot
a847a80
@pauldragonfly
FMT
57f9e6e
@pauldragonfly
Merge branch 'paul/rust-sdk-add-position-test' of github.com:orca-so/…
1fe55df 
…whirlpools into paul/add-increase-liquidity-test-rust-sdk
@pauldragonfly
Update fmt
83973ed
@pauldragonfly
Update not to create mint, instead use instructions
f329cdd
@pauldragonfly
Merge branch 'paul/rust-sdk-add-position-test' into paul/add-increase…
e56e1a9 
…-liquidity-test-rust-sdk
@pauldragonfly
Add multiple pool tests and fix bugs in the program.rs
aa45948
@pauldragonfly
Update full combo of tests
e0128f5
@pauldragonfly
Merge branch 'paul/add-increase-liquidity-test-rust-sdk' of github.co…
ad3d2c8 
…m:orca-so/whirlpools into paul/add-decrese-liquidity-test-rust-sdk
@pauldragonfly
Update multiple combo of tests
e15f60b 
Update program to handle multiple tick array when decrease
@pauldragonfly
Merge branch 'main' into paul/rust-sdk-add-position-test
880f397
@pauldragonfly
Update to use actual address in positions
7861e2e
@pauldragonfly
Merge branch 'main' into paul/rust-sdk-add-position-test
0e9be37
@pauldragonfly
Merge branch 'paul/rust-sdk-add-position-test' of github.com:orca-so/…
73449e4 
…whirlpools into paul/add-increase-liquidity-test-rust-sdk
@pauldragonfly
Fix to use real address for positions
3e9fd92
@pauldragonfly
Merge branch 'paul/add-increase-liquidity-test-rust-sdk' into paul/ad…
cde82bd 
…d-decrese-liquidity-test-rust-sdk
@pauldragonfly
Add harvest test
a5a04c6
@pauldragonfly
Update comment
7ccb120
@pauldragonfly
Add swap test
453ce1e
@pauldragonfly
Merge branch 'paul/add-harvest-test-rust-sdk' into paul/add-swap-test…
17b74fe 
…-rust-sdk
@pauldragonfly
Merge branch 'main' into paul/add-swap-test-rust-sdk
e3b4081
@pauldragonfly
Add close position tests and remove prints
dad97b8
@pauldragonfly
Revert swap.rs to main
0d60d90
@pauldragonfly
Small space fix
0881c9d
@pauldragonfly pauldragonfly marked this pull request as ready for review January 17, 2025 20:04
wjthieme
wjthieme reviewed Jan 17, 2025
View reviewed changes
Copy link
Member

@wjthieme wjthieme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Think only the TS close position test does not have fees yet. Can you add as well?

wjthieme
wjthieme approved these changes Jan 18, 2025
View reviewed changes
Copy link
Member

@wjthieme wjthieme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@pauldragonfly pauldragonfly merged commit 4a79e52 into main Jan 19, 2025
5 checks passed
@pauldragonfly pauldragonfly deleted the paul/add-close-position-test-rust branch January 19, 2025 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wjthieme wjthieme wjthieme approved these changes

@yugure-orca yugure-orca Awaiting requested review from yugure-orca yugure-orca is a code owner

@odcheung odcheung Awaiting requested review from odcheung odcheung is a code owner

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

Successfully merging this pull request may close these issues.
2 participants
@pauldragonfly @wjthieme