[前][次][番号順一覧][スレッド一覧]

ruby-changes:51599

From: eregon <ko1@a...>
Date: Sat, 30 Jun 2018 22:53:16 +0900 (JST)
Subject: [ruby-changes:51599] eregon:r63810 (trunk): Fix spec/ruby/command_line/rubylib_spec.rb for use with make test-spec

eregon	2018-06-30 22:53:10 +0900 (Sat, 30 Jun 2018)

  New Revision: 63810

  https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=63810

  Log:
    Fix spec/ruby/command_line/rubylib_spec.rb for use with make test-spec
    
    * Current code clears ENV['RUBYLIB'], but on Windows it's needed when
      running from build 'src' (or running make test-spec).
    * Patch by MSP-Greg, from https://github.com/ruby/spec/pull/607.
    * Imported manually to fix CI on Windows, without needing a full sync.

  Modified files:
    trunk/spec/ruby/command_line/rubylib_spec.rb
Index: spec/ruby/command_line/rubylib_spec.rb
===================================================================
--- spec/ruby/command_line/rubylib_spec.rb	(revision 63809)
+++ spec/ruby/command_line/rubylib_spec.rb	(revision 63810)
@@ -3,6 +3,7 @@ require_relative '../spec_helper' https://github.com/ruby/ruby/blob/trunk/spec/ruby/command_line/rubylib_spec.rb#L3
 describe "The RUBYLIB environment variable" do
   before :each do
     @rubylib, ENV["RUBYLIB"] = ENV["RUBYLIB"], nil
+    @pre  = @rubylib.nil? ? '' : @rubylib + File::PATH_SEPARATOR
   end
 
   after :each do
@@ -11,14 +12,14 @@ describe "The RUBYLIB environment variab https://github.com/ruby/ruby/blob/trunk/spec/ruby/command_line/rubylib_spec.rb#L12
 
   it "adds a directory to $LOAD_PATH" do
     dir = tmp("rubylib/incl")
-    ENV["RUBYLIB"] = dir
+    ENV["RUBYLIB"] = @pre + dir
     paths = ruby_exe("puts $LOAD_PATH").lines.map(&:chomp)
     paths.should include(dir)
   end
 
   it "adds a File::PATH_SEPARATOR-separated list of directories to $LOAD_PATH" do
     dir1, dir2 = tmp("rubylib/incl1"), tmp("rubylib/incl2")
-    ENV["RUBYLIB"] = "#{dir1}#{File::PATH_SEPARATOR}#{dir2}"
+    ENV["RUBYLIB"] = @pre + "#{dir1}#{File::PATH_SEPARATOR}#{dir2}"
     paths = ruby_exe("puts $LOAD_PATH").lines.map(&:chomp)
     paths.should include(dir1)
     paths.should include(dir2)
@@ -27,7 +28,7 @@ describe "The RUBYLIB environment variab https://github.com/ruby/ruby/blob/trunk/spec/ruby/command_line/rubylib_spec.rb#L28
 
   it "adds the directory at the front of $LOAD_PATH" do
     dir = tmp("rubylib/incl_front")
-    ENV["RUBYLIB"] = dir
+    ENV["RUBYLIB"] = @pre + dir
     paths = ruby_exe("puts $LOAD_PATH").lines.map(&:chomp)
     if PlatformGuard.implementation? :ruby
       # In a MRI checkout, $PWD and some extra -I entries end up as
@@ -42,7 +43,7 @@ describe "The RUBYLIB environment variab https://github.com/ruby/ruby/blob/trunk/spec/ruby/command_line/rubylib_spec.rb#L43
   it "adds the directory after directories added by -I" do
     dash_i_dir = tmp("dash_I_include")
     rubylib_dir = tmp("rubylib_include")
-    ENV["RUBYLIB"] = rubylib_dir
+    ENV["RUBYLIB"] = @pre + rubylib_dir
     paths = ruby_exe("puts $LOAD_PATH", options: "-I #{dash_i_dir}").lines.map(&:chomp)
     paths.should include(dash_i_dir)
     paths.should include(rubylib_dir)
@@ -52,7 +53,7 @@ describe "The RUBYLIB environment variab https://github.com/ruby/ruby/blob/trunk/spec/ruby/command_line/rubylib_spec.rb#L53
   it "adds the directory after directories added by -I within RUBYOPT" do
     rubyopt_dir = tmp("rubyopt_include")
     rubylib_dir = tmp("rubylib_include")
-    ENV["RUBYLIB"] = rubylib_dir
+    ENV["RUBYLIB"] = @pre + rubylib_dir
     paths = ruby_exe("puts $LOAD_PATH", env: { "RUBYOPT" => "-I#{rubyopt_dir}" }).lines.map(&:chomp)
     paths.should include(rubyopt_dir)
     paths.should include(rubylib_dir)
@@ -60,7 +61,7 @@ describe "The RUBYLIB environment variab https://github.com/ruby/ruby/blob/trunk/spec/ruby/command_line/rubylib_spec.rb#L61
   end
 
   it "keeps spaces in the value" do
-    ENV["RUBYLIB"] = " rubylib/incl "
+    ENV["RUBYLIB"] = @pre + " rubylib/incl "
     out = ruby_exe("puts $LOAD_PATH")
     out.should include(" rubylib/incl ")
   end

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

[前][次][番号順一覧][スレッド一覧]