Skip to content

Add missing test suites for components with no coverage#3296

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/identify-missing-test-suites
Draft

Add missing test suites for components with no coverage#3296
Copilot wants to merge 2 commits intomainfrom
copilot/identify-missing-test-suites

Conversation

Copy link

Copilot AI commented Mar 20, 2026

Overview

Adds unit test suites for 8 components in the gamut package that previously had no test coverage: Animation (Rotation), AppWrapper, Box (Box, FlexBox, GridBox), Card, Layout (LayoutGrid, Column), Loading (Shimmer, Spinner), RadialProgress, and Typography (Text).

PR Checklist

  • Related to designs:
  • Related to JIRA ticket: [ABC-123]
  • I have run this code to verify it works
  • This PR includes unit tests for the code change
  • This PR includes testing instructions tests for the code change
  • The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Run the new test suites directly:

cd packages/gamut
NODE_OPTIONS='--loader ts-node/esm' npx jest --testPathPattern="Animation/__tests__|AppWrapper/__tests__|Box/__tests__|Card/__tests__|Layout/__tests__|Loading/__tests__|RadialProgress/__tests__|Typography/__tests__" --no-coverage

All 36 tests across 8 new test files should pass.

PR Links and Envs

Repository PR Link
Monolith Monolith PR
Mono Mono PR

⚡ Quickly spin up Copilot coding agent tasks from anywhere on your macOS or Windows machine with Raycast.

@nx-cloud
Copy link

nx-cloud bot commented Mar 20, 2026

View your CI Pipeline Execution ↗ for commit fa434ef


☁️ Nx Cloud last updated this comment at 2026-03-20 15:42:50 UTC

@jakemhiller
Copy link
Member

@copilot continue

@codecov
Copy link

codecov bot commented Mar 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.58%. Comparing base (501092e) to head (fa434ef).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3296      +/-   ##
==========================================
+ Coverage   89.86%   90.58%   +0.71%     
==========================================
  Files         362      236     -126     
  Lines        5229     4342     -887     
  Branches     1696     1478     -218     
==========================================
- Hits         4699     3933     -766     
+ Misses        521      399     -122     
- Partials        9       10       +1     
Flag Coverage Δ
main ?
pull-request 90.58% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

… Loading, RadialProgress, and Typography

Co-authored-by: jakemhiller <238023+jakemhiller@users.noreply.github.com>
Copy link
Author

Copilot AI commented Mar 20, 2026

@copilot continue

Done! Added 8 new test suites (36 tests total) for the components that were missing coverage — Animation/Rotation, AppWrapper, Box/FlexBox/GridBox, Card, Layout/Column, Loading/Shimmer/Spinner, RadialProgress, and Typography/Text. All tests follow the existing setupRtl pattern and pass. Commit: fa434ef

Copilot AI changed the title [WIP] Identify components missing test suites and low coverage Add missing test suites for components with no coverage Mar 20, 2026
@codecademydev
Copy link
Collaborator

📬 Published Alpha Packages:

@codecademy/gamut@68.1.5-alpha.b4e301.0
@codecademy/gamut-kit@0.6.590-alpha.b4e301.0

@github-actions
Copy link
Contributor

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.

3 participants