cross-browser-tests-runner / cross-browser-tests-runner / 195 / 1

93%

master: 93%

push

travis-ci

BrowserStack Tunnel v7 Errors Fix

BrowserStack tunnel URL now provided v7 tunnel. It creates new processes
with 'fork-N' added to the identifier. This confused the Manager as it
now counts more than valid processes. Fixed Manager core logic to ignore
processes that have 'fork-N' in their tunnel identifier (if there is one).

Verified all 3 tunnels tests post the change.

Even SauceLabs has moved to 4.4.10 from 4.4.9. However; updating to that
fails the unit tests with what looks like some deep issues to solve. In
some unit tests, it is leading to a workflow where an HTTP Request is being
made without a URL. Kept the version to 4.4.9.

Updated ISSUE_TEMPLATE to include details of attachments.

229 of 596 branches covered (38.42%)

Branch coverage included in aggregate %.

778 of 1681 relevant lines covered (46.28%)

14723.54 hits per line