From 9dd46c9ea7ab64d26382da96ed65a21da3293ca9 Mon Sep 17 00:00:00 2001
From: Edwin Eefting <edwin@datux.nl>
Date: Tue, 15 Oct 2024 12:26:25 +0200
Subject: [PATCH] fix test

---
 tests/test_zfsautobackup.py   | 23 ++++++++++++-----------
 tests/test_zfsautobackup40.py |  3 +++
 2 files changed, 15 insertions(+), 11 deletions(-)

diff --git a/tests/test_zfsautobackup.py b/tests/test_zfsautobackup.py
index 4bdc1bf..b7a0d8a 100644
--- a/tests/test_zfsautobackup.py
+++ b/tests/test_zfsautobackup.py
@@ -100,7 +100,8 @@ def test_defaults(self):
 
         with self.subTest("bare defaults, allow empty"):
             with mocktime("20101111000001"):
-                self.assertFalse(ZfsAutobackup("test test_target1 --allow-empty --no-progress".split(" ")).run())
+                self.assertFalse(
+                    ZfsAutobackup("test test_target1 --allow-empty --no-progress --no-bookmarks".split(" ")).run())
 
             r = shelltest("zfs list -H -o name -r -t snapshot,filesystem " + TEST_POOLS)
             self.assertMultiLineEqual(r, """
@@ -173,41 +174,41 @@ def test_defaults(self):
                 self.assertFalse(
                     ZfsAutobackup("test test_target1 --allow-empty --verbose --no-progress".split(" ")).run())
 
-            with mocktime("20111211000001"):
+            with mocktime("20111212000000"):
                 self.assertFalse(ZfsAutobackup(
-                    "test test_target1 --allow-empty --verbose --keep-source 1y1y --keep-target 1d1y --no-progress".split(
+                    "test test_target1 --allow-empty --verbose --keep-source 1d1y --keep-target 1d1y --no-progress".split(
                         " ")).run())
 
             r = shelltest("zfs list -H -o name -r -t snapshot,filesystem " + TEST_POOLS)
-            self.assertMultiLineEqual(r, """
+            self.assertMultiLineEqual("""
 test_source1
 test_source1/fs1
 test_source1/fs1@test-20111211000000
-test_source1/fs1@test-20111211000001
+test_source1/fs1@test-20111212000000
 test_source1/fs1/sub
 test_source1/fs1/sub@test-20111211000000
-test_source1/fs1/sub@test-20111211000001
+test_source1/fs1/sub@test-20111212000000
 test_source2
 test_source2/fs2
 test_source2/fs2/sub
 test_source2/fs2/sub@test-20111211000000
-test_source2/fs2/sub@test-20111211000001
+test_source2/fs2/sub@test-20111212000000
 test_source2/fs3
 test_source2/fs3/sub
 test_target1
 test_target1/test_source1
 test_target1/test_source1/fs1
 test_target1/test_source1/fs1@test-20111211000000
-test_target1/test_source1/fs1@test-20111211000001
+test_target1/test_source1/fs1@test-20111212000000
 test_target1/test_source1/fs1/sub
 test_target1/test_source1/fs1/sub@test-20111211000000
-test_target1/test_source1/fs1/sub@test-20111211000001
+test_target1/test_source1/fs1/sub@test-20111212000000
 test_target1/test_source2
 test_target1/test_source2/fs2
 test_target1/test_source2/fs2/sub
 test_target1/test_source2/fs2/sub@test-20111211000000
-test_target1/test_source2/fs2/sub@test-20111211000001
-""")
+test_target1/test_source2/fs2/sub@test-20111212000000
+""", r)
 
     def test_ignore_othersnaphots(self):
 
diff --git a/tests/test_zfsautobackup40.py b/tests/test_zfsautobackup40.py
index 1cd1349..468382e 100644
--- a/tests/test_zfsautobackup40.py
+++ b/tests/test_zfsautobackup40.py
@@ -417,3 +417,6 @@ def test_migrate_from_mismatching_snapshotname(self):
 test_target1/test_source2/fs2/sub
 test_target1/test_source2/fs2/sub@test-20101111000000
 """)
+
+# TODO
+# - check if holds and no-holds function ok when bookmarks are enabled