SIGINT handler is not killing process

[christian-bromann]

From @artur-michalak on January 4, 2017 12:5

The problem

When i setup webdriver.io with gulp watch and interrupt process with ctrl+c it's not closing.

Environment

• WebdriverIO version: v4.5.2

• Node.js version: v4.4.5 (v6 too)

• Standalone mode or wdio testrunner: jasmine testrunner

• if wdio testrunner, running synchronous or asynchronous tests: asynchronous

• Additional wdio packages used (if applicable):

  "wdio-dot-reporter": "0.0.6", "wdio-jasmine-framework": "^0.2.19", "wdio-selenium-standalone-service": "0.0.7", "gulp-webdriver": "^2.0.3",

Link to Selenium/WebdriverIO logs

[12:56:45] Finished 'e2e:run' after 16 s [12:56:45] Finished 'e2e:dev' after 17 s then on cmd + c click:

End selenium sessions properly ... (press crtl+c again to hard kill the runner)

Killing process, bye! (on cmd + c click) Killing process, bye! (on cmd + c click) Killing process, bye! (on cmd + c click) Killing process, bye! (on cmd + c click)

Code To Reproduce Issue [ Good To Have ]

1. Setup gulp gulp.task('e2e:run', function () { return gulp.src(config.e2e.config).pipe(webdriver({ baseUrl: 'http://localhost:' + config.e2e.port, specs: config.e2e.specs })).on('error', handleErrors); });

// production release of player gulp.task('e2e:dev', function() { gulp.watch(config.e2e.specs, ['e2e:run']); });

2. Run e2e:run task. Everything works like a charm
3. Run e2e:dev which is watching files
4. Interrupt (ctrl + c/cmd+c) twice
5. Gulp process is not killed

Details

I would guess something is wrong with logic here https://github.com/webdriverio/webdriverio/blob/68974ac33aa13fa099b1870d5a81ce5c6d8341db/lib/launcher.js#L418 Perhaps process.exit is not being called.

Copied from original issue: webdriverio/webdriverio#1798

[sandys]

i have the exact same issue. i have a feeling this is because of gulp which is not passing on signals correctly.

[gpolyn]

What is the status of this issue?

[jsdtaylor]

Not sure why this got moved from webdriverio as grunt-webdriver exhibits it as well. Any news?

[christian-bromann]

I am not using gulp-webdriver so if anyone has a solution to this any PRs would be appreciated

[stsvilik]

There is one solution that I found to be working - its not ideal but it is something. Returned through2 stream has an event finish which you could use to exit the process. Ex.

webdriver({
    baseUrl: 'http://localhost:' + config.e2e.port,
    specs: config.e2e.specs
}).on(`finish`, () => process.exit(0));

[jasp402]

@stsvilik effectively closing the process, the only bad thing is that it eliminates all the underlying sub-processes. I have built a nodeJS server and when applying this solution it stops the service.

There should be some way to get PID and stop just that process. Maybe you can use process.kill (pid [, signal])

[stsvilik]

@jasp402 If only we had reference to the right process which spawns webdriver, we could kill just that.