add test code and update wording format.

Author: chinyeungli

about_code_tool/genattrib.py

@@ -91,8 +91,7 @@ def convert_dict_key_to_lower_case(input_list):
 def check_about_file_existance_and_format(input_list):
     try:
         for row in input_list:
-            # Force the path to start with the '/' to do the mapping
-            # with the project structure
+            # Force the path to start with the '/' to map with the project structure
             if not row['about_file'].startswith('/'):
                 row['about_file'] = '/' + row['about_file']
         return input_list
@@ -208,12 +207,15 @@ def main(parser, options, args):
         # Clear the log file
         with open(join(dirname(output_path), LOG_FILENAME), 'w'):
             pass
-    
+
         file_handler = logging.FileHandler(join(dirname(output_path), LOG_FILENAME))
         file_logger.addHandler(file_handler)
         for error_msg in errors:
-                logger.error(error_msg)
-                file_logger.error(error_msg)
+            logger.error(error_msg)
+            file_logger.error(error_msg)
+        if errors:
+            print("%d errors detected." % len(errors))
+
     else:
         # we should never reach this
         assert False, "Unsupported option(s)."

about_code_tool/tests/test_genattrib.py

@@ -51,4 +51,13 @@ def test_check_no_about_file_not_start_with_slash(self):
     def test_check_have_about_file_not_start_with_slash(self):
         input_list = [{'Directory': '/test/', 'about_file': 'test.ABOUT'}]
         expected_list = [{'Directory': '/test/', 'about_file': '/test.ABOUT'}]
-        self.assertTrue(genattrib.check_about_file_existance_and_format(input_list) == expected_list)
+        self.assertTrue(genattrib.check_about_file_existance_and_format(input_list) == expected_list)
+
+    def test_update_path_to_about(self):
+        input_list = ['/test/test1.ABOUT', '/test/test2/', 'test/test3.c']
+        expected_list = ['/test/test1.ABOUT', '/test/test2.ABOUT', 'test/test3.c.ABOUT']
+        output = genattrib.update_path_to_about(input_list)
+        self.assertTrue(expected_list == output)
+
+
+