Improve CI

Author: olliesilvester

.github/workflows/_test.yml

@@ -50,8 +50,8 @@ jobs:
           python-version: ${{ inputs.python-version }}
           pip-install: ".[dev,server]"
 
-      - name: Run tests
-        run: tox -e tests
+      - name: Run unit tests
+        run: tox -e unit_tests
 
       - name: Upload coverage to Codecov
         uses: codecov/codecov-action@v4

pyproject.toml

@@ -51,13 +51,14 @@ write_to = "src/daq_config_server/_version.py"
 reportMissingImports = false # Ignore missing stubs in imported modules
 
 [tool.pytest.ini_options]
+asyncio_mode = "auto"
+asyncio_default_fixture_loop_scope = "function"
 # Run pytest with all our checkers, and don't spam us with massive tracebacks on error
 addopts = """
     --tb=native -vv
-    --asyncio-mode=auto
     """
 markers = """
-    uses_live_server: mark a test which uses the live config server
+    requires_local_server: mark a test which requires locally hosting the config server
 """
 
 # https://iscinumpy.gitlab.io/post/bound-version-constraints/#watch-for-warnings
@@ -79,18 +80,22 @@ legacy_tox_ini = """
 [tox]
 skipsdist=True
 
-[testenv:{pre-commit,type-checking,tests}]
+[testenv]
 # Don't create a virtualenv for the command, requires tox-direct plugin
 direct = True
 passenv = *
 allowlist_externals =
     pytest
     pre-commit
     pyright
+    sphinx-build
+    sphinx-autobuild
+[testenv:{pre-commit,type-checking,unit_tests,system_tests}]
 commands =
     pre-commit: pre-commit run --all-files --show-diff-on-failure {posargs}
     type-checking: pyright src tests {posargs}
-    tests: pytest -m "not uses_live_server" --cov=daq_config_server --cov-report term --cov-report xml:cov.xml {posargs}
+    unit_tests: pytest --cov=daq_config_server --cov-report term --cov-report xml:cov.xml {posargs} tests/unit_tests
+    system_tests: pytest tests/system_tests
 """
 
 [tool.ruff]

src/daq_config_server/__main__.py

@@ -17,6 +17,7 @@
 def main():
     parser = ArgumentParser()
     parser.add_argument("-v", "--version", action="version", version=__version__)
+    parser.parse_args()
 
     if not server_dependencies_exist:
         print(

tests/__init__.py

@@ -1,7 +1,7 @@
 import pytest
-from tests.constants import TEST_DATA_DIR
 
 from daq_config_server.client import ConfigServer
+from tests.constants import TEST_DATA_DIR
 
 SERVER_ADDRESS = "http://0.0.0.0:8555"
 
@@ -20,11 +20,11 @@ def server():
     return ConfigServer(SERVER_ADDRESS)
 
 
+@pytest.mark.requires_local_server
 def test_read_unformatted_file(server: ConfigServer):
     file_path = f"{TEST_DATA_DIR}/beamline_parameters.txt"
     with open(file_path) as f:
         expected_response = f.read()
-    print("hing")
     assert (
         server.read_unformatted_file(f"{TEST_DATA_DIR}/beamline_parameters.txt")
         == expected_response

tests/system_tests/test_client.py

@@ -4,10 +4,10 @@
 import pytest
 from fastapi import status
 from fastapi.testclient import TestClient
-from tests.constants import TEST_DATA_DIR
 
 from daq_config_server import app
 from daq_config_server.constants import ENDPOINTS
+from tests.constants import TEST_DATA_DIR
 
 
 @pytest.fixture