-
-
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
Cannot read property 'BLOB_URI_SCHEME' of undefined after upgrade to RN48 #4433
Comments
This seems like a change needs to be made in react-native, not in Jest. |
Yup, this is bug in react native, fixed in master while updating flow version so its not yet in the release channel. |
I was of two minds where to put it, but decided to put it with jest as
react native is working fine and as expected, so it feels as though the
JavaScript environment isn't being spin up correctly for react native
testing.
I understand Jest is supposed to be react and react native independent, but
I didn't see the react native team changing their stack to make it jest
compatible if it's building and running properly.
I will raise an issue with them
…On 6 Sep. 2017 7:33 pm, "Christoph Nakazawa" ***@***.***> wrote:
Closed #4433 <#4433>.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#4433 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AFmlbfWjNIrV5wBh5y69B850YMACNvoTks5sfmb9gaJpZM4PNsvz>
.
|
Ahh just saw the new response. Thanks Michael, so the fix is a couple of
months away then?
…On 6 Sep. 2017 7:40 pm, "Ryan Knell" ***@***.***> wrote:
I was of two minds where to put it, but decided to put it with jest as
react native is working fine and as expected, so it feels as though the
JavaScript environment isn't being spin up correctly for react native
testing.
I understand Jest is supposed to be react and react native independent,
but I didn't see the react native team changing their stack to make it jest
compatible if it's building and running properly.
I will raise an issue with them
On 6 Sep. 2017 7:33 pm, "Christoph Nakazawa" ***@***.***>
wrote:
> Closed #4433 <#4433>.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <#4433 (comment)>, or mute
> the thread
> <https://github.com/notifications/unsubscribe-auth/AFmlbfWjNIrV5wBh5y69B850YMACNvoTks5sfmb9gaJpZM4PNsvz>
> .
>
|
To anyone coming across this, facebook/react-native#15810 is the relevant issue, and you can avoid the issue for now by rolling back to node v7.4.0 |
Thanks for that @rknell ! Fixed my issue, appreciate you! |
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. |
Node version: 7.8.0
Also used NVM to upgrade to Node v8.4.0, fresh yarn global install, deleted node modules and reinstalled, error still exists.
Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Fresh project, RN0.48.1 fails with error running yarn test. I noticed the error appear on an existing project after upgrading to RN0.48.1 on every test.
Commenting out the blob requirement in the
node_modules/react-native/Libraries/Blob/URL.js:21:21
file allows the app to continue to pass all tests. The app runs without errors with no source code modification.I have a copy of my code here: https://github.com/rknell/rn0481JestBugReproduction
If the current behavior is a bug, please provide the steps to reproduce and either a repl.it demo through https://repl.it/languages/jest or a minimal repository on GitHub that we can
yarn install
andyarn test
.What is the expected behavior?
Tests continue to run as normal
Please provide your exact Jest configuration and mention your Jest, node, yarn/npm version and operating system.
Please see repo for config, further details below
Npm (on node 8 5.3.0)
Yarn 0.27.5
The text was updated successfully, but these errors were encountered: