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

Hanging electron process after infinite loop #11

Open
nicojs opened this issue Apr 27, 2017 · 5 comments
Open

Hanging electron process after infinite loop #11

nicojs opened this issue Apr 27, 2017 · 5 comments

Comments

@nicojs
Copy link

nicojs commented Apr 27, 2017

When killing karma if nightmare hangs in an infinite loop it does not seem to stop the electron process.

Reproduction steps:

  1. Create an infinite loop in a regular test
  2. Start karma the normal way
  3. Wait for this message (or simular)
Electron 1.6.6 (Windows 10 0.0.0): Executed 0 of 3 SUCCESS (0 secs / 0 secs)
Electron 1.6.6 (Windows 10 0.0.0) ERROR
  Disconnected, because no message in 10000 ms.
  1. Kill the process (ctrl+c)

Open your task manager. The electron process hangs.

Steps 1 and 2 can be done using

git clone https://github.com/nicojs/stryker-miniq-demo
cd stryker-miniq-demo
npm i
npm run karma

ENV:
win 10 x64, node 7.9

Original issue reported here: stryker-mutator/stryker-js#289 (comment) by @unlight

@bokuweb
Copy link
Owner

bokuweb commented Apr 27, 2017

@nicojs Thanks for your report and reproduction steps.
I'll do investigation.

Related? segment-boneyard/nightmare#688

@bokuweb
Copy link
Owner

bokuweb commented Aug 14, 2017

@nicojs I've fixed process termination. Please try v0.4.10.

@nicojs
Copy link
Author

nicojs commented Aug 14, 2017

I've just tried it but there still seems to be a hanging elektron process after killing karma with ctrl+c.

hanging-elektron-app

nicoj@NicoJ02 MINGW64 /c/z/github/nicojs/stryker-miniq-demo (master)
$ npm ls karma-nightmare
[email protected] C:\z\github\nicojs\stryker-miniq-demo
`-- [email protected]

nicoj@NicoJ02 MINGW64 /c/z/github/nicojs/stryker-miniq-demo (master)
$ ./node_modules/.bin/karma --version
Karma version: 1.6.0

@bokuweb
Copy link
Owner

bokuweb commented Aug 14, 2017

@nicojs Ummmmm, Thanks, I'll continue to investigate.

@simondel
Copy link

Hi! Is there any update on this?

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

No branches or pull requests

3 participants