-
-
Notifications
You must be signed in to change notification settings - Fork 6.6k
Add failing test for request for ... is not yet fulfilled
#11437
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
base: main
Are you sure you want to change the base?
Add failing test for request for ... is not yet fulfilled
#11437
Conversation
Perfect, thanks @nicolo-ribaudo! I'll try to find the time soonish to dig into this |
@nicolo-ribaudo You apparently need to update test snapshots for
|
The test will fail regardless (or rather, the snapshot will then encode a failing test as correct, which it's not 🙂 ) |
FWIW, I believe the underlying issue here is one of the ones tracked in nodejs/node#37648 (specifically the shared cache thing), so I'm waiting for that to be resolved upstream before spending more time on it in Jest |
✅ Deploy Preview for jestjs ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
a6156c1
to
d62aff4
Compare
Is this still relevant? |
d62aff4
to
7a52abd
Compare
Let's find out |
Yes, still relevant |
Summary
This is a test for #11434