-
Notifications
You must be signed in to change notification settings - Fork 64
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
Node crashes when using toJson #166
Comments
Did you try others enzyme-to-json version ? @boatcoder |
3.4.3, same effect.
…On Sun, Mar 8, 2020 at 4:34 PM Vincent Langlet ***@***.***> wrote:
Did you try other enzyme-to-json version ? @boatcoder
<https://github.com/boatcoder>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#166?email_source=notifications&email_token=AENX3UO2OTAKBCT2ODHGYK3RGP6LRA5CNFSM4LDZSX72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOFAWGY#issuecomment-596249371>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AENX3UM3SEJD7EGRIVH7H2TRGP6LRANCNFSM4LDZSX7Q>
.
|
Also tried it on linux (in a docker) and it crashed the same way
…On Mon, Mar 9, 2020 at 8:30 PM Mark Jones ***@***.***> wrote:
3.4.3, same effect.
On Sun, Mar 8, 2020 at 4:34 PM Vincent Langlet ***@***.***>
wrote:
> Did you try other enzyme-to-json version ? @boatcoder
> <https://github.com/boatcoder>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#166?email_source=notifications&email_token=AENX3UO2OTAKBCT2ODHGYK3RGP6LRA5CNFSM4LDZSX72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOFAWGY#issuecomment-596249371>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AENX3UM3SEJD7EGRIVH7H2TRGP6LRANCNFSM4LDZSX7Q>
> .
>
|
I'm using this on node 12 without any issue... Any ideas @adriantoine ? |
Hi, It doesn't seem to be related to this library, did you have any issues using any other node libraries? Did you try to run your tests without enzyme-to-json? Also can you provide the complete test case to reproduce the issue? Thanks! |
The only change I need to make to the project is to add enzyme-to-json.
Removing it removes the crash. I'm using storybook as well which has it's
own snapshot serializer that does not cause the problem. I'll try to make
some kind of minimal case after friday this week. Got a deadline to get
done
…On Mon, Mar 9, 2020 at 11:32 PM Adrien Antoine ***@***.***> wrote:
Hi,
It doesn't seem to be related to this library, did you have any issues
using any other node libraries? Did you try to run your tests without
enzyme-to-json? Also can you provide the complete test case to reproduce
the issue?
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#166?email_source=notifications&email_token=AENX3UP7WPLYXQ7QEF3BOJLRGWYFRA5CNFSM4LDZSX72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOJ3W3I#issuecomment-596884333>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AENX3ULXREJ2TESJ3NK47ALRGWYFRANCNFSM4LDZSX7Q>
.
|
I'm getting a very similar issue when mounting a component for snapshot testing. Tried enzyme 3.4.4 and 3.5.0 with Jest 24, 25, and 26. Tried it with using the helper Repo is here: https://github.com/20BBrown14/digital-terrain-mc-js/tree/feature/add_layout Offending test is tests/components/header/HeaderView.test.jsx |
I've tried it with v10.18.1, v10.19.0 and 12.16.1 and all of them fail the same way. Not sure what is going on here. This only started once I used toJson, and quits when I don't use it.
enzyme-to-json version: v3.4.4
The text was updated successfully, but these errors were encountered: