Skip to content

Commit 23a1a08

Browse files
authored
Merge pull request #99 from sambrightman/failure-message
Remove spurious failure message
2 parents 9a92e9d + df1cf7c commit 23a1a08

3 files changed

Lines changed: 3 additions & 2 deletions

File tree

lib/evm/builder.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ def copy(from, to)
8686

8787
private
8888

89-
def tar_packaged(url,extension)
89+
def tar_packaged(url, extension)
9090
tar_file_path = File.join(builds_path, @name + '.tar.' + extension)
9191

9292
remote_file = Evm::RemoteFile.new(url)

lib/evm/system.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ def initialize(executable)
77
def run(*args)
88
succeeded = Kernel.system(@executable, *args)
99
unless succeeded
10-
print "Failed! See logs above for error."
10+
STDERR.puts "Failed! See logs above for error."
1111
Kernel.exit($?.exitstatus)
1212
end
1313
end

spec/evm/system_spec.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
end
2626

2727
it 'should exit if the command fails' do
28+
expect(STDERR).to receive(:puts).with('Failed! See logs above for error.')
2829
expect(Kernel).to receive(:exit)
2930

3031
# Based on http://stackoverflow.com/a/4589517

0 commit comments

Comments
 (0)