Skip to content

Conversation

joshfarrant
Copy link
Contributor

@joshfarrant joshfarrant commented Sep 4, 2025

Excludes the following directories from the coverage report

  • packages/react/src/recipes
  • packages/react/src/fixtures
  • packages/react/src/test-utils

This gives us a more accurate figure for our overall test coverage of 85.34%, compared to 77.22% that was reported before

Copy link

changeset-bot bot commented Sep 4, 2025

⚠️ No Changeset found

Latest commit: 6bceac5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Contributor

github-actions bot commented Sep 4, 2025

🟢 No design token changes found

Copy link
Contributor

github-actions bot commented Sep 4, 2025

🟢 No unit test coverage changes found

All components and hooks with tests maintain the same coverage as the main branch.

@joshfarrant joshfarrant changed the title Exclude recipes from coverage Exclude recipes, fixtures, and test-utils from coverage Sep 4, 2025
Copy link
Contributor

github-actions bot commented Sep 4, 2025

🟢 No visual differences found

Our visual comparison tests did not find any differences in the UI.

@joshfarrant joshfarrant marked this pull request as ready for review September 4, 2025 14:29
@Copilot Copilot AI review requested due to automatic review settings September 4, 2025 14:29
@joshfarrant joshfarrant requested a review from a team as a code owner September 4, 2025 14:29
@joshfarrant joshfarrant requested a review from rezrah September 4, 2025 14:30
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR improves test coverage accuracy by excluding non-production code directories from coverage calculations. The change filters out recipes, fixtures, and test utilities that shouldn't be counted toward the main codebase coverage metrics.

Key changes:

  • Excludes recipes and fixtures directories from Jest coverage collection
  • Increases reported coverage from 77.22% to 85.34% by focusing on production code

@joshfarrant joshfarrant force-pushed the joshfarrant/exclude-recipes-from-coverage branch from 3188015 to ef9174b Compare September 4, 2025 14:56
@joshfarrant joshfarrant force-pushed the joshfarrant/exclude-recipes-from-coverage branch from ef9174b to 6bceac5 Compare September 5, 2025 08:25
@primer primer deleted a comment from Copilot AI Sep 5, 2025
@joshfarrant joshfarrant merged commit 46ff01f into main Sep 5, 2025
19 checks passed
@joshfarrant joshfarrant deleted the joshfarrant/exclude-recipes-from-coverage branch September 5, 2025 08:44
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.

2 participants