upstream/certbot
1
0
Fork 0
mirror of https://github.com/certbot/certbot.git synced 2026-06-04 22:33:00 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

Don't double write to log file; fix main test

Browse source
This commit is contained in:
Erica Portnoy 2021-05-13 15:44:24 -07:00
parent 803a4b0988
commit c75beb0116
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
certbot/certbot/_internal/renewal.py
View file

@ -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

5
certbot/tests/main_test.py
View file

@ -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:")