From f7a12c6359888f00cc4c8742ac645f9331f47f98 Mon Sep 17 00:00:00 2001 From: Xavier-Do Date: Mon, 6 Feb 2023 10:16:07 +0100 Subject: [PATCH] [FIX] runbot: fix fetch tests --- runbot/tests/test_repo.py | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/runbot/tests/test_repo.py b/runbot/tests/test_repo.py index 5e848e05..ca3bc1c1 100644 --- a/runbot/tests/test_repo.py +++ b/runbot/tests/test_repo.py @@ -381,15 +381,21 @@ class TestFetch(RunbotCase): # Ensure that Host is not disabled if fetch succeeds after 3 tries with mute_logger("odoo.addons.runbot.models.repo"): self.repo_server._update_fetch_cmd() + self.assertFalse(host.assigned_only, "Host should not be disabled when fetch succeeds") self.assertEqual(self.fetch_count, 3) - # Now ensure that host is disabled after 5 unsuccesful tries self.force_failure = True - self.fetch_count = 0 + with mute_logger("odoo.addons.runbot.models.repo"): self.repo_server._update_fetch_cmd() - self.assertTrue(host.assigned_only) + self.assertFalse(host.assigned_only, "Host should not be disabled when fetch fails by default") + + self.fetch_count = 0 + self.env['ir.config_parameter'].sudo().set_param('runbot.runbot_disable_host_on_fetch_failure', True) + with mute_logger("odoo.addons.runbot.models.repo"): + self.repo_server._update_fetch_cmd() + self.assertTrue(host.assigned_only, "Host should be disabled when fetch fails and runbot_disable_host_on_fetch_failure is set") self.assertEqual(self.fetch_count, 5)