Merge pull request #344 from broadinstitute/dp-fix-run-and

add test coverage on run_and_print
broadinstitute · Jun 15, 2016 · ce459c7 · ce459c7
2 parents b4332c6 + 70ee4ea
commit ce459c7
Showing 1 changed file with 46 additions and 0 deletions.
diff --git a/test/unit/test_util_misc.py b/test/unit/test_util_misc.py
@@ -4,6 +4,52 @@
 
 import util.misc
 import unittest
+import subprocess
+
+
+class TestRunAndPrint(unittest.TestCase):
+
+    def testBasicRunSuccess(self):
+        util.misc.run_and_print(['cat', '/dev/null'],
+            silent=False, buffered=False, check=False)
+        util.misc.run_and_print(['cat', '/dev/null'],
+            silent=False, buffered=False, check=True)
+        util.misc.run_and_print(['cat', '/dev/null'],
+            silent=True, buffered=False, check=False)
+        util.misc.run_and_print(['cat', '/dev/null'],
+            silent=True, buffered=False, check=True)
+        util.misc.run_and_print(['cat', '/dev/null'],
+            silent=False, buffered=True, check=False)
+        util.misc.run_and_print(['cat', '/dev/null'],
+            silent=False, buffered=True, check=True)
+        util.misc.run_and_print(['cat', '/dev/null'],
+            silent=True, buffered=True, check=False)
+        util.misc.run_and_print(['cat', '/dev/null'],
+            silent=True, buffered=True, check=True)
+
+    def testBasicRunFailDontCare(self):
+        util.misc.run_and_print(['cat', '/notdev/notnull'],
+            silent=False, buffered=False, check=False)
+        util.misc.run_and_print(['cat', '/notdev/notnull'],
+            silent=True, buffered=False, check=False)
+        util.misc.run_and_print(['cat', '/notdev/notnull'],
+            silent=False, buffered=True, check=False)
+        util.misc.run_and_print(['cat', '/notdev/notnull'],
+            silent=True, buffered=True, check=False)
+
+    def testBasicRunFailAndCatch(self):
+        self.assertRaises(subprocess.CalledProcessError,
+            util.misc.run_and_print, ['cat', '/notdev/notnull'],
+            silent=False, buffered=False, check=True)
+        self.assertRaises(subprocess.CalledProcessError,
+            util.misc.run_and_print, ['cat', '/notdev/notnull'],
+            silent=False, buffered=True, check=True)
+        self.assertRaises(subprocess.CalledProcessError,
+            util.misc.run_and_print, ['cat', '/notdev/notnull'],
+            silent=True, buffered=False, check=True)
+        self.assertRaises(subprocess.CalledProcessError,
+            util.misc.run_and_print, ['cat', '/notdev/notnull'],
+            silent=True, buffered=True, check=True)
 
 
 class TestFeatureSorter(unittest.TestCase):