Comment out dead test code

It would be handy to resurrect this, but as the comment says, they
don't work, and I haven't time to fix that now. However, the fixed
test causes a pylint warning.

1 files changed, 25 insertions, 25 deletions

diff --git a/obnamlib/repo_interface.py b/obnamlib/repo_interface.py
index 4df397a9..ae38321b 100644
--- a/obnamlib/repo_interface.py
+++ b/obnamlib/repo_interface.py
@@ -1849,37 +1849,37 @@ class RepositoryInterfaceTests(unittest.TestCase):  # pragma: no cover
     # FIXME: These tests fails due to ClientMetadataTree brokenness, it seems.
     # They're disabled, for now. The bug is not exposed by existing code,
     # only by the new interface's tests.
-    if False:
-        def test_removing_file_removes_all_its_file_keys(self):
-            gen_id = self.create_generation()
-            self.repo.add_file(gen_id, '/foo/bar')
-            self.repo.set_file_key(
-                gen_id, '/foo/bar', obnamlib.REPO_FILE_MTIME_SEC, 123)
 
-            # Remove the file. Key should be removed.
-            self.repo.remove_file(gen_id, '/foo/bar')
-            self.assertRaises(
-                obnamlib.RepositoryFileDoesNotExistInGeneration,
-                self.repo.get_file_key,
-                gen_id, '/foo/bar', obnamlib.REPO_FILE_MTIME_SEC)
+    # def test_removing_file_removes_all_its_file_keys(self):
+    #     gen_id = self.create_generation()
+    #     self.repo.add_file(gen_id, '/foo/bar')
+    #     self.repo.set_file_key(
+    #         gen_id, '/foo/bar', obnamlib.REPO_FILE_MTIME_SEC, 123)
 
-            # Add the file back. Key should still be removed.
-            self.repo.add_file(gen_id, '/foo/bar')
-            value = self.repo.get_file_key(
-                gen_id, '/foo/bar', obnamlib.REPO_FILE_MTIME_SEC)
-            self.assertEqual(value, 0)
+    #     # Remove the file. Key should be removed.
+    #     self.repo.remove_file(gen_id, '/foo/bar')
+    #     self.assertRaises(
+    #         obnamlib.RepositoryFileDoesNotExistInGeneration,
+    #         self.repo.get_file_key,
+    #         gen_id, '/foo/bar', obnamlib.REPO_FILE_MTIME_SEC)
 
-        def test_can_add_a_file_then_remove_then_add_it_again(self):
-            gen_id = self.create_generation()
+    #     # Add the file back. Key should still be removed.
+    #     self.repo.add_file(gen_id, '/foo/bar')
+    #     value = self.repo.get_file_key(
+    #         gen_id, '/foo/bar', obnamlib.REPO_FILE_MTIME_SEC)
+    #     self.assertEqual(value, 0)
+
+    # def test_can_add_a_file_then_remove_then_add_it_again(self):
+    #     gen_id = self.create_generation()
 
-            self.repo.add_file(gen_id, '/foo/bar')
-            self.assertTrue(self.repo.file_exists(gen_id, '/foo/bar'))
+    #     self.repo.add_file(gen_id, '/foo/bar')
+    #     self.assertTrue(self.repo.file_exists(gen_id, '/foo/bar'))
 
-            self.repo.remove_file(gen_id, '/foo/bar')
-            self.assertFalse(self.repo.file_exists(gen_id, '/foo/bar'))
+    #     self.repo.remove_file(gen_id, '/foo/bar')
+    #     self.assertFalse(self.repo.file_exists(gen_id, '/foo/bar'))
 
-            self.repo.add_file(gen_id, '/foo/bar')
-            self.assertTrue(self.repo.file_exists(gen_id, '/foo/bar'))
+    #     self.repo.add_file(gen_id, '/foo/bar')
+    #     self.assertTrue(self.repo.file_exists(gen_id, '/foo/bar'))
 
     def test_unlocking_client_forgets_set_file_keys(self):
         gen_id = self.create_generation()