From 3174f06fd56685595941bd1c09f3b9cc267891ac Mon Sep 17 00:00:00 2001
From: Lubomír Sedlář <lsedlar@redhat.com>
Date: Mar 10 2016 11:47:49 +0000
Subject: [checks] Test printing in all cases


Signed-off-by: Lubomír Sedlář <lsedlar@redhat.com>

---

diff --git a/tests/test_checks.py b/tests/test_checks.py
index d3b102d..3b948be 100755
--- a/tests/test_checks.py
+++ b/tests/test_checks.py
@@ -43,10 +43,12 @@ class CheckDependenciesTestCase(unittest.TestCase):
             'create_jigdo': False
         }
 
-        with mock.patch('os.path.exists') as exists:
-            exists.side_effect = self.dont_find(['/usr/bin/jigdo-lite'])
-            result = checks.check(conf)
+        with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
+            with mock.patch('os.path.exists') as exists:
+                exists.side_effect = self.dont_find(['/usr/bin/jigdo-lite'])
+                result = checks.check(conf)
 
+        self.assertEqual('', out.getvalue())
         self.assertTrue(result)
 
     def test_isohybrid_not_required_without_productimg_phase(self):
@@ -56,10 +58,12 @@ class CheckDependenciesTestCase(unittest.TestCase):
             'runroot': True,
         }
 
-        with mock.patch('os.path.exists') as exists:
-            exists.side_effect = self.dont_find(['/usr/bin/isohybrid'])
-            result = checks.check(conf)
+        with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
+            with mock.patch('os.path.exists') as exists:
+                exists.side_effect = self.dont_find(['/usr/bin/isohybrid'])
+                result = checks.check(conf)
 
+        self.assertEqual('', out.getvalue())
         self.assertTrue(result)
 
     def test_isohybrid_not_required_on_not_bootable(self):
@@ -68,10 +72,12 @@ class CheckDependenciesTestCase(unittest.TestCase):
             'runroot': True,
         }
 
-        with mock.patch('os.path.exists') as exists:
-            exists.side_effect = self.dont_find(['/usr/bin/isohybrid'])
-            result = checks.check(conf)
+        with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
+            with mock.patch('os.path.exists') as exists:
+                exists.side_effect = self.dont_find(['/usr/bin/isohybrid'])
+                result = checks.check(conf)
 
+        self.assertEqual('', out.getvalue())
         self.assertTrue(result)
 
     def test_isohybrid_not_required_on_arm(self):
@@ -96,10 +102,12 @@ class CheckDependenciesTestCase(unittest.TestCase):
             'runroot': True,
         }
 
-        with mock.patch('os.path.exists') as exists:
-            exists.side_effect = self.dont_find(['/usr/bin/isohybrid'])
-            result = checks.check(conf)
+        with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
+            with mock.patch('os.path.exists') as exists:
+                exists.side_effect = self.dont_find(['/usr/bin/isohybrid'])
+                result = checks.check(conf)
 
+        self.assertEqual('', out.getvalue())
         self.assertTrue(result)
 
     def test_genisoimg_not_needed_in_runroot(self):
@@ -107,10 +115,12 @@ class CheckDependenciesTestCase(unittest.TestCase):
             'runroot': True,
         }
 
-        with mock.patch('os.path.exists') as exists:
-            exists.side_effect = self.dont_find(['/usr/bin/genisoimage'])
-            result = checks.check(conf)
+        with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
+            with mock.patch('os.path.exists') as exists:
+                exists.side_effect = self.dont_find(['/usr/bin/genisoimage'])
+                result = checks.check(conf)
 
+        self.assertEqual('', out.getvalue())
         self.assertTrue(result)
 
     def test_genisoimg_needed_for_productimg(self):