From 19143d83036cdcbfc98c16f202a207c888ce18e3 Mon Sep 17 00:00:00 2001 From: Peter Eckersley Date: Thu, 22 Dec 2016 13:07:00 -0800 Subject: [PATCH] Increase test coverage --- certbot/tests/cert_manager_test.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/certbot/tests/cert_manager_test.py b/certbot/tests/cert_manager_test.py index 6caaab878..07f7cedaa 100644 --- a/certbot/tests/cert_manager_test.py +++ b/certbot/tests/cert_manager_test.py @@ -181,8 +181,8 @@ class CertificatesTest(BaseCertManagerTest): shutil.rmtree(tempdir) @mock.patch('certbot.cert_manager.ocsp.RevocationChecker.ocsp_revoked') - def test_report_human_readable(self, mock_ocsp): - mock_ocsp.return_value = None + def test_report_human_readable(self, mock_revoked): + mock_revoked.return_value = None from certbot import cert_manager import datetime, pytz expiry = pytz.UTC.fromutc(datetime.datetime.utcnow()) @@ -221,8 +221,9 @@ class CertificatesTest(BaseCertManagerTest): self.assertTrue('VALID' in out and not 'INVALID' in out) cert.is_test_cert = True + mock_revoked.return_value = True out = get_report() - self.assertTrue('INVALID: TEST_CERT' in out) + self.assertTrue('INVALID: TEST_CERT, REVOKED' in out) cert = mock.MagicMock(lineagename="indescribable") cert.target_expiry = expiry