add comments and code clean up

Author: Giampaolo Cimino

tests/test_copy.py

@@ -83,19 +83,18 @@ def _delay_file_utime(self, filepath, delta_sec=None):
         file_access_mod_time = int(calendar.timegm(datetime.now().timetuple())) + delta_sec
         times = (file_access_mod_time, file_access_mod_time)
         os.utime(filepath, times)
-        
+
     def test_copy_file_if_newer_dst_older(self):
         try:
             #create first dst ==> dst is older the src ==> file should be copied
             dst_dir = self._create_sandbox_dir()
             dst_file1 = self._touch(dst_dir, "file1.txt")
             self._write_file(dst_file1)
 
-            #time.sleep(3) #wait before create src file, to ensure src is older
-
             src_dir = self._create_sandbox_dir()
             src_file1 = self._touch(src_dir, "file1.txt")
             self._write_file(src_file1)
+            #ensure src file is newer than dst, changing its modification time
             self._delay_file_utime(src_file1, delta_sec=60)
 
             src_fs = open_fs('osfs://' + src_dir)
@@ -137,8 +136,6 @@ def test_copy_file_if_newer_dst_is_newer(self):
             src_file1 = self._touch(src_dir, "file1.txt")
             self._write_file(src_file1)
 
-            #time.sleep(3) #wait before create dst file, to ensure dst is older
-
             dst_dir = self._create_sandbox_dir()
             dst_file1 = self._touch(dst_dir, "file1.txt")
             self._write_file(dst_file1)
@@ -164,11 +161,10 @@ def test_copy_fs_if_newer_dst_older(self):
             dst_file1 = self._touch(dst_dir, "file1.txt")
             self._write_file(dst_file1)
 
-            #time.sleep(3) #wait before create src file, to ensure src is older
-
             src_dir = self._create_sandbox_dir()
             src_file1 = self._touch(src_dir, "file1.txt")
             self._write_file(src_file1)
+            #ensure src file is newer than dst, changing its modification time
             self._delay_file_utime(src_file1, delta_sec=60)
 
             src_fs = open_fs('osfs://' + src_dir)
@@ -200,7 +196,7 @@ def test_copy_fs_if_newer_when_dst_doesnt_exists(self):
 
             src_file2 = self._touch(src_dir, "one_level_down" + os.sep + "file2.txt")
             self._write_file(src_file2)
-            
+
             dst_dir = self._create_sandbox_dir()
 
             src_fs = open_fs('osfs://' + src_dir)
@@ -230,11 +226,10 @@ def test_copy_fs_if_newer_dont_copy_when_dst_exists(self):
             src_file1 = self._touch(src_dir, "file1.txt")
             self._write_file(src_file1)
 
-            #time.sleep(3) #wait before create dst file, to ensure dst is older
-
             dst_dir = self._create_sandbox_dir()
             dst_file1 = self._touch(dst_dir, "file1.txt")
             self._write_file(dst_file1)
+            #ensure dst file is newer than src, changing its modification time
             self._delay_file_utime(dst_file1, delta_sec=60)
 
             src_fs = open_fs('osfs://' + src_dir)
@@ -271,6 +266,7 @@ def test_copy_dir_if_newer_one_dst_doesnt_exist(self):
             dst_dir = self._create_sandbox_dir()
             dst_file1 = self._touch(dst_dir, "file1.txt")
             self._write_file(dst_file1)
+            #ensure dst file is newer than src, changing its modification time
             self._delay_file_utime(dst_file1, delta_sec=60)
 
             src_fs = open_fs('osfs://' + src_dir)
@@ -300,7 +296,6 @@ def test_copy_dir_if_newer_same_fs(self):
             dst_dir = self._create_sandbox_dir(home=src_dir)
 
             src_fs = open_fs('osfs://' + src_dir)
-            # dst_fs = open_fs('osfs://' + dst_dir)
 
             copied = []
             def on_copy(src_fs, src_path, dst_fs, dst_path):