From 15ed372df617caa8342493763b461b144eeb66b8 Mon Sep 17 00:00:00 2001 From: Peter Eckersley Date: Tue, 20 Dec 2016 14:32:39 -0800 Subject: [PATCH] Fix existing tests --- certbot/tests/cert_manager_test.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/certbot/tests/cert_manager_test.py b/certbot/tests/cert_manager_test.py index 7f4261fd0..4fd66c34a 100644 --- a/certbot/tests/cert_manager_test.py +++ b/certbot/tests/cert_manager_test.py @@ -179,7 +179,9 @@ class CertificatesTest(BaseCertManagerTest): self.assertTrue(mock_utility.called) shutil.rmtree(tempdir) - def test_report_human_readable(self): + @mock.patch('certbot.cert_manager.ocsp.RevocationChecker.ocsp_status') + def test_report_human_readable(self, mock_ocsp): + mock_ocsp.side_effect = lambda _cert, _chain, status: status from certbot import cert_manager import datetime, pytz expiry = pytz.UTC.fromutc(datetime.datetime.utcnow()) @@ -190,7 +192,7 @@ class CertificatesTest(BaseCertManagerTest): cert.is_test_cert = False parsed_certs = [cert] - mock_config = mock.MagicMock() + mock_config = mock.MagicMock(certname=None, lineagename=None) # pylint: disable=protected-access out = cert_manager._report_human_readable(mock_config, parsed_certs) self.assertTrue("INVALID: EXPIRED" in out) @@ -216,7 +218,7 @@ class CertificatesTest(BaseCertManagerTest): cert.is_test_cert = True out = cert_manager._report_human_readable(mock_config, parsed_certs) - self.assertTrue('INVALID: TEST CERT' in out) + self.assertTrue('INVALID: TEST_CERT' in out) class SearchLineagesTest(BaseCertManagerTest):