Use alternative signalling mechanism to fix tests

For unknown reasons on some CI runners, using Process.terminate()
was no longer working. This resulted in test processes not being cleaned
up and interfering with later tests which re-use the same PV names.
The new mechanism more cleanly terminates the subprocesses by letting
them exit of their own volition, rather than being forcibly killed.

Also adding some extra configuration to fix warnings from pytest-asyncio
plugin, which will at some point become errors.

Author: AlexanderWells-diamond

setup.cfg

@@ -60,6 +60,9 @@ extend-ignore =
 addopts =
     --tb=native -vv --doctest-modules --ignore=iocStats --ignore=epicscorelibs --ignore=docs
     --cov=softioc --cov-report term --cov-report xml:cov.xml
+# Enables all discovered async tests and fixtures to be automatically marked as async, even if
+# they don't have a specific marker https://github.com/pytest-dev/pytest-asyncio#auto-mode
+asyncio_mode = auto
 
 [coverage:run]
 # This is covered in the versiongit test suite so exclude it here

tests/test_record_values.py

@@ -2,7 +2,6 @@
 from typing import List
 import numpy
 import pytest
-import asyncio
 
 from enum import Enum
 from math import isnan, inf, nan
@@ -350,13 +349,13 @@ def run_ioc(record_configurations: list, conn, set_enum, get_enum):
                     if val is not None:
                         conn.send(record.get())
 
-    conn.close()
-
     # Keep process alive while main thread works.
     # This is most applicable to CAGET tests.
-    asyncio.run_coroutine_threadsafe(
-        asyncio.sleep(TIMEOUT), dispatcher.loop
-    ).result()
+    while (True):
+        if conn.poll(TIMEOUT):
+            val = conn.recv()
+            if val == "EXIT":
+                break
 
 
 def run_test_function(
@@ -475,7 +474,10 @@ def run_test_function(
         # Purge cache to suppress spurious "IOC disconnected" exceptions
         _channel_cache.purge()
 
-        ioc_process.terminate()
+
+        parent_conn.send("EXIT")
+
+        # ioc_process.terminate()
         ioc_process.join(timeout=TIMEOUT)
         if ioc_process.exitcode is None:
             pytest.fail("Process did not terminate")

tests/test_records.py

@@ -167,11 +167,8 @@ def validate_ioc_test_func(self, record_func, queue, validate_pass: bool):
 
         queue.put("IOC ready")
 
-        # Keep process alive while main thread works.
-        # This is most applicable to CAGET tests.
-        asyncio.run_coroutine_threadsafe(
-            asyncio.sleep(TIMEOUT), dispatcher.loop
-        ).result()
+        # Keep process alive while main thread runs CAGET
+        queue.get(timeout=5)
 
     def validate_test_runner(
             self,
@@ -224,7 +221,7 @@ def validate_test_runner(
                 assert ret_val == expected_value
 
         finally:
-            process.terminate()
+            queue.put("EXIT")
             process.join(timeout=3)