reduxjs / redux-templates

Official Redux templates for Vite, Create-React-App, and more
https://redux.js.org
MIT License
1.43k stars 255 forks source link

Tests fail to run after install #79

Closed kalisjoshua closed 1 year ago

kalisjoshua commented 1 year ago

Using the command npx create-react-app my-app --template redux-typescript to start a new project when attempting to run npm test I get the following output and no tests run. I have made no changes before attempting to run tests.

/usr/local/bin/node[37209]: ../src/module_wrap.cc:599:MaybeLocal<v8::Promise> node::loader::ImportModuleDynamically(Local<v8::Context>, Local<v8::Data>, Local<v8::Value>, Local<v8::String>, Local<v8::FixedArray>): Assertion `(it) != (env->id_to_function_map.end())' failed.
 1: 0x10fd5a325 node::Abort() (.cold.1) [/usr/local/bin/node]
 2: 0x10e6a6ab9 node::Abort() [/usr/local/bin/node]
 3: 0x10e6a6861 node::Assert(node::AssertionInfo const&) [/usr/local/bin/node]
 4: 0x10e6657a1 node::loader::ImportModuleDynamically(v8::Local<v8::Context>, v8::Local<v8::Data>, v8::Local<v8::Value>, v8::Local<v8::String>, v8::Local<v8::FixedArray>) [/usr/local/bin/node]
 5: 0x10e9a840d v8::internal::Isolate::RunHostImportModuleDynamicallyCallback(v8::internal::MaybeHandle<v8::internal::Script>, v8::internal::Handle<v8::internal::Object>, v8::internal::MaybeHandle<v8::internal::Object>) [/usr/local/bin/node]
 6: 0x10ee84265 v8::internal::Runtime_DynamicImportCall(int, unsigned long*, v8::internal::Isolate*) [/usr/local/bin/node]
 7: 0x10f2c9774 Builtins_CEntry_Return1_DontSaveFPRegs_ArgvInRegister_NoBuiltinExit [/usr/local/bin/node]

I running these versions:

markerikson commented 1 year ago

That looks like a Node internal error of some kind. I actually have no idea what it means just looking at it, and I don't think this is anything specific to CRA or our template.

kalisjoshua commented 1 year ago

@markerikson you are correct. This is a bug in 19.8.0. Upgrading to 19.8.1 fixed the issue. Sorry for the noise. But thank you for the quick response.