Merge pull request #1059 from mrwacky42/add-tests

Add a test, skip a test
certbot · Oct 21, 2015 · 3ea810d · 3ea810d
2 parents 368b69f + 39a44b9
commit 3ea810d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/letsencrypt/plugins/manual.py b/letsencrypt/plugins/manual.py
@@ -130,10 +130,7 @@ def _perform_single(self, achall):
         if self.conf("test-mode"):
             logger.debug("Test mode. Executing the manual command: %s", command)
             # sh shipped with OS X does't support echo -n
-            if sys.platform == "darwin":
-                executable = "/bin/bash"
-            else:
-                executable = None
+            executable = "/bin/bash" if sys.platform == "darwin" else None
             try:
                 self._httpd = subprocess.Popen(
                     command,

diff --git a/letsencrypt/plugins/util_test.py b/letsencrypt/plugins/util_test.py
@@ -98,6 +98,10 @@ def test_listening_ipv6(self, mock_get_utility, mock_process, mock_net):
         self.assertEqual(mock_get_utility.call_count, 1)
         mock_process.assert_called_once_with(4420)
 
+    @mock.patch("letsencrypt.plugins.util.psutil.net_connections")
+    def test_access_denied_exception(self, mock_net):
+        mock_net.side_effect = psutil.AccessDenied("")
+        self.assertFalse(self._call(12345))
 
 if __name__ == "__main__":
     unittest.main()  # pragma: no cover