Skip to content

test(frontend): add tests for levels, starboard, reaction roles - #392

Merged
LucasSantana-Dev merged 5 commits into
mainfrom
test/frontend-remaining-pages
Mar 30, 2026
Merged

LucasSantana-Dev merged 5 commits into
mainfrom
test/frontend-remaining-pages

Conversation

@LucasSantana-Dev

@LucasSantana-Dev LucasSantana-Dev commented Mar 30, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Added comprehensive test coverage for Levels.tsx page (31 tests)
  • Added comprehensive test coverage for Starboard.tsx page (20 tests)
  • Added comprehensive test coverage for ReactionRoles.tsx page (19 tests)

Coverage

All three pages now have >60% line coverage with tests covering:

  • Loading states and skeletons
  • Empty states
  • Data rendering (leaderboards, entries, mappings)
  • User interactions (form inputs, toggles, buttons)
  • API calls and error handling
  • Success and failure scenarios

Test count: 70 tests total

  • Levels: 31 tests
  • Starboard: 20 tests
  • ReactionRoles: 19 tests

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Tests
    • Added comprehensive test suites for Levels, ReactionRoles, and Starboard pages to improve code quality and reliability.

@vercel

vercel Bot commented Mar 30, 2026 •

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
lucky Ready Ready Preview, Comment Mar 30, 2026 10:56pm

Request Review

@coderabbitai

coderabbitai Bot commented Mar 30, 2026 •

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Three new test suites added for frontend page components using Vitest and React Testing Library. The tests mock guild store, API methods, and toast notifications to validate rendering, data loading, user interactions, error handling, and state transitions across Levels, ReactionRoles, and Starboard pages.

Changes

Cohort / File(s) Summary
Page Component Test Suites
packages/frontend/src/pages/Levels.test.tsx, packages/frontend/src/pages/ReactionRoles.test.tsx, packages/frontend/src/pages/Starboard.test.tsx
Comprehensive test coverage for page components. Tests validate empty states (no guild selected), loading skeletons, successful data loading and rendering, empty data states, form interactions (toggling, editing inputs), API calls with expected payloads, success/error toasts, error recovery with retry, and graceful handling of missing/null API responses. Includes assertions on button disabled states during pending operations and proper display of role/channel information.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding comprehensive test coverage for three frontend pages (Levels, Starboard, and ReactionRoles), which aligns with the 70 new tests and changes totaling +1,611 lines.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch test/frontend-remaining-pages

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot removed the bot label Mar 30, 2026
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
16.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@github-actions

github-actions Bot commented Mar 30, 2026 •

Copy link
Copy Markdown

Size Change: 0 B

Total Size: 318 kB

ℹ️ View Unchanged
Filename Size
packages/frontend/dist/assets/ActionPanel-CPPMxhFF.js 401 B
packages/frontend/dist/assets/api-3qPvg92E.js 2.9 kB
packages/frontend/dist/assets/authStore-DBeEdplV.js 557 B
packages/frontend/dist/assets/AutoMessages-B5SGnQRY.js 2.68 kB
packages/frontend/dist/assets/AutoMod-BIev-2yJ.js 4.1 kB
packages/frontend/dist/assets/badge-CtZYPo1h.js 502 B
packages/frontend/dist/assets/Card-CLEr0GPg.js 456 B
packages/frontend/dist/assets/CommandsConfig-BXhrTbl5.js 1.47 kB
packages/frontend/dist/assets/Config-D57d-pFi.js 1.68 kB
packages/frontend/dist/assets/CustomCommands-BxlA9eRj.js 2.16 kB
packages/frontend/dist/assets/DashboardOverview-5PhfZF9m.js 2.54 kB
packages/frontend/dist/assets/dialog-CQehvQgs.js 949 B
packages/frontend/dist/assets/EmbedBuilder-7tHvn4jo.js 3.34 kB
packages/frontend/dist/assets/Features-DzxvVb1H.js 2.85 kB
packages/frontend/dist/assets/GuildAutomation-CeYLVHyE.js 2.92 kB
packages/frontend/dist/assets/guildStore-BuJ9Xfyw.js 794 B
packages/frontend/dist/assets/index-DB-ntyfa.css 14 kB
packages/frontend/dist/assets/index-pfwciEbx.js 27.6 kB
packages/frontend/dist/assets/input-1mJNYRIq.js 466 B
packages/frontend/dist/assets/label-B0w5nWAd.js 488 B
packages/frontend/dist/assets/LastFm-DnWvDY9S.js 1.74 kB
packages/frontend/dist/assets/Levels-D2dn-hXg.js 2.63 kB
packages/frontend/dist/assets/Login-BB_QV5YE.js 2.51 kB
packages/frontend/dist/assets/Lyrics-Dg08GGUZ.js 1.33 kB
packages/frontend/dist/assets/Moderation-D853bYCM.js 3.85 kB
packages/frontend/dist/assets/Music-BMpwnMtJ.js 6.13 kB
packages/frontend/dist/assets/MusicConfig-C8I9qK4Q.js 1.61 kB
packages/frontend/dist/assets/PrivacyPolicy-DUO0quHd.js 1.38 kB
packages/frontend/dist/assets/ReactionRoles-Iu10nhih.js 1.88 kB
packages/frontend/dist/assets/rolldown-runtime-COnpUsM8.js 467 B
packages/frontend/dist/assets/SectionHeader-C3U2Dpr0.js 384 B
packages/frontend/dist/assets/select-BLfozU6F.js 1.23 kB
packages/frontend/dist/assets/ServerLogs-DbDUiuLF.js 2.9 kB
packages/frontend/dist/assets/ServerSettings-CTp3sI-Q.js 4.22 kB
packages/frontend/dist/assets/ServersPage-VL5GlTsG.js 2.92 kB
packages/frontend/dist/assets/Skeleton-Cm2SGg8-.js 235 B
packages/frontend/dist/assets/Starboard-DSgWQM64.js 2.08 kB
packages/frontend/dist/assets/StatTile-ieCmGz9d.js 605 B
packages/frontend/dist/assets/switch-Ubn95ylk.js 542 B
packages/frontend/dist/assets/TermsOfService-B0ThRl8c.js 1.37 kB
packages/frontend/dist/assets/TrackHistory-CwpFViZg.js 1.9 kB
packages/frontend/dist/assets/TwitchNotifications-CVxXBBPA.js 2.29 kB
packages/frontend/dist/assets/usePageMetadata-DNK___tM.js 326 B
packages/frontend/dist/assets/utils-BHIP-_5u.js 148 B
packages/frontend/dist/assets/vendor-forms-KXRApnAU.js 25.4 kB
packages/frontend/dist/assets/vendor-radix-Uy8m4UkR.js 33.5 kB
packages/frontend/dist/assets/vendor-react-DllCF3mE.js 55.6 kB
packages/frontend/dist/assets/vendor-state-BC0s6PNN.js 22 kB
packages/frontend/dist/assets/vendor-ui-DLN5_jR3.js 64.2 kB

compressed-size-action

@sonarqubecloud

Copy link
Copy Markdown

@LucasSantana-Dev
LucasSantana-Dev merged commit 042934e into main Mar 30, 2026
12 of 13 checks passed
@LucasSantana-Dev
LucasSantana-Dev deleted the test/frontend-remaining-pages branch March 30, 2026 23:00
LucasSantana-Dev added a commit that referenced this pull request May 13, 2026
* test(bot): add coverage for client, player, and queue handlers

* test(frontend): add embed builder and guild automation tests

* test(frontend): add tests for levels, starboard, and reaction roles

* fix(frontend): correct mock types in page tests

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

This branch was successfully deployed

1 active deployment
Preview — 58e197d1 Deployed Mar 30, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant