diff options
author | Nobuyoshi Nakada <nobu@ruby-lang.org> | 2020-12-08 11:43:42 +0900 |
---|---|---|
committer | Nobuyoshi Nakada <nobu@ruby-lang.org> | 2020-12-08 11:43:42 +0900 |
commit | ea18c8bb96c019a609b37671ceec5f51d61476cd (patch) | |
tree | 7ae5beb5d7f30fe517173b7e4d2f29ae70635921 /test/ruby | |
parent | 5c2ff88be2e515613dfe54823e8429656f688e9f (diff) |
Made method-local instance variables local variables
Diffstat (limited to 'test/ruby')
-rw-r--r-- | test/ruby/test_dir.rb | 40 |
1 files changed, 16 insertions, 24 deletions
diff --git a/test/ruby/test_dir.rb b/test/ruby/test_dir.rb index cf18af134c..d614dc69cf 100644 --- a/test/ruby/test_dir.rb +++ b/test/ruby/test_dir.rb @@ -88,60 +88,52 @@ class TestDir < Test::Unit::TestCase end def test_chdir - @pwd = Dir.pwd - @env_home = ENV["HOME"] - @env_logdir = ENV["LOGDIR"] + pwd = Dir.pwd + env_home = ENV["HOME"] + env_logdir = ENV["LOGDIR"] ENV.delete("HOME") ENV.delete("LOGDIR") assert_raise(Errno::ENOENT) { Dir.chdir(@nodir) } assert_raise(ArgumentError) { Dir.chdir } - ENV["HOME"] = @pwd + ENV["HOME"] = pwd Dir.chdir do - assert_equal(@pwd, Dir.pwd) + assert_equal(pwd, Dir.pwd) assert_raise(RuntimeError) { Dir.chdir(@root) } - assert_equal(@pwd, Dir.pwd) + assert_equal(pwd, Dir.pwd) Dir.chdir(@root) do assert_equal(@root, Dir.pwd) end - assert_equal(@pwd, Dir.pwd) + assert_equal(pwd, Dir.pwd) end ensure begin - Dir.chdir(@pwd) + Dir.chdir(pwd) rescue - abort("cannot return the original directory: #{ @pwd }") - end - if @env_home - ENV["HOME"] = @env_home - else - ENV.delete("HOME") - end - if @env_logdir - ENV["LOGDIR"] = @env_logdir - else - ENV.delete("LOGDIR") + abort("cannot return the original directory: #{ pwd }") end + ENV["HOME"] = env_home + ENV["LOGDIR"] = env_logdir end def test_chdir_conflict - @pwd = Dir.pwd + pwd = Dir.pwd q = Queue.new t = Thread.new do q.pop - Dir.chdir(@pwd) rescue $! + Dir.chdir(pwd) rescue $! end - Dir.chdir(@pwd) do + Dir.chdir(pwd) do q.push nil assert_instance_of(RuntimeError, t.value) end t = Thread.new do q.pop - Dir.chdir(@pwd){} rescue $! + Dir.chdir(pwd){} rescue $! end - Dir.chdir(@pwd) do + Dir.chdir(pwd) do q.push nil assert_instance_of(RuntimeError, t.value) end |