cross-browser-tests-runner / cross-browser-tests-runner / 195 / 15

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.

335 of 809 branches covered (41.41%)

Branch coverage included in aggregate %.

1400 of 2299 relevant lines covered (60.9%)

10.4 hits per line