Removing the 0 items from commit message.

Author: tusharmakkar08

gitfs/worker/sync.py

@@ -174,11 +174,12 @@ def commit(self, jobs):
                 number_of_removal += len(removal_set)
                 number_of_additions += len(addition_set)
                 updates = updates | removal_set | addition_set
-
-            message = "Update {} items. Added {} items. " \
-                      "Removed {} items.".format(len(updates),
-                                                 number_of_additions,
-                                                 number_of_removal)
+            message = "Update {} items. ".format(len(updates))
+            if number_of_additions:
+                message += "Added {} items. ".format(number_of_additions)
+            if number_of_removal:
+                message += "Removed {} items. ".format(number_of_removal)
+            message = message.strip()
 
         old_head = self.repository.head.target
         new_commit = self.repository.commit(message, self.author,

tests/integrations/current/test_write.py

@@ -59,7 +59,7 @@ def test_delete_a_directory(self, gitfs_log):
             shutil.rmtree("{}/a_directory/".format(self.current_path))
 
         with pull(self.sh):
-            self.assert_commit_message("Update 2 items. Added 0 items. Removed 2 items.")
+            self.assert_commit_message("Update 2 items. Removed 2 items.")
             self.assert_new_commit()
 
         assert os.path.exists(path) is False
@@ -93,7 +93,7 @@ def test_link_a_file(self, gitfs_log):
             os.link(filename, link_name)
 
         with pull(self.sh):
-            self.assert_commit_message("Update 2 items. Added 2 items. Removed 0 items.")
+            self.assert_commit_message("Update 2 items. Added 2 items.")
 
         is_link = os.path.isfile(link_name)
         assert is_link is not False
@@ -158,7 +158,7 @@ def test_create_embedded_directory(self, gitfs_log):
                 assert os.path.exists(keep_file)
 
             self.assert_new_commit()
-            commit_msg = "Update 2 items. Added 2 items. Removed 0 items."
+            commit_msg = "Update 2 items. Added 2 items."
             self.assert_commit_message(commit_msg)
 
     def test_create_directory_inside_an_already_existing_directory(self, gitfs_log):
@@ -205,7 +205,7 @@ def test_create_embedded_directory_on_multiple_levels(self, gitfs_log):
                 assert os.path.exists(keep_file)
 
             self.assert_new_commit()
-            commit_msg = "Update {} items. Added {} items. Removed 0 items.".format(len(keep_files), len(keep_files))
+            commit_msg = "Update {} items. Added {} items.".format(len(keep_files), len(keep_files))
             self.assert_commit_message(commit_msg)
 
     def test_create_embedded_directory_big_depth(self, gitfs_log):
@@ -283,7 +283,7 @@ def test_fsync(self, gitfs_log):
 
         with pull(self.sh):
             self.assert_new_commit()
-            self.assert_commit_message("Update 1 items. Added 2 items. Removed 0 items.")
+            self.assert_commit_message("Update 1 items. Added 2 items.")
 
     def test_create(self, gitfs_log):
         filename = "{}/new_empty_file".format(self.current_path)
@@ -357,8 +357,7 @@ def test_create_multiple_files(self, gitfs_log):
             self.assert_new_commit()
 
         with pull(self.sh):
-            self.assert_commit_message("Update {} items. Added {} items. Removed 0 items.".format(no_of_files,
-                                                                                                  no_of_files))
+            self.assert_commit_message("Update {} items. Added {} items.".format(no_of_files, no_of_files))
 
     def test_delete_file(self, gitfs_log):
         filename = "{}/deletable_file".format(self.current_path)