test-cmd.rb 2.7.0 → 2.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c42ecb86960f6c733054ea3cd7ed041c23accec019f7addb7a2a595020b507bf
-  data.tar.gz: 05fd256aa2c70bb23e8e9ef3e3f8e91eb9bcff9fa2680f48a059960333e74b06
+  metadata.gz: a9c323a0dbc2f142481d09008829ca2f3e6d8dc53d4f4c3dcf1efacc3096f58b
+  data.tar.gz: d3ff8c6062fe90054e4953a5ea69c6aacb4048b420c6b6344b6985ebee751e29
 SHA512:
-  metadata.gz: e2c4e2b3858f7b090359a46756a54408d28c729603f0f47ccda5a68c35e2c11dd807dade25cf79d88af938861a112b582d9ecea0198d4b2023ab6fba504ba63d
-  data.tar.gz: 7efb00552507bbbb2b1d66e599ccdd165e7acb3628f5a5e97daa2ec0bdafca5e63ebce274d937a737224836db806a171d4f8bbd07b2195bc092aa529ddacc2d8
+  metadata.gz: 13e73947059bbb45f6c9c6af29619e7d22751af6dda7574aa5c2b87485786d0c8e2f2e88f2739c9c4087c88e55536c6f18e617ce033fd2a0616494e72fea185b
+  data.tar.gz: 6556f3ff5a3bc05f71ecbb4242ed3cccdfd7aa5f3dc528ca09daf6c850e797595cf1de6c3c426b7a4e1061145588c84215ba034a09228fe09f5a9a51ea059d1b

data/lib/test/command/version.rb

@@ -4,5 +4,5 @@ end unless defined?(Test)
 class Test::Command
   ##
   # @string [String]
-  VERSION = "2.7.0"
+  VERSION = "2.7.2"
 end

data/lib/test/command.rb

@@ -102,8 +102,10 @@ class Test::Command
       @producer = Thread.new do
         @stdout = read_limited(@out.r, @limit[:stdout])
         @stderr = read_limited(@err.r, @limit[:stderr])
-        Process.wait
-        @status = $?
+        @status = Process.waitpid2(@pid).last
+      rescue Errno::ECHILD
+        errpid  = Process.spawn("false")
+        @status = Process.waitpid2(errpid).last
       ensure
         @out.r.close
         @err.r.close
@@ -117,6 +119,7 @@ class Test::Command
     # writer thread does not block forever.
     @in_r.close unless @in_r.equal?(IO::NULL)
     @status = Process.waitpid2(Process.spawn("false")).last
+    self
   end
 
   ##

data/test/cmd_test.rb

@@ -239,5 +239,68 @@ class Test::Command
       assert_equal true, ::Test::Command.new("/a/path/that/is/not/found").command_not_found?
       assert_equal true, ::Test::Command.new("/a/path/that/is/not/found").not_found?
     end
+
+    def test_spawn_returns_self_when_not_found
+      command = ::Test::Command.new("/a/path/that/is/not/found")
+      assert_equal command, command.spawn
+    end
+
+    def test_spawn_sets_not_found_on_self
+      command = ::Test::Command.new("/a/path/that/is/not/found")
+      command.spawn
+      assert_equal true, command.not_found?
+    end
+
+    def test_stdout_when_not_found
+      command = ::Test::Command.new("/a/path/that/is/not/found")
+      assert_equal "", command.stdout
+    end
+
+    def test_stderr_when_not_found
+      command = ::Test::Command.new("/a/path/that/is/not/found")
+      assert_equal "No such file or directory - /a/path/that/is/not/found",
+                   command.stderr
+    end
+  end
+end
+
+class Test::Command
+  ##
+  # A command's status belongs to the command, and is not taken from
+  # another child that happens to be ready to be reaped.
+  #
+  # A bare `Process.wait` reaps whichever child is ready, so a child
+  # that has exited and been reaped by nobody - another test's, another
+  # thread's - could answer for a command that never spawned it.
+  class StatusTest < Test
+    def test_failure_is_not_answered_by_a_successful_decoy
+      decoy = Process.spawn("true")
+      sleep 0.1
+      assert_equal false, ::Test::Command.new("false").success?
+    ensure
+      reap(decoy)
+    end
+
+    def test_success_is_not_answered_by_a_failing_decoy
+      decoy = Process.spawn("false")
+      sleep 0.1
+      assert_equal true, ::Test::Command.new("true").success?
+    ensure
+      reap(decoy)
+    end
+
+    private
+
+    ##
+    # Reaps a decoy the command left alone, and says nothing when the
+    # command took it: the test that got there first is the point of
+    # these.
+    # @param [Integer, nil] pid
+    # @return [void]
+    def reap(pid)
+      Process.waitpid(pid) if pid
+    rescue Errno::ECHILD
+      nil
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: test-cmd.rb
 version: !ruby/object:Gem::Version
-  version: 2.7.0
+  version: 2.7.2
 platform: ruby
 authors:
 - Robert Gleeson
@@ -124,7 +124,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.20
+rubygems_version: 4.0.16
 specification_version: 4
 summary: test-cmd.rb is a Go-inspired, object-oriented interface for running commands
   on UNIX-like systems, in the spirit of Go's `os/exec`. A command can be built by