Skip to content
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

Upgrades joaquin #224

Closed
wants to merge 23 commits into from
Closed

Upgrades joaquin #224

wants to merge 23 commits into from

Conversation

jackbravo
Copy link
Contributor

Description

Attempts to fix cypress browser crash.

Using fix proposed at: cypress-io/github-action#104 (comment)

Performance impact

Security impact

Checklist

  • My code matches the project's code style and yarn lint:fix passes.
  • I've added tests for the new feature, and yarn test passes.
  • I have detailed the new feature in the relevant documentation.
  • I have added this feature to 'Pending' in the RELEASE_NOTES.md file (if one exists).
  • If this is a breaking change I've explained why.

@jackbravo
Copy link
Contributor Author

seems to not be running the cypress CI check, so this won't work 😛 .

@benjie
Copy link
Member

benjie commented Dec 9, 2020

Thanks for keeping chipping away at this @jackbravo. I've no idea what's up or even if it's Cypress' fault or if the issue lies elsewhere and Cypress is paying for it.

@jackbravo
Copy link
Contributor Author

@benjie I feel like I need to install a virtual machine with ubuntu 16.04 to test cypress locally. This flow of testing is very slow :-p. And sadly my work machine today is a mac.

@jackbravo jackbravo force-pushed the upgrades-joaquin branch 2 times, most recently from 1cab2ea to 41d3fd8 Compare December 10, 2020 05:47
@benjie
Copy link
Member

benjie commented Dec 10, 2020

I'm running Ubuntu LTS 20.04 and can't reproduce here. I wonder if upgrading main branch's Cypress only would have the same issues?

@jackbravo
Copy link
Contributor Author

jackbravo commented Dec 11, 2020 via email

@mattvb91
Copy link

mattvb91 commented Jan 4, 2021

@jackbravo did you manage to make any progress with the out of memory? I upgraded our actions to Ubuntu 20.04 and still experiencing the same issue

@jackbravo
Copy link
Contributor Author

@mattvb91 sadly I don't have made any progress, and haven't got the time to investigate other options.

@benjie
Copy link
Member

benjie commented Jan 5, 2021

I also attempted to fix it in #222 but no luck. Can anyone reproduce this outside of GitHub actions?

@benjie
Copy link
Member

benjie commented Jan 5, 2021

I've reproduced it locally! To do so was quite simple in the end:

NODE_ENV=test yarn build
NODE_ENV=test yarn server start &
NODE_ENV=test yarn worker start &
yarn e2e run run --browser chrome

Turns out we can't even load the browser in this circumstance either; the issue seems to be Next.js related rather than Cypress related! 🎉

@benjie
Copy link
Member

benjie commented Jan 5, 2021

We get an error like:

                               [                                                                                             
                                 {                                                                                                                              "type": "object",                                                                         
                                   "subtype": "error",                                                                                                          "className": "TypeError",                                                                                                    "description": "TypeError: Object(...) is not a function\n    at b.render (http://localhos
t:7777/_next/static/chunks/pages/_app-95a06bfb7a4260377f90.js:1:7029)\n    at Wo (http://localhost:7777/_next/static/chunks/framework.b334d46914b6052ad622.js:1:102515)\n    at jo (http://localhost:7777/_next/static/chunks/framework.b334d46914b6052ad622.js:1:102312)\n    at Wi (http://localhost:7777/_next/static/chunks/framework.b334d46914b6052ad622.js:1:143599)\n    at xu 
(http://localhost:7777/_next/static/chunks/framework.b334d46914b6052ad622.js:1:129506)\n    at Su (http://localhost:7777/_next/static/chunks/framework.b334d46914b6052ad622.js:1:129434)\n    at ku (http://localhost:7777/_next/static/chunks/framework.b
334d46914b6052ad622.js:1:129297)\n    at pu (http://localhost:7777/_next/static/chunks/framework.b334d46914b6052ad622.js:1:126263)\n    at http://localhost:7777/_next/static/chunks/framework.b334d46914b6052ad622.js:1:75652\n    at t.unstable_runWithPriority (http://localhost:7777/_next/static/chunks/framework.b334d46914b6052ad622.js:1:10121)",                              
                                   "objectId": "{\"injectedScriptId\":12,\"id\":8563}",

Seems to be related to Object(s.jsxDEV) not being a function (probably not existing).

Screenshot_20210105_174010

@benjie
Copy link
Member

benjie commented Jan 5, 2021

Fixed in 3afcb19 (#222)

@benjie benjie closed this in #222 Jan 5, 2021
@benjie
Copy link
Member

benjie commented Jan 5, 2021

Thanks for your help debugging this everyone! Turned out to be next build not working quite right in NODE_ENV=test

@jackbravo jackbravo deleted the upgrades-joaquin branch January 17, 2021 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants