Skip to content

test(frontend): add Ready badge snapshot at weighted-quorum boundary - #553

Merged
thegreatfeez merged 1 commit into
Ac0rdP:mainfrom
Olumide-01:feat/statusbadge-weighted-quorum-boundary
Aug 31, 2026
Merged

test(frontend): add Ready badge snapshot at weighted-quorum boundary#553
thegreatfeez merged 1 commit into
Ac0rdP:mainfrom
Olumide-01:feat/statusbadge-weighted-quorum-boundary

Conversation

@Olumide-01

Copy link
Copy Markdown
Contributor

Summary

A proposal becomes executable the moment its approval weight meets the quorum weight. This adds a snapshot test that locks in how the "Ready" status badge looks at exactly that weighted-quorum transition boundary (where approval weight exactly meets quorum weight), guarding against accidental visual regressions there.

Changes

  • Added a new snapshot test renders ready badge at the weighted-quorum transition boundary in frontend/src/components/StatusBadge.test.tsx.
  • Generated the corresponding snapshot in frontend/src/components/__snapshots__/StatusBadge.test.tsx.snap.
  • All existing five status snapshots (pending, ready, executed, expired, revoked) remain intact.

Acceptance Criteria

  • A new snapshot test exists for the Ready badge at the weighted-quorum boundary. ✅
  • All existing status snapshots still pass. ✅
  • The test suite runs green. ✅

Closes #401

Add a snapshot test documenting the Ready badge at the exact weighted-quorum
transition boundary where approval weight first meets quorum weight.
@vercel

vercel Bot commented Aug 31, 2026

Copy link
Copy Markdown

@Olumide-01 is attempting to deploy a commit to the thegreatfeez's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Aug 31, 2026

Copy link
Copy Markdown

@Olumide-01 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

@thegreatfeez
thegreatfeez merged commit 200521f into Ac0rdP:main Aug 31, 2026
1 check failed
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.

Update StatusBadge snapshot tests to cover the weighted-quorum "Ready" transition boundary case

2 participants