mwrap user+dev discussion/patches/pulls/bugs/help
 help / color / mirror / code / Atom feed
* [PATCH] test_mwrap: fix test reliability
@ 2023-01-08 21:40 Eric Wong
  0 siblings, 0 replies; only message in thread
From: Eric Wong @ 2023-01-08 21:40 UTC (permalink / raw)
  To: mwrap-public

Not sure what drugs I was on when I wrote this :x
---
 test/test_mwrap.rb | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/test/test_mwrap.rb b/test/test_mwrap.rb
index 29bbdd2..176dca6 100644
--- a/test/test_mwrap.rb
+++ b/test/test_mwrap.rb
@@ -5,6 +5,7 @@ require 'test/unit'
 require 'mwrap'
 require 'rbconfig'
 require 'tempfile'
+require 'io/wait'
 
 class TestMwrap < Test::Unit::TestCase
   RB = "#{RbConfig::CONFIG['bindir']}/#{RbConfig::CONFIG['RUBY_INSTALL_NAME']}"
@@ -128,10 +129,12 @@ class TestMwrap < Test::Unit::TestCase
         assert_equal "\n", r.gets
         buf = +''
         10.times { Process.kill(:USR1, pid) }
-        while IO.select([r], nil, nil, 0.1)
+        while r.wait_readable(0.1)
           case tmp = r.read_nonblock(1000, exception: false)
           when String
             buf << tmp
+          when nil
+            break
           end
         end
         w2.close

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2023-01-08 21:40 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-08 21:40 [PATCH] test_mwrap: fix test reliability Eric Wong

Code repositories for project(s) associated with this public inbox

	https://80x24.org/mwrap.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).