spec/helpers/ruby_exe_spec.rb in mspec-1.5.17 vs spec/helpers/ruby_exe_spec.rb in mspec-1.5.18

@@ -1,7 +1,8 @@
-require File.dirname(__FILE__) + '/../spec_helper'
-require 'mspec/helpers/ruby_exe'
+require 'spec_helper'
+require 'mspec/guards'
+require 'mspec/helpers'
 require 'rbconfig'
 
 class RubyExeSpecs
 end
 
@@ -29,15 +30,10 @@
   it "returns ENV['RUBY_EXE'] when passed :env" do
     ENV['RUBY_EXE'] = "kowabunga"
     @script.ruby_exe_options(:env).should == "kowabunga"
   end
 
-  it "returns 'bin/rbx' when passed :engine and RUBY_NAME is 'rbx'" do
-    Object.const_set :RUBY_NAME, 'rbx'
-    @script.ruby_exe_options(:engine).should == 'bin/rbx'
-  end
-
   it "returns 'bin/jruby' when passed :engine and RUBY_NAME is 'jruby'" do
     Object.const_set :RUBY_NAME, 'jruby'
     @script.ruby_exe_options(:engine).should == 'bin/jruby'
   end
 
@@ -49,21 +45,50 @@
   it "returns 'maglev-ruby' when passed :engine and RUBY_NAME is 'maglev'" do
     Object.const_set :RUBY_NAME, 'maglev'
     @script.ruby_exe_options(:engine).should == 'maglev-ruby'
   end
 
+  it "returns 'topaz' when passed :engine and RUBY_NAME is 'topaz'" do
+    Object.const_set :RUBY_NAME, 'topaz'
+    @script.ruby_exe_options(:engine).should == 'topaz'
+  end
+
   it "returns RUBY_NAME + $(EXEEXT) when passed :name" do
-    bin = RUBY_NAME + (Config::CONFIG['EXEEXT'] || Config::CONFIG['exeext'] || '')
+    bin = RUBY_NAME + (RbConfig::CONFIG['EXEEXT'] || RbConfig::CONFIG['exeext'] || '')
     name = File.join ".", bin
     @script.ruby_exe_options(:name).should == name
   end
 
   it "returns $(bindir)/$(RUBY_INSTALL_NAME) + $(EXEEXT) when passed :install_name" do
-    bin = Config::CONFIG['RUBY_INSTALL_NAME'] + (Config::CONFIG['EXEEXT'] || Config::CONFIG['exeext'] || '')
-    name = File.join Config::CONFIG['bindir'], bin
+    bin = RbConfig::CONFIG['RUBY_INSTALL_NAME'] + (RbConfig::CONFIG['EXEEXT'] || RbConfig::CONFIG['exeext'] || '')
+    name = File.join RbConfig::CONFIG['bindir'], bin
     @script.ruby_exe_options(:install_name).should == name
   end
+
+  describe "under Rubinius" do
+    before :each do
+      @ruby_version = RUBY_VERSION
+    end
+
+    after :each do
+      Object.const_set :RUBY_VERSION, @ruby_version
+    end
+
+    it "returns 'bin/rbx' when passed :engine, RUBY_NAME is 'rbx' and RUBY_VERSION < 1.9" do
+      Object.const_set :RUBY_VERSION, "1.8.7"
+      Object.const_set :RUBY_NAME, 'rbx'
+
+      @script.ruby_exe_options(:engine).should == 'bin/rbx'
+    end
+
+    it "returns 'bin/rbx -X19' when passed :engine, RUBY_NAME is 'rbx' and RUBY_VERSION >= 1.9" do
+      Object.const_set :RUBY_VERSION, "1.9.2"
+      Object.const_set :RUBY_NAME, 'rbx'
+
+      @script.ruby_exe_options(:engine).should == 'bin/rbx -X19'
+    end
+  end
 end
 
 describe "#resolve_ruby_exe" do
   before :all do
     @verbose = $VERBOSE
@@ -83,28 +108,39 @@
   it "returns the value returned by #ruby_exe_options if it exists and is executable" do
     PlatformGuard.stub!(:windows?).and_return(false)
     @script.should_receive(:ruby_exe_options).and_return(@name)
     File.should_receive(:exists?).with(@name).and_return(true)
     File.should_receive(:executable?).with(@name).and_return(true)
+    File.should_receive(:expand_path).with(@name).and_return(@name)
     @script.resolve_ruby_exe.should == @name
   end
 
   it "returns the value returned by #ruby_exe_options if it exists on Windows platforms" do
     PlatformGuard.stub!(:windows?).and_return(true)
     @script.should_receive(:ruby_exe_options).and_return(@name)
     File.should_receive(:exists?).with(@name).and_return(true)
     File.should_not_receive(:executable?)
+    File.should_receive(:expand_path).with(@name).and_return(@name)
     @script.resolve_ruby_exe.should == @name
   end
 
+  it "expands the path portion of the result of #ruby_exe_options" do
+    PlatformGuard.stub!(:windows?).and_return(false)
+    @script.should_receive(:ruby_exe_options).and_return("#{@name} -Xfoo")
+    File.should_receive(:exists?).with(@name).and_return(true)
+    File.should_receive(:executable?).with(@name).and_return(true)
+    File.should_receive(:expand_path).with(@name).and_return("/usr/bin/#{@name}")
+    @script.resolve_ruby_exe.should == "/usr/bin/#{@name} -Xfoo"
+  end
+
   it "returns nil if no exe is found" do
     File.should_receive(:exists?).at_least(:once).and_return(false)
     @script.resolve_ruby_exe.should be_nil
   end
 end
 
-describe Object, "#ruby_exe" do
+describe Object, "#ruby_cmd" do
   before :all do
     @verbose = $VERBOSE
     $VERBOSE = nil
 
     @ruby_flags = ENV["RUBY_FLAGS"]
@@ -123,37 +159,87 @@
     Object.const_set :RUBY_EXE, @ruby_exe
     ENV["RUBY_FLAGS"] = @ruby_flags
     $VERBOSE = @verbose
   end
 
-  it "executes the argument if it is a file that exists" do
+  it "returns a command that runs the given file if it is a file that exists" do
     File.should_receive(:exists?).with(@file).and_return(true)
-    @script.should_receive(:`).with("ruby_spec_exe -w -Q some/ruby/file.rb")
-    @script.ruby_exe @file
+    @script.ruby_cmd(@file).should == "ruby_spec_exe -w -Q some/ruby/file.rb"
   end
 
-  it "executes the file with options and arguments" do
+  it "includes the given options and arguments with a file" do
     File.should_receive(:exists?).with(@file).and_return(true)
-    @script.should_receive(:`).with(
-      "ruby_spec_exe -w -Q -w -Cdir some/ruby/file.rb < file.txt")
-    @script.ruby_exe @file, :options => "-w -Cdir", :args => "< file.txt"
+    @script.ruby_cmd(@file, :options => "-w -Cdir", :args => "< file.txt").should ==
+      "ruby_spec_exe -w -Q -w -Cdir some/ruby/file.rb < file.txt"
   end
 
-  it "executes the argument with -e" do
+  it "returns a command that runs code using -e" do
     File.should_receive(:exists?).with(@code).and_return(false)
-    @script.should_receive(:`).with(
-      %(ruby_spec_exe -w -Q -e "some \\"real\\" 'ruby' code"))
-    @script.ruby_exe @code
+    @script.ruby_cmd(@code).should == %(ruby_spec_exe -w -Q -e "some \\"real\\" 'ruby' code")
   end
 
-  it "executes the code with options and arguments" do
+  it "includes the given options and arguments with -e" do
     File.should_receive(:exists?).with(@code).and_return(false)
-    @script.should_receive(:`).with(
-      %(ruby_spec_exe -w -Q -W0 -Cdir -e "some \\"real\\" 'ruby' code" < file.txt))
-    @script.ruby_exe @code, :options => "-W0 -Cdir", :args => "< file.txt"
+    @script.ruby_cmd(@code, :options => "-W0 -Cdir", :args => "< file.txt").should ==
+      %(ruby_spec_exe -w -Q -W0 -Cdir -e "some \\"real\\" 'ruby' code" < file.txt)
   end
 
-  it "executes with options and arguments but without code or file" do
-    @script.should_receive(:`).with("ruby_spec_exe -w -Q -c > file.txt")
-    @script.ruby_exe nil, :options => "-c", :args => "> file.txt"
+  it "returns a command with options and arguments but without code or file" do
+    @script.ruby_cmd(nil, :options => "-c", :args => "> file.txt").should ==
+      "ruby_spec_exe -w -Q -c > file.txt"
+  end
+end
+
+describe Object, "#ruby_exe" do
+  before :all do
+    @script = RubyExeSpecs.new
+  end
+
+  before :each do
+    @script.stub!(:`)
+  end
+
+  it "executes (using `) the result of calling #ruby_cmd with the given arguments" do
+    code = "code"
+    options = {}
+    @script.should_receive(:ruby_cmd).with(code, options).and_return("ruby_cmd")
+    @script.should_receive(:`).with("ruby_cmd")
+    @script.ruby_exe(code, options)
+  end
+
+  describe "with :dir option" do
+    it "executes the command in the given working directory" do
+      Dir.should_receive(:chdir).with("tmp")
+      @script.ruby_exe nil, :dir => "tmp"
+    end
+  end
+
+  describe "with :env option" do
+    it "preserves the values of existing ENV keys" do
+      ENV["ABC"] = "123"
+      ENV.should_receive(:[]).with("RUBY_FLAGS")
+      ENV.should_receive(:[]).with("ABC")
+      @script.ruby_exe nil, :env => { :ABC => "xyz" }
+    end
+
+    it "adds the :env entries to ENV" do
+      ENV.should_receive(:[]=).with("ABC", "xyz")
+      @script.ruby_exe nil, :env => { :ABC => "xyz" }
+    end
+
+    it "deletes the :env entries in ENV when an exception is raised" do
+      ENV.should_receive(:delete).with("XYZ")
+      @script.ruby_exe nil, :env => { :XYZ => "xyz" }
+    end
+
+    it "resets the values of existing ENV keys when an exception is raised" do
+      ENV["ABC"] = "123"
+      ENV.should_receive(:[]=).with("ABC", "xyz")
+      ENV.should_receive(:[]=).with("ABC", "123")
+
+      @script.should_receive(:`).and_raise(Exception)
+      lambda do
+        @script.ruby_exe nil, :env => { :ABC => "xyz" }
+      end.should raise_error(Exception)
+    end
   end
 end