Skip to content

fix: handle a simulation without a result - #556

Merged
Glittersup merged 1 commit into
TricklePay:mainfrom
Tek-Lizzy:fix/issue-500
Oct 24, 2026
Merged

Glittersup merged 1 commit into
TricklePay:mainfrom
Tek-Lizzy:fix/issue-500

Conversation

@Tek-Lizzy

Copy link
Copy Markdown
Contributor

Summary

Guard against successful token-balance simulations that omit their result. Instead of causing a TypeError, the balance read now throws a clear error: “Balance read did not return a value.” Added a regression test for this response.

Linked issue

Closes #500

What was tested

  • git diff --check passed.
  • Editor diagnostics reported no errors in the changed files.
  • The focused test and typecheck could not be completed because dependencies were unavailable after the interrupted npm ci. Lint was not run.

Checklist

  • npm test passes
  • npm run typecheck passes
  • npm run lint passes
  • Browser testing not applicable; this change does not affect the UI.

@Glittersup
Glittersup merged commit afdcfb5 into TricklePay:main Oct 24, 2026
2 checks passed
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.

Guard against a missing simulation result in the token balance read

2 participants