Visual tests should fail on protractor 199

[Blackbaud-SteveBrush]

A pull request was merged into master even though it failed during the visual step (the step returned with a zero exit code). This should catch those errors.

See: https://api.travis-ci.org/v3/job/404683473/log.txt

[codecov-io]

Codecov Report

Merging #1830 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1830   +/-   ##
=======================================
  Coverage   99.98%   99.98%           
=======================================
  Files         410      410           
  Lines        8589     8589           
  Branches     1269     1269           
=======================================
  Hits         8588     8588           
  Misses          1        1

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5088a48...3f744e5. Read the comment docs.

[Blackbaud-BobbyEarl]

Leaving a comment. This functionality was purposefully added to skyux2, then to builder, then to here. The use-case, unfortunately, wasn't detailed very well. I'm fine removing this now as it does appear to be masking legitimate failures, but it's something we should keep an eye on before removing it from builder.