Changeset 65e66c9 in mod_gnutls for test/runtest.py

Timestamp:

Nov 28, 2020, 3:46:31 PM (3 months ago)

Author:

Fiona Klute <fiona.klute@…>

Branches:

asyncio

Parents:

7eb4233

Message:

Turn "service ready" check functions into coroutines

This way checks that call subprocesses don't block other tasks.

File:

• test/runtest.py (modified) (2 diffs)

test/runtest.py

@@ -65 +65 @@
 
 
-def check_ocsp_responder():
+async def check_ocsp_responder():
     # Check if OCSP responder works
     issuer_cert = 'authority/x509.pem'
     check_cert = 'authority/server/x509.pem'
-    command = ['ocsptool', '--ask', '--nonce',
-               '--load-issuer', issuer_cert,
-               '--load-cert', check_cert]
-    return subprocess.run(command).returncode == 0
-
-
-def check_msva():
+    proc = await asyncio.create_subprocess_exec(
+        'ocsptool', '--ask', '--nonce',
+        '--load-issuer', issuer_cert, '--load-cert', check_cert)
+    return (await proc.wait()) == 0
+
+
+async def check_msva():
     # Check if MSVA is up
     cert_file = 'authority/client/x509.pem'
@@ -81 +81 @@
     with open(uid_file, 'r') as file:
         uid = file.read().strip()
-        command = ['msva-query-agent', 'https', uid, 'x509pem', 'client']
-        with open(cert_file, 'r') as cert:
-            return subprocess.run(command, stdin=cert).returncode == 0
+    with open(cert_file, 'r') as cert:
+        proc = await asyncio.create_subprocess_exec(
+            'msva-query-agent', 'https', uid, 'x509pem', 'client',
+            stdin=cert)
+        return (await proc.wait()) == 0