From 9db013aee284387bd2126041c7b7614ab5432711 Mon Sep 17 00:00:00 2001 From: Jussi Kukkonen Date: Sun, 5 Jul 2020 21:03:04 +0300 Subject: [PATCH] test_repository_tool: Add missing clear_all flag Currently TestTimestamp creates custom databases but only clears the default ones. This means next create_*db() call will fail meaning every test after this one will fail (currently TestTimestamp happens to be last but the effect can be seen by renaming it to TestATimestamp). Also remove the clear_*db() calls from TestRepository::Setup(): they are likely to be a workaround for a similar problem earlier (earlier test failed to cleanup). Signed-off-by: Jussi Kukkonen --- tests/test_repository_tool.py | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/tests/test_repository_tool.py b/tests/test_repository_tool.py index f314ff4a02..7b94027803 100755 --- a/tests/test_repository_tool.py +++ b/tests/test_repository_tool.py @@ -81,9 +81,6 @@ def tearDownClass(cls): def setUp(self): - tuf.roledb.clear_roledb(clear_all=True) - tuf.keydb.clear_keydb(clear_all=True) - tuf.roledb.create_roledb('test_repository') tuf.keydb.create_keydb('test_repository') @@ -1110,8 +1107,8 @@ def setUp(self): def tearDown(self): - tuf.roledb.clear_roledb() - tuf.keydb.clear_keydb() + tuf.roledb.clear_roledb(clear_all=True) + tuf.keydb.clear_keydb(clear_all=True)