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.
-
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