search

rejeep / evm

Emacs Version Manager
223 stars 34 forks source link

Remove spurious failure message #99

Closed sambrightman closed 6 years ago

sambrightman commented 6 years ago

Test for the system command failure message rather than allowing it to be confusingly printed in the test output.

sambrightman commented 6 years ago

Converted to STDERR but left test in keeping with other tests. In future http://www.rubydoc.info/github/rspec/rspec-expectations/RSpec%2FMatchers%3Aoutput probably makes sense, to avoid injection logic everywhere and testing a specific call.