Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(jest-globals-import): fix false negative in used globals check #575

Merged
merged 2 commits into from May 11, 2024

Conversation

puglyfe
Copy link
Contributor

@puglyfe puglyfe commented May 10, 2024

Closes #574

This adds an extra check to ensure that any used jest globals don't already have a corresponding ImportSpecifier.

@puglyfe puglyfe changed the title jest-globals-import: fix false negative in used globals check fix(jest-globals-import): fix false negative in used globals check May 10, 2024
Copy link
Owner

@skovhus skovhus left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link

codecov bot commented May 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.60%. Comparing base (a94e2b5) to head (654e875).
Report is 48 commits behind head on main.

❗ Current head 654e875 differs from pull request most recent head fb8ccef. Consider uploading reports for the commit fb8ccef to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #575      +/-   ##
==========================================
+ Coverage   92.38%   92.60%   +0.22%     
==========================================
  Files          26       27       +1     
  Lines        1944     2002      +58     
  Branches      405      415      +10     
==========================================
+ Hits         1796     1854      +58     
  Misses        102      102              
  Partials       46       46              

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

@skovhus skovhus enabled auto-merge (squash) May 11, 2024 10:53
@skovhus skovhus merged commit 907545d into skovhus:main May 11, 2024
1 check passed
@puglyfe puglyfe deleted the jest-imports-scope branch May 13, 2024 16:15
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.

jest-globals-import: false negative when global is explicitly imported
2 participants