7 files changed, 72 insertions, 14 deletions

diff --git a/CHANGELOG b/CHANGELOG
index af0f58e..53309fb 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,2 +1,9 @@
+0.2 - 21dec2011
+    * Improvements for testsuites
+    * Improve log system
+    * Handle file with weird rights
+    * Drag'n'drop support
+    * Various bugfixes
+
 0.1 - 30oct2011
     * initial version
diff --git a/RELEASE b/RELEASE
index f5b97ed..376900b 100644
--- a/RELEASE
+++ b/RELEASE
@@ -1,3 +1,9 @@
+update changelog:
+    git log
+
+create a tag
+    git tag -s $VERSION
+
 archive's creation :
     git archive --format=tar.gz --prefix=mat-$VERSION/ $VERSION > mat-$VERSION.tar.gz
 
diff --git a/TODO b/TODO
index 05749dd..5ccf195 100644
--- a/TODO
+++ b/TODO
@@ -5,18 +5,14 @@ GENERAL:
     - Fix the ugly function get_sharedir
     - Modify creation/modification date
     (except for stupid formats like zip)
-    - better log management
-    - cleaning a file (mat-gui) in a directory where you have no write
-    permission : pretend a cleaned file was created which is wrong !
-
-TESTSUITE:
-    - return code ?
 
 GUI:
     - display harmrful meta per file
-    - drag'n'drop does not work with filenames containing spaces
 
-FORMATS
+FORMATS:
     - microsoft office < 2007 (ugly and insane)
     - fix zip (fix python's ziplib)
     - videos format ?
+
+MISC:
+    - a nautilus extension (right-clic -> clean with MAT)
diff --git a/mat-cli b/mat-cli
index 530af9a..c18fefb 100755
--- a/mat-cli
+++ b/mat-cli
@@ -149,8 +149,10 @@ def main():
     for filename in filenames:
         class_file = mat.create_class_file(filename, args.backup,
             args.add2archive)
-        if class_file is not None:
+        if class_file is not False:
             func(class_file, filename, args.force)
+        else:
+            print('Unable to process %s' % filename)
 
 if __name__ == '__main__':
     main()
diff --git a/mat/mat.py b/mat/mat.py
index 343d9d5..08112ae 100644
--- a/mat/mat.py
+++ b/mat/mat.py
@@ -108,7 +108,17 @@ def create_class_file(name, backup, add2archive):
     if not os.path.isfile(name):
         # check if the file exists
         logging.error('%s is not a valid file' % name)
-        return
+        return False
+
+    if not os.access(name, os.R_OK):
+        #check read permissions
+        logging.error('%s is is not readable' % name)
+        return False
+
+    if not os.access(name, os.W_OK):
+        #check write permission
+        logging.error('%s is not writtable' % name)
+        return False
 
     filename = ''
     try:
@@ -119,7 +129,7 @@ def create_class_file(name, backup, add2archive):
     parser = hachoir_parser.createParser(filename)
     if not parser:
         logging.info('Unable to parse %s' % filename)
-        return
+        return False
 
     mime = parser.mime_type
 
@@ -135,6 +145,6 @@ def create_class_file(name, backup, add2archive):
         stripper_class = strippers.STRIPPERS[mime]
     except KeyError:
         logging.info('Don\'t have stripper for %s format' % mime)
-        return
+        return False
 
     return stripper_class(filename, parser, mime, backup, add2archive)
diff --git a/test/clitest.py b/test/clitest.py
index cb615ce..4155d45 100644
--- a/test/clitest.py
+++ b/test/clitest.py
@@ -72,10 +72,33 @@ 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()
     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 ecbc03e..bfeb548 100644
--- a/test/libtest.py
+++ b/test/libtest.py
@@ -64,11 +64,25 @@ class TestisCleanlib(test.MATTest):
             current_file = mat.create_class_file(clean, False, True)
             self.assertTrue(current_file.is_clean())
 
+class TestFileAttributes(unittest.TestCase):
+    '''
+        test various stuffs about files (readable, writable, exist, ...)
+    '''
+    def test_not_readable(self):
+        self.assertFalse(mat.create_class_file('not_readable', False, True))
+
+    def test_not_writtable(self):
+        self.assertFalse(mat.create_class_file('not_writtable', False, True))
+
+    def test_not_exist(self):
+        self.assertFalse(mat.create_class_file('ilikecookies', False, True))
 
 if __name__ == '__main__':
     Suite = unittest.TestSuite()
     Suite.addTest(unittest.makeSuite(TestRemovelib))
     Suite.addTest(unittest.makeSuite(TestListlib))
     Suite.addTest(unittest.makeSuite(TestisCleanlib))
-    unittest.TextTestRunner(verbosity=test.VERBOSITY).run(Suite)
+    Suite.addTest(unittest.makeSuite(TestFileAttributes))
+    test_result = unittest.TextTestRunner(verbosity=test.VERBOSITY).run(Suite)
+    sys.exit(len(test_result.failures))