From c75beb0116a151f107f6d395f98f84ddcf709142 Mon Sep 17 00:00:00 2001 From: Erica Portnoy Date: Thu, 13 May 2021 15:44:24 -0700 Subject: [PATCH] Don't double write to log file; fix main test --- certbot/certbot/_internal/renewal.py | 1 - certbot/tests/main_test.py | 5 +++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/certbot/certbot/_internal/renewal.py b/certbot/certbot/_internal/renewal.py index cfdc5ada1..6b4ab1163 100644 --- a/certbot/certbot/_internal/renewal.py +++ b/certbot/certbot/_internal/renewal.py @@ -301,7 +301,6 @@ def should_renew(config, lineage): logger.info("Certificate not due for renewal, but simulating renewal for dry run") return True display_util.notify("Certificate not yet due for renewal") - logger.debug("Certificate not yet due for renewal") return False diff --git a/certbot/tests/main_test.py b/certbot/tests/main_test.py index 5689c0281..6cf38e2ce 100644 --- a/certbot/tests/main_test.py +++ b/certbot/tests/main_test.py @@ -1146,8 +1146,9 @@ class MainTest(test_util.ConfigTestCase): log_out="Auto-renewal forced") self.assertEqual(get_utility().add_message.call_count, 1) - self._test_renewal_common(False, ['-tvv', '--debug', '--keep'], - log_out="not yet due", should_renew=False) + _, get_utility, _ = self._test_renewal_common(False, ['-tvv', '--debug', '--keep'], + should_renew=False) + self.assertIn('not yet due', get_utility().notification.call_args[0][0]) def _dump_log(self): print("Logs:")