Add new ruby_exe's exit_status option

Author: andrykonchin

lib/mspec/helpers/ruby_exe.rb

@@ -135,16 +135,16 @@ def ruby_exe(code = :not_given, opts = {})
  code = tmpfile
  end
 
- exception = opts.fetch(:exception, true)
+ expected_exit_status = opts.fetch(:exit_status, 0)
 
  begin
  platform_is_not :opal do
  command = ruby_cmd(code, opts)
  output = `#{command}`
 
  last_status = Process.last_status
- if !last_status.success? && exception
- raise "ruby_exe(#{command}) failed: #{last_status.inspect}"
+ if last_status.exitstatus != expected_exit_status
+ raise "Expected exit status is #{expected_exit_status} but actual is #{last_status.exitstatus}. Command ruby_exe(#{command})"
  end
 
  output

spec/helpers/ruby_exe_spec.rb

@@ -221,25 +221,23 @@ class RubyExeSpecs
  end
  end
 
- describe "with :exception option" do
- context "when Ruby command fails" do
- before do
- status_failed = double(Process::Status, success?: false)
- allow(Process).to receive(:last_status).and_return(status_failed)
- end
-
- it "raises exception when exception: true" do
- expect {
- @script.ruby_exe("path", exception: true)
- }.to raise_error(%r{ruby_exe\(.+\) failed:})
- end
-
- it "does not raise exception when exception: false" do
- output = "output"
- allow(@script).to receive(:`).and_return(output)
-
- expect(@script.ruby_exe("path", exception: false)).to eq output
- end
+ describe "with :exit_status option" do
+ before do
+ status_failed = double(Process::Status, success?: false, exitstatus: 4)
+ allow(Process).to receive(:last_status).and_return(status_failed)
+ end
+
+ it "raises exception when command ends with not expected status" do
+ expect {
+ @script.ruby_exe("path", exit_status: 1)
+ }.to raise_error(%r{Expected exit status is 1 but actual is 4. Command ruby_exe\(.+\)})
+ end
+
+ it "does not raise exception when command ends with expected status" do
+ output = "output"
+ allow(@script).to receive(:`).and_return(output)
+
+ expect(@script.ruby_exe("path", exit_status: 4)).to eq output
  end
  end
 end