Skip to content

[Testing] Add contract fuzz tests for create_stream with edge-case amounts - #436

Closed
Godbrand0 wants to merge 1 commit into
LabsCrypt:mainfrom
Godbrand0:fix/create-stream-fuzz-tests
Closed

[Testing] Add contract fuzz tests for create_stream with edge-case amounts#436
Godbrand0 wants to merge 1 commit into
LabsCrypt:mainfrom
Godbrand0:fix/create-stream-fuzz-tests

Conversation

@Godbrand0

Copy link
Copy Markdown
Contributor

Resolves #425

Changes

  • Added proptest for fuzzing tests on stream edge cases in Soroban
  • Tested that amount = 0 correctly handles intercept errors without panicking.
  • Validated no overflow errors against i128::MAX volume deposits.
  • Verified stream handling matches minimum constraints under 1 ledger increments.
  • Confirmed exact truncating and refund bounds on remainders failing divisible metrics. Updated corresponding module documentation outlining exactly when stream fractions distribute.

- Covered zero amount stream creation
- Covered max amount stream creation (i128::MAX)
- Covered single ledger streams
- Documented and tested truncation handling for indivisible amounts
@ogazboiz

Copy link
Copy Markdown
Contributor

hey, main was failing CI from broken auth imports + frontend parse errors. fixed and pushed to main now. please rebase to pick up the fixes:

git fetch upstream
git rebase upstream/main
git push --force-with-lease

if there's a conflict, resolve it locally and we'll review once CI is green.

@ogazboiz

ogazboiz commented May 1, 2026

Copy link
Copy Markdown
Contributor

hey, main CI is fully green now. there were several real TypeScript and test issues from #444's merge that have all been fixed. please rebase one more time:

git fetch upstream
git rebase upstream/main
git push --force-with-lease

your PR's diff should be much cleaner after this rebase since main now has the canonical versions of cancel.ts, sorobanService.ts, and the withdraw test.

@ogazboiz

ogazboiz commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

closing for inactivity — this has been open ~4 weeks with no contributor response after the last review. main has moved significantly since (#499 contract reconciliation, #706 auth-mock fix, ~150 new issues filed). if you want to continue this work, open a fresh PR rebased on current main. happy to review it then.

join us on Telegram if you want to coordinate: https://t.me/+DOylgFv1jyJlNzM0

@ogazboiz ogazboiz closed this Jun 1, 2026
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.

[Testing] Add contract fuzz tests for create_stream with edge-case amounts

2 participants