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

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.

958 of 1039 branches covered (92.2%)

Branch coverage included in aggregate %.

2682 of 2797 relevant lines covered (95.89%)

309325.93 hits per line