-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Unsupported node version #730
Comments
Well, looks like |
Hey! Are you using npm3? That might be it. I'm fixing this for the final 0.9.0 release. |
Yep, |
If you could try whether #732 solves your issue, that would be fantastic. I believe it does, so I'm gonna close out this issue but if it doesn't, I'll reopen and fix it :) |
Summary:This fixes unmock resolution for node_modules when using npm3. The way this is solved is by checking whether the parent dependency is unmocked and both modules are within a `node_modules` folder. This has taken a while, mainly because it required #599 to be merged. I also added some more caching that makes test runs even faster. The react-native test suite now completes in about 6s (down from 7.5-8s). There might be more in the near future ;) This fixes #554, #730, facebook/react#5183, facebook/relay#832 and will be part of 0.9.0. I'll publish 0.9.0-fb3 today with this fix. Closes #732 Differential Revision: D2959610 fb-gh-sync-id: c374b7a2bcdfddf768905356a08948d9156eb028 shipit-source-id: c374b7a2bcdfddf768905356a08948d9156eb028
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Hi @cpojer!
I'm trying to migrate our
rnpm
tests to Jest, but I have a weird exception here:Looks like if I use
mock-fs
from Jest tests,semver.satisfies
returnsundefined
(???). If I run the same code from REPL, it works.The text was updated successfully, but these errors were encountered: