test-cmd.rb 0.11.1 → 0.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 82a43ee0666fd9baea96c1974997092b1a739f730327e0a30ffa5d2e5b48cdaa
-  data.tar.gz: 2ca5079265f236de839baa8c6731433f99a9b0650c60e7c122db03c2b1a5e7bc
+  metadata.gz: d794fcd3192700e47d0964074f19732cd9ebc6881804d267cdb182fd31c19f4a
+  data.tar.gz: 139a5dda07f237f60918421a1ceefddb75f31807aaa9bb394c1f5e8680001a86
 SHA512:
-  metadata.gz: 4a6ec40cdce6d791ca2e4880cd6bc31af6151706bb92dd467c0d2b99ce3551e2dd13efd2ecea46d32c2e2feb30995466ade1b16529e93d52e2321b2b29233788
-  data.tar.gz: 3b286a1f7e87e1066837b4a6c5f61421409637becb71792ab06919b5b7620759bd0f6b036e6694cc1f18e14144733bdefb117bd98a61f2c66a4b9f540661f52d
+  metadata.gz: ad22764743a6f3e5d7a15457c8fe495f3f03598ca145cdb31754e69ce24a8a7c59bc76d74eb6022b2231b1ec311439e70da3bbfdc8c240cdcabb9f7c595d8fbf
+  data.tar.gz: 9cb6ed663a9c544a8a6d1b83781251a36eddf114d821f26d2f79c705d8945ea7d69215d77e627d8b44e8b82db203029bc4d28fa0eb161f541291259a66ea76fc

data/README.md

@@ -8,8 +8,8 @@ a command.
 ### Callbacks
 
 The success and failure callbacks provide hooks for when
-a command exits successfully or unsuccessfully. The callback
-is passed an instance of
+a command exits successfully or unsuccessfully. The callbacks
+are passed an instance of
 [Test::Cmd](https://0x1eef.github.io/x/test-cmd.rb/Test/Cmd.html):
 
 ``` ruby
@@ -55,46 +55,6 @@ class CmdTest < Test::Unit::TestCase
 end
 ```
 
-### IO#sync
-
-Sometimes it can be neccessary to bypass Ruby's internal buffer and flush
-output to the operating system immediately, otherwise there can be unexpected
-results. Consider the following example, where the output will be
-`bar\nfoo\n` rather than `foo\nbar\n`:
-
-``` ruby
-##
-# test.rb
-fork do
-  sleep(1)
-  puts "bar"
-end
-puts "foo"
-Process.wait
-
-##
-# cmd.rb
-p cmd("ruby", "test.rb").stdout # => "bar\nfoo\n"
-```
-
-And with output flushed to the operating system immediately:
-
-``` ruby
-##
-# test.rb
-$stdout.sync = true
-fork do
-  sleep(1)
-  puts "bar"
-end
-puts "foo"
-Process.wait
-
-##
-# cmd.rb
-p cmd("ruby", "test.rb").stdout # => "foo\nbar\n"
-```
-
 ## Documentation
 
 A complete API reference is available at
@@ -102,8 +62,6 @@ A complete API reference is available at
 
 ## Install
 
-**Rubygems.org**
-
 test-cmd.rb can be installed via rubygems.org:
 
     gem install test-cmd.rb

data/lib/test/cmd.rb

@@ -1,13 +1,12 @@
-module Test
-end unless defined?(Test)
+unless defined?(Test)
+  module Test
+  end
+end
 
 ##
 # test-cmd.rb provides an object oriented interface
 # for spawning a command.
 class Test::Cmd
-  require "tempfile"
-  require "securerandom"
-
   ##
   # @param [String] cmd
   #  A command to spawn
@@ -37,11 +36,13 @@ class Test::Cmd
 
     tap do
       @spawned = true
-      @out_io, @err_io = spawn_io
-      Process.spawn(@cmd, *@argv, {out: @out_io, err: @err_io})
+      @out_io, out = IO.pipe
+      @err_io, err = IO.pipe
+      Process.spawn(@cmd, *@argv, {out:, err:})
       Process.wait
       @status = $?
     ensure
+      [out, err].each(&:close)
       [stdout, stderr]
     end
   end
@@ -52,7 +53,7 @@ class Test::Cmd
   def stdout
     @stdout ||= begin
       spawn
-      out_io.tap(&:rewind).read.tap { out_io.close }
+      out_io.read.tap { out_io.close }
     rescue IOError
       @stdout
     end
@@ -64,7 +65,7 @@ class Test::Cmd
   def stderr
     @stderr ||= begin
       spawn
-      err_io.tap(&:rewind).read.tap { err_io.close }
+      err_io.read.tap { err_io.close }
     rescue IOError
       @stderr
     end
@@ -141,23 +142,6 @@ class Test::Cmd
   private
 
   attr_reader :out_io, :err_io
-
-  def spawn_io
-    [
-      [".testcmd.stdout.#{ns}.", SecureRandom.alphanumeric(3)],
-      [".testcmd.stderr.#{ns}.", SecureRandom.alphanumeric(3)]
-    ].map do
-      Tempfile.new(_1).tap do |file|
-        File.chmod(0, file.path)
-      ensure
-        file.unlink
-      end
-    end
-  end
-
-  def ns
-    [Process.pid, object_id].join(".")
-  end
 end
 
 module Kernel

data/test/{test_cmd_test.rb → cmd_test.rb}

@@ -3,6 +3,7 @@ require_relative "setup"
 class Test::Cmd
   class Test < Test::Unit::TestCase
     private
+
     def ruby(str)
       cmd "ruby", "-e", str
     end
@@ -57,7 +58,7 @@ class Test::Cmd
       end
       puts "foo"
       Process.wait
-    CODE
+      CODE
       assert_equal "foo\nbar\n", ruby(code).stdout
     end
   end
@@ -93,13 +94,6 @@ class Test::Cmd
       end
     end
 
-    def test_io_unlink_after_spawn
-      %i[out_io err_io].each do |io|
-        path = spawned_command.send(io).__getobj__.path
-        assert_equal false, File.exist?(path)
-      end
-    end
-
     private
 
     def spawned_command

data/test-cmd.rb.gemspec

@@ -5,7 +5,7 @@ Gem::Specification.new do |gem|
   gem.authors = ["0x1eef"]
   gem.email = ["0x1eef@protonmail.com"]
   gem.homepage = "https://github.com/0x1eef/test-cmd.rb#readme"
-  gem.version = "0.11.1"
+  gem.version = "0.12.0"
   gem.required_ruby_version = ">= 3.0"
   gem.licenses = ["0BSD"]
   gem.files = `git ls-files`.split($/)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-cmd.rb
 version: !ruby/object:Gem::Version
-  version: 0.11.1
+  version: 0.12.0
 platform: ruby
 authors:
 - '0x1eef'
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-05-17 00:00:00.000000000 Z
+date: 2024-06-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: test-unit
@@ -99,8 +99,8 @@ files:
 - lib/test-cmd.rb
 - lib/test/cmd.rb
 - test-cmd.rb.gemspec
+- test/cmd_test.rb
 - test/setup.rb
-- test/test_cmd_test.rb
 homepage: https://github.com/0x1eef/test-cmd.rb#readme
 licenses:
 - 0BSD