Skip to content

test: add unit tests for issues #673, #674, #675, #676 - #1254

Merged
vjuliaife merged 1 commit into
vjuliaife:mainfrom
greyforreal:fix/issues-673-674-675-676
Oct 18, 2026
Merged

vjuliaife merged 1 commit into
vjuliaife:mainfrom
greyforreal:fix/issues-673-674-675-676

Conversation

@greyforreal

Copy link
Copy Markdown
Contributor

Closes #676

Unit test stroopsToXlm formatting

  • Verifies correct zero-padding to 4 decimals
  • Tests string, bigint, and number inputs
  • Validates whole numbers, tiny fractions, and edge cases

Closes #675

Unit test Nav network badge

  • Tests mainnet vs testnet badge rendering
  • Verifies correct colors and text based on env var
  • Tests fallback behavior for unrecognized values
  • Validates badge is independent of auth state

Closes #674

Unit test Nav logout button

  • Verifies clearSession() and router.push('/') are called
  • Tests correct execution order (session cleared before redirect)
  • Validates button only renders for authenticated users

Closes #673

Unit test Nav dashboard links per user role

  • Tests importer role shows 'Bond dashboard' -> /app
  • Tests surety_admin role shows 'Surety admin' -> /surety
  • Verifies logo href changes based on role
  • Validates email rendering for both roles

…fe#675, vjuliaife#676

- vjuliaife#676: Unit test stroopsToXlm formatting
  - Verifies correct zero-padding to 4 decimals
  - Tests string, bigint, and number inputs
  - Validates whole numbers, tiny fractions, and edge cases

- vjuliaife#675: Unit test Nav network badge
  - Tests mainnet vs testnet badge rendering
  - Verifies correct colors and text based on env var
  - Tests fallback behavior for unrecognized values
  - Validates badge is independent of auth state

- vjuliaife#674: Unit test Nav logout button
  - Verifies clearSession() and router.push('/') are called
  - Tests correct execution order (session cleared before redirect)
  - Validates button only renders for authenticated users

- vjuliaife#673: Unit test Nav dashboard links per user role
  - Tests importer role shows 'Bond dashboard' -> /app
  - Tests surety_admin role shows 'Surety admin' -> /surety
  - Verifies logo href changes based on role
  - Validates email rendering for both roles
@drips-wave

drips-wave Bot commented Oct 18, 2026

Copy link
Copy Markdown

@greyforreal Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@vercel

vercel Bot commented Oct 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
tariff-shield-web Error Error Oct 18, 2026 2:06am UTC

@vjuliaife
vjuliaife merged commit 4a78a04 into vjuliaife:main Oct 18, 2026
1 of 2 checks passed

This branch had an error being deployed

1 failed deployment
Preview — 8dee614d Deployed Oct 18, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants