Replies: 12 comments 1 reply
-
Hi, did you already try to clean your node_modules? |
Beta Was this translation helpful? Give feedback.
-
Thanks @timdeschryver, yes I did try deleting node_modules a few times, and just tried your suggestion of building first. still getting the same errors. I did commit and push changes and the ci builds and tests appear to be working so it must be something Im missing in my setup. I'm a bit stumped. |
Beta Was this translation helpful? Give feedback.
-
Hi @cam-m 👋 You probably saw in the issue template description that reproduction is required when reporting a bug. Please provide Stackblitz or GitHub repo with a reproduction. |
Beta Was this translation helpful? Give feedback.
-
Thanks @markostanimirovic , I'm not sure how i would provide a reproduction since all i've got is a fork of the ngrx/platform repo with some changes for a pull request. When I push the changes to the PR the CI run tests appear to work and pass fine. They just don't work on my machine. I will try checking out the repo fresh and see if that fixes it. |
Beta Was this translation helpful? Give feedback.
-
Oh, I thought you have this issue in another repo. 😅 Did you try:
|
Beta Was this translation helpful? Give feedback.
-
Also, try switching to Node |
Beta Was this translation helpful? Give feedback.
-
Ha no worries :)
I did try that with no success (although not entirely at the cmd line).
I’ll post back on the result of a clean repo build.
|
Beta Was this translation helpful? Give feedback.
-
ooh you might be onto something there, I did recently bump up node so will
look at that too
|
Beta Was this translation helpful? Give feedback.
-
Hi @cam-m, Did you fix tests by switching to Node |
Beta Was this translation helpful? Give feedback.
-
No that didn’t work unfortunately.
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: Marko Stanimirović ***@***.***>
Sent: Sunday, February 5, 2023 6:44:53 AM
To: ngrx/platform ***@***.***>
Cc: Cameron Maunder ***@***.***>; Mention ***@***.***>
Subject: Re: [ngrx/platform] Jest failed to parse a file (Issue #3764)
Hi @cam-m<https://github.com/cam-m>,
Did you fix tests by switching to Node 14.20?
—
Reply to this email directly, view it on GitHub<#3764 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAEZXDNHYP5WVC23ELXKI63WV2WTLANCNFSM6AAAAAAUNEFDSY>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
I'll convert this issue to a Q&A discussion. I use these versions on my machine (Windows 11 OS):
|
Beta Was this translation helpful? Give feedback.
-
After trying numerous node versions (14, 16, 18) I just took a fresh clone of the repo, and (after one false start due to what appears to be an internal node issue) I got all tests to run and pass. None the wiser as to what cause the initial issue, but I can move on. For the record: node: v16.19.1 |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Which @ngrx/* package(s) are the source of the bug?
component-store, component, data, effects, entity, eslint-plugin, router-store, schematics, store-devtools, store
Minimal reproduction of the bug/regression with instructions
I'm sure I'm missing something stupid, but trying commit a PR and getting this when I run yarn test:
and the error:
This is just one example, I get the same issue thrown on multiple other spec files
It seemed to work fine 2 months ago?
Minimal reproduction of the bug/regression with instructions
run yarn test
Versions of NgRx, Angular, Node, affected browser(s) and operating system(s)
NGRX 15.2.1
Windows 10
Other information
No response
I would be willing to submit a PR to fix this issue
Beta Was this translation helpful? Give feedback.
All reactions