Revize 7d0aa304

Re #8472 - Added few unit tests that perform further testing of _run_for_output method

proj/services/cryptography.py

@@ -57 +57 @@
 
             if proc.returncode != 0:
                 # if the process did not result in zero result code, then raise an exception
-                if err is not None:
+                if err is not None and len(err) > 0:
                     raise CryptographyException(executable, args, err.decode())
                 else:
                     raise CryptographyException(executable, args,

proj/tests/services/cryptography/run_for_output_test.py

@@ +1 @@
+import pytest
+
+from proj.services.cryptography import CryptographyException
+
+
+def test_simple_exec(service):
+    out = service._run_for_output(["version"])
+    assert "OpenSSL" in out.decode()
+
+
+def test_simple_exec_without_parameters(service):
+    out = service._run_for_output()
+    assert "OpenSSL>" in out.decode()
+
+
+def test_nonexistent_executable(service):
+    with pytest.raises(CryptographyException) as e:
+        service._run_for_output(executable="nonexistent_executable#")
+    assert """"nonexistent_executable#" not found in the current PATH.""" in e.value.message
+
+
+def test_exception_str(service):
+    with pytest.raises(CryptographyException) as e:
+        service._run_for_output(executable="nonexistent_executable")
+    assert """EXECUTABLE: nonexistent_executable
+        ARGS: ('nonexistent_executable',)
+        MESSAGE: "nonexistent_executable" not found in the current PATH.""" in e.value.__str__()