From 707511006cfb82d829bca465757c2517dbb10aad Mon Sep 17 00:00:00 2001 From: Erica Portnoy Date: Fri, 14 Feb 2020 13:58:34 -0800 Subject: [PATCH] Remove subprocess import and reference in test --- certbot-apache/certbot_apache/_internal/configurator.py | 1 - certbot-apache/tests/configurator_test.py | 3 +-- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/certbot-apache/certbot_apache/_internal/configurator.py b/certbot-apache/certbot_apache/_internal/configurator.py index c8c0f59a2..f7ad9983f 100644 --- a/certbot-apache/certbot_apache/_internal/configurator.py +++ b/certbot-apache/certbot_apache/_internal/configurator.py @@ -8,7 +8,6 @@ import fnmatch import logging import re import socket -import subprocess import time import six diff --git a/certbot-apache/tests/configurator_test.py b/certbot-apache/tests/configurator_test.py index 71bd221cb..3cda7a771 100644 --- a/certbot-apache/tests/configurator_test.py +++ b/certbot-apache/tests/configurator_test.py @@ -1790,8 +1790,7 @@ class InstallSslOptionsConfTest(util.ApacheTest): "Constants.ALL_SSL_OPTIONS_HASHES must be appended with the sha256 " "hash of {0} when it is updated.".format(one_file)) - @mock.patch("certbot_apache._internal.configurator.subprocess.Popen") - def test_openssl_version(self, mock_popen): + def test_openssl_version(self): some_string_contents = """ SSLOpenSSLConfCmd OpenSSL configuration command