karma-runner / karma-browserstack-launcher

A Karma plugin. Launch any browser on BrowserStack!
MIT License
150 stars 82 forks source link

fixes issue #62 by making sure to call workerManager.stopPolling in u… #131

Closed cjiang-ias closed 6 years ago

cjiang-ias commented 6 years ago

…se cases where the tunnel is launched externally

The bit of code that called workerManager.stopPolling() only existed in the code path when the plugin was in charge of launching the browser (within createBrowserStackTunnel).

I've added that logic to the code path where the BS tunnel is launched externally. This fixes the issue where the karma-browserstack-launcher process hangs even after completing tests.

googlebot commented 6 years ago

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers
cjiang-ias commented 6 years ago

am going to open this again from my personal github account