Skip to content

Add staking estimator, dashboard tests, and SEP-10 nonce guard - #1308

Open
wheval wants to merge 1 commit into
ceejaylaboratory:mainfrom
wheval:feat/issues-1238-1234-1229-1177-dashboard-theme-staking-tests-sep10
Open

wheval wants to merge 1 commit into
ceejaylaboratory:mainfrom
wheval:feat/issues-1238-1234-1229-1177-dashboard-theme-staking-tests-sep10

Conversation

@wheval

@wheval wheval commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • added a dashboard staking yield estimator with deposit and lock-duration controls
  • added focused dashboard tests for the staking estimator, dashboard placement, and existing theme coverage
  • stored SEP-10 challenge nonces in Redis with a five-minute TTL and consumed them during token verification to prevent replay
  • fixed the JWT blacklist logger call so the auth controller test compiles

Validation

  • ../node_modules/.bin/vitest run src/components/StakingCalculator.test.tsx src/components/DashboardOverview.test.tsx src/contexts/ThemeContext.test.tsx — 29 passed
  • npm run pretest in backend — Prisma Client generated
  • ../node_modules/.bin/jest --config jest.config.js --runInBand --runTestsByPath src/api/controllers/auth.controller.test.ts --coverage=false — 9 passed
  • ../node_modules/.bin/vitest run in dashboard — 429 passed, 31 failed in existing areas outside this change, including App uiConfig, FlashLoan contract-address fixtures, CircuitBreaker mock expectations, and timer-sensitive component tests

Notes

This adds focused component coverage for the dashboard changes, but repo-wide dashboard coverage was not verified because the existing full dashboard suite is failing before coverage can be trusted.

Closes #1238
Closes #1234
Closes #1229
Closes #1177

@drips-wave

drips-wave Bot commented Sep 29, 2026

Copy link
Copy Markdown

@wheval 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment