diff options
author | Samuel Williams <samuel.williams@oriontransfer.co.nz> | 2020-07-16 15:19:47 +1200 |
---|---|---|
committer | Samuel Williams <samuel.williams@oriontransfer.co.nz> | 2020-07-20 13:20:58 +1200 |
commit | e6e9cef06e9023a80991518297d9f2c0c12488ce (patch) | |
tree | a9226cb928c7d2600dcbbb1bbdecbbce0eb277f9 | |
parent | a3ac1bf450d2aa0c6b0b34fee417992b4352ff2c (diff) | |
download | ruby-e6e9cef06e9023a80991518297d9f2c0c12488ce.tar.gz |
Improve consistency of tests.
-rw-r--r-- | test/fiber/scheduler.rb | 1 | ||||
-rw-r--r-- | test/fiber/test_enumerator.rb | 6 | ||||
-rw-r--r-- | test/fiber/test_io.rb | 11 |
3 files changed, 13 insertions, 5 deletions
diff --git a/test/fiber/scheduler.rb b/test/fiber/scheduler.rb index 820f441dcd..740496674a 100644 --- a/test/fiber/scheduler.rb +++ b/test/fiber/scheduler.rb @@ -1,6 +1,7 @@ # frozen_string_literal: true require 'fiber' +require 'socket' begin require 'io/nonblock' diff --git a/test/fiber/test_enumerator.rb b/test/fiber/test_enumerator.rb index d332726dc3..f88657cdc4 100644 --- a/test/fiber/test_enumerator.rb +++ b/test/fiber/test_enumerator.rb @@ -1,19 +1,19 @@ # frozen_string_literal: true require 'test/unit' -require 'socket' require_relative 'scheduler' class TestFiberEnumerator < Test::Unit::TestCase MESSAGE = "Hello World" def test_read_characters - skip unless defined?(UNIXSocket) + skip "UNIXSocket is not defined!" unless defined?(UNIXSocket) i, o = UNIXSocket.pair + unless i.nonblock? && o.nonblock? i.close o.close - skip + skip "I/O is not non-blocking!" end message = String.new diff --git a/test/fiber/test_io.rb b/test/fiber/test_io.rb index e7f06f7429..bf200061d8 100644 --- a/test/fiber/test_io.rb +++ b/test/fiber/test_io.rb @@ -6,10 +6,15 @@ class TestFiberIO < Test::Unit::TestCase MESSAGE = "Hello World" def test_read - skip unless defined?(UNIXSocket) + skip "UNIXSocket is not defined!" unless defined?(UNIXSocket) i, o = UNIXSocket.pair - skip unless i.nonblock? && o.nonblock? + + unless i.nonblock? && o.nonblock? + i.close + o.close + skip "I/O is not non-blocking!" + end message = nil @@ -31,5 +36,7 @@ class TestFiberIO < Test::Unit::TestCase thread.join assert_equal MESSAGE, message + assert_predicate(i, :closed?) + assert_predicate(o, :closed?) end end |