clerk / javascript

Official Javascript repository for Clerk authentication
https://clerk.com
MIT License
981 stars 221 forks source link

chore(e2e,repo): Disable turbo cache for e2e tests #3680

Closed panteliselef closed 2 weeks ago

panteliselef commented 2 weeks ago

Description

We have identified an issue where integration tests will hit the cache even when "breaking" changes are introduced in PRs. As a result, code that has not run again the e2e tests could be merged, which defeats the purpose of having required e2e tests.

Temporary disable the turbo cache for our integration tests

🚨 This PR also disables the e2e tests for AP flows related to sign-in/up. Seems like due to the above issue, a change that got merged broke those test but it was not discovered until now. The change that broke the tests has not been identified yet, but this this where tests fail, here and here

Checklist

Type of change

changeset-bot[bot] commented 2 weeks ago

🦋 Changeset detected

Latest commit: f4de407b107bba12433382c7c252ef302501a18b

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