-
Notifications
You must be signed in to change notification settings - Fork 309
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
Intern 4.4.0 is causing OOM Node errors when running Firefox #967
Comments
Hmmm...well that's annoying. The self tests for 4.4.0 passed last week, both locally and on Travis, but today FF (64.0.2) is hanging for me. 4.3.3 and 4.3.4 appear to be working properly. |
From a quick test, it looks like the issue may be with Leadfoot. Downgrading leadfoot to 2.2.3 seems to resolve the issue. |
@jason0x43 I had a hunch that was it too! Thank you for looking into it! |
The issue is corrected in Leadfoot 2.2.5. I published Intern 4.4.1 to ensure the updated Leadfoot is picked up in new installs. |
geckodriver doesn't recognize the /window command, and the code that was intended to handing an unknown command in that situation was causing an infinite loop. This commit allows geckodriver to use the JWP window handle commands, and also prevents the infinite loop. fixes theintern#967
Expected behavior
Tests should run
Current behavior
Running functional tests targeting Firefox causes OOM Node error.
Possible solution
When browser is switched to Chrome then the problem goes away. We had this happen in 2 different projects. See webcompat/webcompat.com#2764
Steps to reproduce (for bugs)
I am gonna see if I can produce a simplified example once I have time
Environment
Intern version: 4.4.0
Node version: 8
NPM version: 3
Browser version: Firefox 58 or 64, etc
Additional information
Some info one of our engineers discovered, not sure if relevant:
Crash on a Mac after ~3 minutes:
CI Log:
It seems the browser just hangs and tries to do something:
The text was updated successfully, but these errors were encountered: