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

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.

259 of 429 branches covered (60.37%)

Branch coverage included in aggregate %.

864 of 1117 relevant lines covered (77.35%)

214063.41 hits per line