process_helper 0.0.4 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 154c47c6e0861152617336f08883dcba48f765ea
-  data.tar.gz: 13eb97ca9b9f66ebbb57eee5bc64f9beccfa7f61
+  metadata.gz: e70edb04eaa1cef03255ffd6976dca7b6218afaf
+  data.tar.gz: a5d9dc19df23427d66e3974504cefeea00e636f8
 SHA512:
-  metadata.gz: fafd1e2c31f8a260fc222c3c78c48910447141c2235205ea755c1ba489773a611ee2e9a692d8b9867058b808df9fb86698f85922e053f972539138eee39b31bf
-  data.tar.gz: 128f4cde70775dac6c33302b60fcbcc39e633dd61404ea567364a1da2ec1dbce7ff93b05858eb111289e261e2a461eceefd96f59aa0586259b878f2fe8934ede
+  metadata.gz: 1ed2dd1a21c2e738f34971adf8b3328f1a30a1991dccf81b7bb6431e59fd2a0343f0e04cae2aaebf4091887148adf044d15ed52459c003d772c8db413e469ba5
+  data.tar.gz: 84a8dda43af5ac0486e709a3edd7b963221f5f34fac2c2daa1b39b87604c39ba8e081904b5500b192007d6a380e828002c0944c2bbe39b279d1772fafb23500a

data/lib/process_helper.rb

@@ -10,7 +10,7 @@ require 'stringio'
 # Full documentation at https://github.com/thewoolleyman/process_helper
 module ProcessHelper
   # Don't forget to keep version in sync with gemspec
-  VERSION = '0.0.4'.freeze
+  VERSION = '0.1.0'.freeze
 
   # rubocop:disable Style/ModuleFunction
   extend self
@@ -103,6 +103,10 @@ module ProcessHelper
         result = IO.select(nil, [stdin], nil, timeout)
         raise Timeout::Error if result.nil?
         retry
+      rescue Errno::EIO
+        # GNU/Linux raises EIO on read operation when pty slave is closed - see pty.rb docs
+        return output if options[:pseudo_terminal]
+        raise
       end
     rescue Timeout::Error
       handle_timeout_error(output, options)

data/process_helper.gemspec

@@ -3,7 +3,7 @@
 Gem::Specification.new do |spec|
   spec.name = 'process_helper'
   # Don't forget to keep version in sync with ProcessHelper::Version
-  spec.version = '0.0.4'
+  spec.version = '0.1.0'
   spec.authors = ['Chad Woolley', 'Glenn Oppegard']
   spec.email = ['oppegard@gmail.com', 'thewoolleyman@gmail.com']
   spec.summary = "Makes it easier to spawn ruby sub-processes with proper capturing /
@@ -21,6 +21,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'bundler', '~> 1.6'
   spec.add_development_dependency 'codeclimate-test-reporter', '~> 1'
   spec.add_development_dependency 'rake', '~> 10'
+  spec.add_development_dependency 'rainbow', '~> 2.1.0' # workaround for https://github.com/sickill/rainbow/issues/48#issuecomment-275454921, gem update --system no longer works
   spec.add_development_dependency 'rspec', '~> 3.1'
   spec.add_development_dependency 'rspec-retry', '~> 0.4'
   spec.add_development_dependency 'rubocop', '= 0.38.0' # exact version for static analyis libs

data/spec/options/include_output_in_exception_spec.rb

@@ -18,7 +18,7 @@ RSpec.describe ':include_output_in_exception option' do
           )
         end.to raise_error(
             ProcessHelper::UnexpectedExitStatusError,
-            /Command output: "ls:.*\/does_not_exist: No such file or directory\n"/)
+            /Command output: "ls:.*\/does_not_exist.*No such file or directory\n"/)
             .and(output(/No such file or directory/).to_stdout)
       end
 

data/spec/pty_handling_spec.rb

@@ -4,7 +4,6 @@ RSpec.describe 'pty handling' do
   attr_reader :clazz, :max_process_wait
 
   before do
-    skip('pty specs do not work on travis or circle ci (try docker with -t option)') if ENV['CI']
     @clazz = Clazz.new
   end
 
@@ -34,6 +33,15 @@ RSpec.describe 'pty handling' do
       .and(not_output.to_stderr)
   end
 
+  it 'handles linux behavior of raising Errno::EIO when pty slave is closed' do
+    # nothing really special about this test, all the specs in this file
+    # fail on linux, just documenting...
+    expect do
+      clazz.process('echo "hi" && exit 1', pty: true, exp_st: 1)
+    end.to output(/hi/).to_stdout
+      .and(not_output.to_stderr)
+  end
+
   it 'does not require a newline or flush via getch' do
     skip('TODO: this test just hangs, including in a debugger')
     expect do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: process_helper
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.1.0
 platform: ruby
 authors:
 - Chad Woolley
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-19 00:00:00.000000000 Z
+date: 2017-04-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -53,6 +53,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '10'
+- !ruby/object:Gem::Dependency
+  name: rainbow
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.1.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.1.0
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -176,7 +190,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 2.5.2
 signing_key: 
 specification_version: 4
 summary: Makes it easier to spawn ruby sub-processes with proper capturing / of stdout