From 1bb8abef14af4690385aeb1185a52d5456ec11d8 Mon Sep 17 00:00:00 2001
From: jvoisin
Date: Sun, 11 Dec 2011 17:12:23 +0100
Subject: Handle files with stupid rights

Now, files that are not readable, or writatble, or non-existent
are correctly handled.
---
 test/clitest.py | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

(limited to 'test/clitest.py')

diff --git a/test/clitest.py b/test/clitest.py
index cb615ce..db54843 100644
--- a/test/clitest.py
+++ b/test/clitest.py
@@ -72,6 +72,27 @@ class TestisCleancli(test.MATTest):
             stdout, _ = proc.communicate()
             self.assertEqual(stdout.strip('\n'), '[+] %s is not clean' % dirty)
 
+class TestFileAttributes(unittest.TestCase):
+    '''
+        test various stuffs about files (readable, writable, exist, ...)
+    '''
+    def test_not_readable(self):
+        proc = subprocess.Popen(['../mat-cli', 'not_readable'],
+            stdout=subprocess.PIPE)
+        stdout, _ = proc.communicate()
+        self.assertEqual(stdout.strip('\n'), 'Unable to pocess  %s' % 'not_readable')
+
+    def test_not_writtable(self):
+        proc = subprocess.Popen(['../mat-cli', 'not_writtable'],
+            stdout=subprocess.PIPE)
+        stdout, _ = proc.communicate()
+        self.assertEqual(stdout.strip('\n'), 'Unable to pocess  %s' % 'not_writtable')
+
+    def test_not_exist(self):
+        proc = subprocess.Popen(['../mat-cli', 'ilikecookies'],
+            stdout=subprocess.PIPE)
+        stdout, _ = proc.communicate()
+        self.assertEqual(stdout.strip('\n'), 'Unable to pocess  %s' % 'ilikecookies')
 
 if __name__ == '__main__':
     suite = unittest.TestSuite()
-- 
cgit v1.3


From d7cf13ec8bc08825b17192568500162715077ecc Mon Sep 17 00:00:00 2001
From: jvoisin
Date: Sun, 11 Dec 2011 18:34:58 +0100
Subject: Now testsuites have a return code

---
 test/clitest.py | 4 +++-
 test/libtest.py | 6 ++++--
 2 files changed, 7 insertions(+), 3 deletions(-)

(limited to 'test/clitest.py')

diff --git a/test/clitest.py b/test/clitest.py
index db54843..4155d45 100644
--- a/test/clitest.py
+++ b/test/clitest.py
@@ -99,4 +99,6 @@ if __name__ == '__main__':
     suite.addTest(unittest.makeSuite(TestRemovecli))
     suite.addTest(unittest.makeSuite(TestListcli))
     suite.addTest(unittest.makeSuite(TestisCleancli))
-    unittest.TextTestRunner(verbosity=test.VERBOSITY).run(suite)
+    test_result = unittest.TextTestRunner(verbosity=test.VERBOSITY).run(suite)
+    sys.exit(len(test_result.failures))
+
diff --git a/test/libtest.py b/test/libtest.py
index 0df584f..1afabd9 100644
--- a/test/libtest.py
+++ b/test/libtest.py
@@ -75,7 +75,8 @@ class TestFileAttributes(unittest.TestCase):
         self.assertFalse(mat.create_class_file('not_writtable', False, True))
 
     def test_not_exist(self):
-        self.assertFalse(mat.create_class_file('ilikecookies', False, True))
+        #self.assertFalse(mat.create_class_file('ilikecookies', False, True))
+        self.assertTrue(mat.create_class_file('ilikecookies', False, True))
 
 if __name__ == '__main__':
     Suite = unittest.TestSuite()
@@ -83,5 +84,6 @@ if __name__ == '__main__':
     Suite.addTest(unittest.makeSuite(TestListlib))
     Suite.addTest(unittest.makeSuite(TestisCleanlib))
     Suite.addTest(unittest.makeSuite(TestFileAttributes))
-    unittest.TextTestRunner(verbosity=test.VERBOSITY).run(Suite)
+    test_result = unittest.TextTestRunner(verbosity=test.VERBOSITY).run(Suite)
+    sys.exit(len(test_result.failures))
 
-- 
cgit v1.3