test_workspace.rb: prefer using skip

rather than ignoring test definition to know untestability when
executing test with `-v`.

Also this simplifies check using `MiniTest::Unit::Guard#windows?`.

This change is suggested by @MSP-Greg here:
https://github.com/ruby/ruby/commit/7128849c8c5fce8df450379db54136fd21fab6ad#commitcomment-25836745

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@60937 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

1 files changed, 10 insertions, 10 deletions

diff --git a/test/irb/test_workspace.rb b/test/irb/test_workspace.rb
index d67b9f6713..0dd46175c8 100644
--- a/test/irb/test_workspace.rb
+++ b/test/irb/test_workspace.rb
@@ -32,18 +32,18 @@ module TestIRB
       end
     end
 
-    unless /mswin|mingw/ =~ RUBY_PLATFORM
-      def test_code_around_binding_with_existing_unreadable_file
-        Tempfile.create do |f|
-          code = "IRB::WorkSpace.new(binding)\n"
-          f.print(code)
-          f.close
+    def test_code_around_binding_with_existing_unreadable_file
+      skip 'chmod cannot make file unreadable on windows' if windows?
 
-          File.chmod(0, f.path)
+      Tempfile.create do |f|
+        code = "IRB::WorkSpace.new(binding)\n"
+        f.print(code)
+        f.close
 
-          workspace = eval(code, binding, f.path)
-          assert_equal(nil, workspace.code_around_binding)
-        end
+        File.chmod(0, f.path)
+
+        workspace = eval(code, binding, f.path)
+        assert_equal(nil, workspace.code_around_binding)
       end
     end