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

test(repo): Add cmd to clear test cache #1880

Merged
merged 2 commits into from
Oct 19, 2023
Merged

Conversation

tmilewski
Copy link
Member

Description

Allow for clearing of test cache.

This is largely intended for Jest, but can be easily expanded for other options.

Notes:

  • The separate script is required because the required arguments can't be passed to all packages as they're not all directly utilizing Jest.
  • --useStderr is required as the call fails if the cache dir doesn't exist. We can safely ignore that.

Checklist

  • npm test runs as expected.
  • npm run build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other: Test (Jest) tooling

Packages affected

  • @clerk/clerk-js
  • @clerk/clerk-react
  • @clerk/nextjs
  • @clerk/remix
  • @clerk/types
  • @clerk/themes
  • @clerk/localizations
  • @clerk/clerk-expo
  • @clerk/backend
  • @clerk/clerk-sdk-node
  • @clerk/shared
  • @clerk/fastify
  • @clerk/chrome-extension
  • gatsby-plugin-clerk
  • build/tooling/chore

@tmilewski tmilewski self-assigned this Oct 13, 2023
@tmilewski tmilewski requested a review from a team as a code owner October 13, 2023 17:23
@changeset-bot
Copy link

changeset-bot bot commented Oct 13, 2023

🦋 Changeset detected

Latest commit: a8b3563

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

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

Not sure what this means? Click here to learn what changesets are.

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

@LekoArts
Copy link
Member

Is this the result of you running into problems or why would you need to clear the Jest cache?

@tmilewski
Copy link
Member Author

@LekoArts I had been running into points where, yes, I had to clear it. Normally, I'd just clear it with the added flags to the main script, however, we can't because they don't all use Jest causing a failure whenever it hits QUnit.

@tmilewski tmilewski force-pushed the tom/jest-cache-clear branch from 42cc32b to 287aba6 Compare October 18, 2023 21:24
@tmilewski tmilewski enabled auto-merge October 18, 2023 21:24
@tmilewski tmilewski added this pull request to the merge queue Oct 19, 2023
Merged via the queue into main with commit d003418 Oct 19, 2023
11 of 12 checks passed
@tmilewski tmilewski deleted the tom/jest-cache-clear branch October 19, 2023 08:53
@clerk-cookie
Copy link
Collaborator

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@clerk clerk locked as resolved and limited conversation to collaborators Oct 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants