From abe0bc47fbed1a348989fb74c2b4eb13170c7c30 Mon Sep 17 00:00:00 2001 From: Tomas Kopecek Date: Dec 08 2023 10:16:36 +0000 Subject: Fix test Test was not mocking properly log calls, creating log file at local filesystem. --- diff --git a/tests/test_hub/test_cg_importer.py b/tests/test_hub/test_cg_importer.py index e449000..d2328e6 100644 --- a/tests/test_hub/test_cg_importer.py +++ b/tests/test_hub/test_cg_importer.py @@ -127,23 +127,28 @@ class TestCGImporter(unittest.TestCase): self.path_build.return_value = path x = kojihub.CG_Importer() + x.log_warning = mock.MagicMock() # directory exists self.lexists.return_value = True with self.assertRaises(koji.GenericError): x.check_build_dir(delete=False) self.rmtree.assert_not_called() + x.log_warning.assert_not_called() # directory exists + delete self.lexists.return_value = True x.check_build_dir(delete=True) self.rmtree.assert_called_once_with(path) + x.log_warning.assert_called_once_with("Deleting build directory: /random_path/random_dir") + x.log_warning.reset_mock() # directory doesn't exist self.rmtree.reset_mock() self.lexists.return_value = False x.check_build_dir() self.rmtree.assert_not_called() + x.log_warning.assert_not_called() def test_prep_build_exists(self): self.path_work.return_value = os.path.dirname(__file__)