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

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.

253 of 620 branches covered (40.81%)

Branch coverage included in aggregate %.

1109 of 1786 relevant lines covered (62.09%)

8.1 hits per line