right_popen 1.1.3 → 3.0.1

data/lib/right_popen/linux/accumulator.rb

@@ -1,127 +0,0 @@
-#--  -*- mode: ruby; encoding: utf-8 -*-
-# Copyright: Copyright (c) 2011 RightScale, Inc.
-#
-# Permission is hereby granted, free of charge, to any person obtaining
-# a copy of this software and associated documentation files (the
-# 'Software'), to deal in the Software without restriction, including
-# without limitation the rights to use, copy, modify, merge, publish,
-# distribute, sublicense, and/or sell copies of the Software, and to
-# permit persons to whom the Software is furnished to do so, subject to
-# the following conditions:
-#
-# The above copyright notice and this permission notice shall be
-# included in all copies or substantial portions of the Software.
-#
-# THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
-# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-# MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
-# IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
-# CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
-# TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
-# SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
-#++
-
-module RightScale
-  module RightPopen
-
-    # @deprecated this seems like test harness code smell, not production code
-    class Accumulator
-      READ_CHUNK_SIZE = 4096
-
-      def initialize(process, inputs, read_callbacks, outputs, write_callbacks)
-        warn 'WARNING: RightScale::RightPopen::Accumulator is deprecated and will be removed.'
-        @process = process
-        @inputs = inputs
-        @outputs = outputs
-        null = Proc.new {}
-        @reads = {}
-        @writes = {}
-        inputs.zip(read_callbacks).each do |pair|
-          input, callback = pair
-          @reads[input] = callback
-        end
-        outputs.zip(write_callbacks).each do |pair|
-          output, callback = pair
-          @writes[output] = callback
-        end
-        @writebuffers = {}
-        @status = nil
-      end
-
-      def status
-        unless @status
-          @status = ::Process.waitpid2(@process.pid, ::Process::WNOHANG)
-        end
-        @status
-      end
-
-      def tick(sleep_time = 0.1)
-        return true unless @status.nil?
-
-        status
-
-        inputs = @inputs.dup
-        outputs = @outputs.dup
-        ready = nil
-        while ready.nil?
-          begin
-            # in theory, we should note "exceptional conditions" and
-            # permit procs for those, too.  In practice there are only
-            # two times when exceptional conditions occur: out of band
-            # data in TCP connections and "packet mode" for
-            # pseudoterminals.  We care about neither of these,
-            # therefore ignore exceptional conditions.
-            ready = IO.select(inputs, outputs, nil, sleep_time)
-          rescue Errno::EAGAIN, Errno::EINTR
-          end
-        end unless inputs.empty? && outputs.empty?
-
-        ready[0].each do |fdes|
-          if fdes.eof?
-            fdes.close
-            @inputs.delete(fdes)
-          else
-            chunk = fdes.readpartial(READ_CHUNK_SIZE)
-            @reads[fdes].call(chunk) if @reads[fdes]
-          end
-        end unless ready.nil? || ready[0].nil?
-        ready[1].each do |fdes|
-          buffered = @writebuffers[fdes]
-          buffered = @writes[fdes].call if @writes[fdes] if buffered.nil? || buffered.empty?
-          if buffered.nil?
-            fdes.close
-            @outputs.delete(fdes)
-          elsif !buffered.empty?
-            begin
-              amount = fdes.write_nonblock buffered
-              @writebuffers[fdes] = buffered[amount..-1]
-            rescue Errno::EPIPE
-              # subprocess closed the pipe; fine.
-              fdes.close
-              @outputs.delete(fdes)
-            end
-          end
-        end unless ready.nil? || ready[1].nil?
-
-        return !@status.nil?
-      end
-
-      def number_waiting_on
-        @inputs.size + @outputs.size
-      end
-
-      def cleanup
-        @inputs.each {|p| p.close unless p.closed? }
-        @outputs.each {|p| p.close unless p.closed? }
-        @status = ::Process.waitpid2(@process.pid) if @status.nil?
-      end
-
-      def run_to_completion(sleep_time=0.1)
-        until tick(sleep_time)
-          break if number_waiting_on == 0
-        end
-        cleanup
-      end
-    end
-  end
-end

data/lib/right_popen/linux/utilities.rb

@@ -1,94 +0,0 @@
-#--  -*- mode: ruby; encoding: utf-8 -*-
-# Copyright: Copyright (c) 2011 RightScale, Inc.
-#
-# Permission is hereby granted, free of charge, to any person obtaining
-# a copy of this software and associated documentation files (the
-# 'Software'), to deal in the Software without restriction, including
-# without limitation the rights to use, copy, modify, merge, publish,
-# distribute, sublicense, and/or sell copies of the Software, and to
-# permit persons to whom the Software is furnished to do so, subject to
-# the following conditions:
-#
-# The above copyright notice and this permission notice shall be
-# included in all copies or substantial portions of the Software.
-#
-# THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
-# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-# MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
-# IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
-# CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
-# TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
-# SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
-#++
-
-require File.expand_path(File.join(File.dirname(__FILE__), "process"))
-require File.expand_path(File.join(File.dirname(__FILE__), "accumulator"))
-
-module RightScale
-  module RightPopen
-
-    # @deprecated this seems like test harness code smell, not production code
-    module Utilities
-      module_function
-
-      SIGNAL_LOOKUP = Signal.list.invert
-
-      def reason(status)
-        if status.exitstatus
-          "with exit status #{status.exitstatus}"
-        else
-          "due to SIG#{SIGNAL_LOOKUP[status.termsig]}"
-        end
-      end
-      private :reason
-      
-      def run(cmd, parameters={})
-        status, out, err = run_collecting_output(cmd, parameters)
-        unless status.success?
-          raise "Command \"#{cmd}\" failed #{reason(status)}: " +
-            "stdout #{out}, stderr #{err}"
-        end
-        [out, err]
-      end
-
-      def run_with_stdin_collecting_output(cmd, input, parameters={})
-        out = StringIO.new
-        err = StringIO.new
-        first = true
-        status = run_with_blocks(cmd,
-                                 Proc.new {
-                                   if (first)
-                                     first = false
-                                     input
-                                   else
-                                     nil
-                                   end},
-                                 Proc.new {|s| out.write(s)},
-                                 Proc.new {|s| err.write(s)})
-        [status, out.string, err.string]
-      end
-      alias_method :run_input, :run_with_stdin_collecting_output
-
-      def run_collecting_output(cmd, parameters={})
-        out = StringIO.new
-        err = StringIO.new
-        status = run_with_blocks(cmd, nil, Proc.new {|s| out.write(s)},
-                                 Proc.new {|s| err.write(s)})
-        [status, out.string, err.string]
-      end
-      alias_method :spawn, :run_collecting_output
-
-      def run_with_blocks(cmd, stdin_block, stdout_block, stderr_block, parameters={})
-        warn 'WARNING: RightScale::RightPopen::Utilities are deprecated and will be removed.'
-        process = Process.new(parameters)
-        process.spawn(cmd, ::RightScale::RightPopen::TargetProxy.new(parameters))
-        process.wait_for_exec
-        a = Accumulator.new(process,
-                            [process.stdout, process.stderr], [stdout_block, stderr_block],
-                            [process.stdin], [stdin_block])
-        a.run_to_completion
-        a.status[1]
-      end
-    end
-  end
-end

data/lib/right_popen/version.rb

@@ -1,28 +0,0 @@
-#--  -*- mode: ruby; encoding: utf-8 -*-
-# Copyright: Copyright (c) 2011-2013 RightScale, Inc.
-#
-# Permission is hereby granted, free of charge, to any person obtaining
-# a copy of this software and associated documentation files (the
-# 'Software'), to deal in the Software without restriction, including
-# without limitation the rights to use, copy, modify, merge, publish,
-# distribute, sublicense, and/or sell copies of the Software, and to
-# permit persons to whom the Software is furnished to do so, subject to
-# the following conditions:
-#
-# The above copyright notice and this permission notice shall be
-# included in all copies or substantial portions of the Software.
-#
-# THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
-# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-# MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
-# IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
-# CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
-# TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
-# SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
-#++
-
-module RightScale
-  module RightPopen
-    VERSION = "1.1.3"
-  end
-end

data/spec/background.rb

@@ -1,28 +0,0 @@
-#--  -*- mode: ruby; encoding: utf-8 -*-
-# Copyright: Copyright (c) 2011 RightScale, Inc.
-#
-# Permission is hereby granted, free of charge, to any person obtaining
-# a copy of this software and associated documentation files (the
-# 'Software'), to deal in the Software without restriction, including
-# without limitation the rights to use, copy, modify, merge, publish,
-# distribute, sublicense, and/or sell copies of the Software, and to
-# permit persons to whom the Software is furnished to do so, subject to
-# the following conditions:
-#
-# The above copyright notice and this permission notice shall be
-# included in all copies or substantial portions of the Software.
-#
-# THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
-# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-# MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
-# IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
-# CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
-# TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
-# SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
-#++
-
-fork {
-  sleep 30
-  puts "Done!"
-}
-exit 0

data/spec/increment.rb

@@ -1,2 +0,0 @@
-input = gets.to_i
-$stdout.puts input + 1

data/spec/print_env.rb

@@ -1,2 +0,0 @@
-puts "__test__=#{ENV['__test__']}" if ENV['__test__']
-puts "PATH=#{ENV['PATH']}"

data/spec/produce_mixed_output.rb

@@ -1,12 +0,0 @@
-count = ARGV[0] ? ARGV[0].to_i : 1
-exit_code = ARGV[1] ? ARGV[1].to_i : 0
-
-STDOUT.sync=true
-STDERR.sync=true
-
-count.times do |i|
-  $stderr.puts "stderr #{i}" if 0 == i % 10
-  $stdout.puts "stdout #{i}"
-end
-
-exit exit_code

data/spec/produce_output.rb

@@ -1,2 +0,0 @@
-$stdout.puts ARGV[0]
-$stderr.puts ARGV[1]

data/spec/produce_status.rb

@@ -1 +0,0 @@
-exit ARGV[0].to_i

data/spec/produce_stderr_only.rb

@@ -1,5 +0,0 @@
-count = ARGV[0] ? ARGV[0].to_i : 1
-
-count.times do |i|
-  $stderr.puts "stderr #{i}"
-end

data/spec/produce_stdout_only.rb

@@ -1,5 +0,0 @@
-count = ARGV[0] ? ARGV[0].to_i : 1
-
-count.times do |i|
-  $stdout.puts "stdout #{i}"
-end

data/spec/right_popen/linux/accumulator_spec.rb

@@ -1,267 +0,0 @@
-#--  -*- mode: ruby; encoding: utf-8 -*-
-# Copyright: Copyright (c) 2011 RightScale, Inc.
-#
-# Permission is hereby granted, free of charge, to any person obtaining
-# a copy of this software and associated documentation files (the
-# 'Software'), to deal in the Software without restriction, including
-# without limitation the rights to use, copy, modify, merge, publish,
-# distribute, sublicense, and/or sell copies of the Software, and to
-# permit persons to whom the Software is furnished to do so, subject to
-# the following conditions:
-#
-# The above copyright notice and this permission notice shall be
-# included in all copies or substantial portions of the Software.
-#
-# THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
-# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-# MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
-# IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
-# CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
-# TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
-# SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
-#++
-
-require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
-
-module RightScale::RightPopen
-  describe Accumulator do
-    before(:each) do
-      @process = flexmock("process")
-      @process.should_receive(:pid).and_return(42)
-    end
-
-    describe "#tick" do
-      context 'with a live child' do
-        before(:each) do
-          @input = flexmock("input")
-          @output = flexmock("output")
-          @read = flexmock("read")
-          @write = flexmock("write")
-        end
-        
-        it 'should skip calling select if no pipes are given' do
-          a = Accumulator.new(@process, [], [], [], [])
-          flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).once.and_return(nil)
-          a.tick.should be_false
-        end
-
-        it 'should just check waitpid if the select times out' do
-          a = Accumulator.new(@process, [@input], [@read], [@output], [@write])
-          flexmock(::IO).should_receive(:select).with([@input], [@output], nil, 0.1).once.and_return([[], [], []])
-          flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).once.and_return(nil)
-          a.tick.should be_false
-        end
-
-        it 'should use the timeout value in the select' do
-          value = flexmock("value")
-          a = Accumulator.new(@process, [@input], [@read], [@output], [@write])
-          flexmock(::IO).should_receive(:select).with([@input], [@output], nil, value).once.and_return([[], [], []])
-          flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).once.and_return(nil)
-          a.tick(value).should be_false
-        end
-
-        it 'should retry the select when seeing Errno::EAGAIN or Errno::EINTR' do
-          a = Accumulator.new(@process, [@input], [@read], [@output], [@write])
-          flexmock(::IO).should_receive(:select).with([@input], [@output], nil, 0.1).times(3).and_raise(Errno::EAGAIN).and_raise(Errno::EINTR).and_return([[], [], []])
-          flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).once.and_return(nil)
-          a.tick.should be_false
-        end
-
-        it 'should read data from the pipe and call the reader if it is ready' do
-          value = flexmock("value")
-          a = Accumulator.new(@process, [@input], [@read], [@output], [@write])
-          flexmock(::IO).should_receive(:select).with([@input], [@output], nil, 0.1).once.and_return([[@input], [], []])
-          @input.should_receive(:eof?).once.and_return(false)
-          @input.should_receive(:readpartial).with(Accumulator::READ_CHUNK_SIZE).once.and_return(value)
-          @read.should_receive(:call).with(value).once
-          flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).once.and_return(nil)
-          a.tick.should be_false
-        end
-
-        it 'should read data from the pipe and throw it away if no reader' do
-          value = flexmock("value")
-          a = Accumulator.new(@process, [@input], [], [@output], [@write])
-          flexmock(::IO).should_receive(:select).with([@input], [@output], nil, 0.1).once.and_return([[@input], [], []])
-          @input.should_receive(:eof?).once.and_return(false)
-          @input.should_receive(:readpartial).with(Accumulator::READ_CHUNK_SIZE).once.and_return(value)
-          flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).once.and_return(nil)
-          a.tick.should be_false
-        end
-
-        it 'should call the writer and then write data to the pipe if it is ready' do
-          value = flexmock("value")
-          a = Accumulator.new(@process, [@input], [@read], [@output], [@write])
-          flexmock(::IO).should_receive(:select).with([@input], [@output], nil, 0.1).once.and_return([[], [@output], []])
-          @write.should_receive(:call).with().once.and_return(value)
-          value.should_receive(:[]).with(30..-1).and_return("")
-          value.should_receive("empty?").and_return(false)
-          @output.should_receive(:write_nonblock).with(value).once.and_return(30)
-          flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).once.and_return(nil)
-          a.tick.should be_false
-        end
-
-        it 'should only call the writer when it is stalling' do
-          value = flexmock("value")
-          other = flexmock("other value")
-          a = Accumulator.new(@process, [@input], [@read], [@output], [@write])
-          flexmock(::IO).should_receive(:select).with([@input], [@output], nil, 0.1).and_return([[], [@output], []])
-          @write.should_receive(:call).with().once.and_return(value)
-          value.should_receive(:[]).with(30..-1).and_return(other)
-          other.should_receive(:[]).with(20..-1).and_return("")
-          value.should_receive("empty?").and_return(false)
-          other.should_receive("empty?").and_return(false)
-          @output.should_receive(:write_nonblock).with(value).once.and_return(30)
-          @output.should_receive(:write_nonblock).with(other).once.and_return(20)
-          flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).and_return(nil)
-          a.tick.should be_false
-          a.tick.should be_false
-        end
-
-        it 'should not read data from the pipe any more if EOF has been reached' do
-          value = flexmock("value")
-          a = Accumulator.new(@process, [@input], [@read], [], [])
-          flexmock(::IO).should_receive(:select).with([@input], [], nil, 0.1).once.and_return([[@input], [], []])
-          @input.should_receive(:eof?).once.and_return(true)
-          @input.should_receive(:close).once
-          flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).twice.and_return(nil)
-          a.tick.should be_false
-          a.tick.should be_false
-        end
-
-        it 'should not write data to the pipe any more if the caller has no more data' do
-          value = flexmock("value")
-          a = Accumulator.new(@process, [], [], [@output], [@write])
-          flexmock(::IO).should_receive(:select).with([], [@output], nil, 0.1).once.and_return([[], [@output], []])
-          @write.should_receive(:call).once.and_return(nil)
-          @output.should_receive(:close).once
-          flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).twice.and_return(nil)
-          a.tick.should be_false
-          a.tick.should be_false
-        end
-
-        it 'should not write data to the pipe any more if the caller is nil' do
-          a = Accumulator.new(@process, [], [], [@output], [nil])
-          flexmock(::IO).should_receive(:select).with([], [@output], nil, 0.1).once.and_return([[], [@output], []])
-          @output.should_receive(:close).once
-          flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).twice.and_return(nil)
-          a.tick.should be_false
-          a.tick.should be_false
-        end
-      end
-
-      it 'should update the status if waitpid is successful' do
-        a = Accumulator.new(@process, [], [], [], [])
-        status = flexmock("status")
-        flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).once.and_return(status)
-        a.tick.should be_true
-      end
-
-      it 'should return true if the process has already been waited on' do
-        a = Accumulator.new(@process, [], [], [], [])
-        status = flexmock("status")
-        flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).once.and_return(status)
-        a.tick.should be_true
-        a.tick.should be_true
-      end
-    end
-
-    describe "#number_waiting_on" do
-      it 'should return 0 when no pipes are left' do
-        a = Accumulator.new(@process, [], [], [], [])
-        a.number_waiting_on.should == 0
-      end
-
-      it 'should return 1 when one pipe is left' do
-        pipe = flexmock("pipe")
-        a = Accumulator.new(@process, [pipe], [nil], [], [])
-        a.number_waiting_on.should == 1
-      end
-
-      it 'should return add readers and writers' do
-        pipe = flexmock("pipe")
-        a = Accumulator.new(@process, [pipe, pipe, pipe], [], [pipe], [])
-        a.number_waiting_on.should == 4
-      end
-
-      it 'should transition from 0 to 1 as pipes are removed' do
-        pipe = flexmock("pipe")
-        read = flexmock("read")
-        a = Accumulator.new(@process, [pipe], [read], [], [])
-        flexmock(::IO).should_receive(:select).with([pipe], [], nil, 0.1).once.and_return([[pipe], [], []])
-        pipe.should_receive(:eof?).and_return(true)
-        pipe.should_receive(:close)
-        flexmock(::Process).should_receive(:waitpid2).with(42, ::Process::WNOHANG).once.and_return(nil)
-        a.number_waiting_on.should == 1
-        a.tick.should be_false
-        a.number_waiting_on.should == 0
-      end
-    end
-
-    describe "#cleanup" do
-      it 'should do nothing if no pipes are left open and the process is reaped' do
-        pending 'needs refactoring if actually in use'
-        a = Accumulator.new(@process, [], [], [], [])
-        flexmock(::Process).should_receive(:waitpid2).never
-        a.cleanup
-      end
-
-      it 'should just call waitpid if no pipes are left open' do
-        value = flexmock("value")
-        a = Accumulator.new(@process, [], [], [], [])
-        flexmock(::Process).should_receive(:waitpid2).with(42).once.and_return(value)
-        a.cleanup
-      end
-
-      it 'should close all open pipes' do
-        pending 'needs refactoring if actually in use'
-        a, b, c = flexmock("a"), flexmock("b"), flexmock("c")
-        acc = Accumulator.new(@process, [a, b], [], [c], [])
-        [a, b].each {|fdes| fdes.should_receive(:close).with().once }
-        [a, b].each {|fdes| fdes.should_receive(:closed?).and_return(false) }
-        [c].each {|fdes| fdes.should_receive(:closed?).and_return(true) }
-        flexmock(::Process).should_receive(:waitpid2).never
-        acc.cleanup
-      end
-
-      it 'should close all open pipes and reap zombies if needed' do
-        value = flexmock("value")
-        a, b, c = flexmock("a"), flexmock("b"), flexmock("c")
-        acc = Accumulator.new(@process, [b, c], [], [a], [])
-        [a, b].each {|fdes| fdes.should_receive(:close).with().once }
-        [a, b].each {|fdes| fdes.should_receive(:closed?).and_return(false) }
-        [c].each {|fdes| fdes.should_receive(:closed?).and_return(true) }
-        flexmock(::Process).should_receive(:waitpid2).with(42).once.and_return(value)
-        acc.cleanup
-      end
-    end
-
-    describe "#run_to_completion" do
-      it 'should run ticks until it is true' do
-        value = flexmock("value")
-        acc = flexmock(Accumulator.new(@process, [], [], [], []))
-        acc.should_receive(:tick).with(value).times(3).and_return(false).and_return(false).and_return(true)
-        acc.should_receive(:cleanup).once
-        acc.should_receive(:number_waiting_on).and_return(1)
-        acc.run_to_completion(value)
-      end
-      
-      it 'should abort the loop early if there are no remaining pipes' do
-        value = flexmock("value")
-        acc = flexmock(Accumulator.new(@process, [], [], [], []))
-        acc.should_receive(:tick).with(value).twice.and_return(false)
-        acc.should_receive(:cleanup).once
-        acc.should_receive(:number_waiting_on).and_return(1).and_return(0)
-        acc.run_to_completion(value)
-      end
-      
-      it 'should abort the loop after one iteration if there never were any pipes' do
-        value = flexmock("value")
-        acc = flexmock(Accumulator.new(@process, [], [], [], []))
-        acc.should_receive(:tick).with(value).once.and_return(false)
-        acc.should_receive(:cleanup).once
-        acc.should_receive(:number_waiting_on).and_return(0)
-        acc.run_to_completion(value)
-      end
-    end
-  end
-end