From 6d67d9c2af57233743187a92b7e651174d6eb42b Mon Sep 17 00:00:00 2001
From: Eric Wong <normalperson@yhbt.net>
Date: Mon, 26 Aug 2013 00:35:03 +0000
Subject: process (qx): disambiguate err/err_str, add no_raise

I forgot :err is already handled by Process.spawn, so split out
the functionality into err_str where we want to use it.

Also, add a :no_raise flag which will allow us to better handle
avprobe/soxi calls which can fail and avoid needless exceptions.
---
 test/test_process.rb | 29 +++++++++++++++++++++++++----
 1 file changed, 25 insertions(+), 4 deletions(-)

(limited to 'test')

diff --git a/test/test_process.rb b/test/test_process.rb
index a480312..012f18f 100644
--- a/test/test_process.rb
+++ b/test/test_process.rb
@@ -4,9 +4,30 @@
 require './test/helper'
 require 'dtas/process'
 class TestProcess < Minitest::Unit::TestCase
- include DTAS::Process
+  include DTAS::Process
 
- def test_qx_env
-   assert_equal "WORLD\n", qx({"HELLO" => "WORLD"}, 'echo $HELLO')
- end
+  def test_qx_env
+    assert_equal "WORLD\n", qx({"HELLO" => "WORLD"}, 'echo $HELLO')
+  end
+
+  def test_qx_err
+    err = "/dev/null"
+    assert_equal "", qx('echo HELLO >&2', err: err)
+    assert_equal "/dev/null", err
+  end
+
+  def test_qx_err_str
+    s = ""
+    assert_equal "", qx('echo HELLO >&2', err_str: s)
+    assert_equal "HELLO\n", s
+  end
+
+  def test_qx_raise
+    assert_raises(RuntimeError) { qx('false') }
+  end
+
+  def test_qx_no_raise
+    status = qx('false', no_raise: true)
+    refute status.success?, status.inspect
+  end
 end
-- 
cgit v1.2.3-24-ge0c7