Fix tests: use more meaningful old time instead of epoch

[ppu.git] / tests / test_remove_old_files.py

diff --git a/tests/test_remove_old_files.py b/tests/test_remove_old_files.py
index 87fc3e319bdb60e5ad8a9937ec3bbd102ab97d72..ea36506dd4d4198acc97dd31abd7e8fae64581a6 100755 (executable)
--- a/tests/test_remove_old_files.py
+++ b/tests/test_remove_old_files.py
@@ -1,12 +1,16 @@
 #! /usr/bin/env python
 
+from time import time
 import os
 import shutil
+import subprocess
 import sys
 from tempfile import mkdtemp
 
 
 tmp_dir = None
+old_time = time() - 1000 * 24 * 3600  # 1000 days ago
+
 for path in os.environ["PATH"].split(os.pathsep):
     path = path.strip('"')
     test_prog_path = os.path.join(path, 'remove-old-files.py')
@@ -54,9 +58,9 @@ def assert_files_not_exist(files):
 def test_remove_old_files():
     create_files(['test1', 'test2'])
     assert_files_exist(['test1', 'test2'])
-    os.utime('test2', (0, 0))
-    assert os.system(
-        "%s %s --older 100 ." % (sys.executable, test_prog_path)) == 0
+    os.utime('test2', (old_time, old_time))
+    assert subprocess.call(
+        [sys.executable, test_prog_path, "--older", "100", "."]) == 0
     assert_files_exist('test1')
     assert_files_not_exist('test2')
 
@@ -66,9 +70,9 @@ def test_recursive():
     test3 = os.path.join('subdir', 'test3')
     test4 = os.path.join('subdir', 'test4')
     assert_files_exist([test3, test4])
-    os.utime(test4, (0, 0))
-    assert os.system(
-        "%s %s --older 100 ." % (sys.executable, test_prog_path)) == 0
+    os.utime(test4, (old_time, old_time))
+    assert subprocess.call(
+        [sys.executable, test_prog_path, "--older", "100", "."]) == 0
     assert_files_exist(test3)
     assert_files_not_exist(test4)
 
@@ -78,12 +82,12 @@ def test_remove_empty_directory():
     test3 = os.path.join('subdir', 'test3')
     test4 = os.path.join('subdir', 'test4')
     assert_files_exist([test3, test4])
-    os.utime(test3, (0, 0))
-    os.utime(test4, (0, 0))
-    assert os.system(
-        "%s %s --older 100 ." % (sys.executable, test_prog_path)) == 0
+    os.utime(test3, (old_time, old_time))
+    os.utime(test4, (old_time, old_time))
+    assert subprocess.call(
+        [sys.executable, test_prog_path, "--older", "100", "."]) == 0
     assert_files_exist('subdir')
     assert_files_not_exist([test3, test4])
-    assert os.system(
-        "%s %s -e --older 100 ." % (sys.executable, test_prog_path)) == 0
+    assert subprocess.call(
+        [sys.executable, test_prog_path, "-e", "--older", "100", "."]) == 0
     assert_files_not_exist('subdir')