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.

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

Advance fail test #10973

Closed
wants to merge 5 commits into from
Closed

Advance fail test #10973

wants to merge 5 commits into from

Conversation

samsiegart
Copy link
Contributor

@samsiegart samsiegart commented Feb 9, 2025

It seems like in this scenario, the encumbered balance is not returning to 0. The lp-withdraw test next in the sequence fails because there's not enough USDC in the pool for all the LP tokens. I tried adding a sleep to see if it was just a delay, but I checked the vstorage viewer for my local chain several minutes later and it's still not reconstituted:

Partial test logs: https://gist.github.com/samsiegart/5d58b1e9e81b636fd3eb63a8940f8433
Chain logs for test run: https://gist.github.com/samsiegart/66776e48893a77f4cbd5a61e64abf87e

Vstorage after running test:

{"blockHeight":"854","values":[{"encumberedBalance":{"brand":{},"value":1583980000},"shareWorth":{"denominator":{"brand":{},"value":8000000001},"numerator":{"brand":{},"value":8036824001}},"totalBorrows":{"brand":{},"value":6137950000},"totalContractFees":{"brand":{},"value":9206000},"totalPoolFees":{"brand":{},"value":36824000},"totalRepays":{"brand":{},"value":4553970000}}]}
Screenshot 2025-02-09 at 11 14 09 AM

@samsiegart samsiegart requested a review from a team as a code owner February 9, 2025 19:17
Copy link

cloudflare-workers-and-pages bot commented Feb 9, 2025

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3c9d859
Status: ✅  Deploy successful!
Preview URL: https://0abbfe3b.agoric-sdk.pages.dev
Branch Preview URL: https://advance-fail.agoric-sdk.pages.dev

View logs

@samsiegart samsiegart force-pushed the test-forward-fail branch 2 times, most recently from 616fab6 to a1f13a5 Compare February 11, 2025 04:39
@samsiegart samsiegart closed this Feb 11, 2025
@samsiegart samsiegart deleted the advance-fail branch February 11, 2025 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants