perf-test: parallelise step #6 + register pytest integration marker

1. .github/workflows/performance-test.yml
   - Run Performance Test (step #6): `python -m unittest …`
     (single-threaded) → `pytest -v -n auto …`. pytest discovers
     unittest.TestCase subclasses automatically and pytest-xdist
     parallelises them across the runner's vCPUs. No test refactoring
     needed. Expected ~3 min → ~1.5 min.
   - Run Connectivity Tests (step #8): xdist `-n auto` (~4 workers on
     ubuntu-latest) → `-n 8`. These tests are I/O-bound on Neo4j
     round-trips; oversubscribing CPU cores fills network-wait time.
     Upstream advertises max_concurrent=40 on /status — 8 workers
     stays well within budget. Expected ~7 min → ~4 min.

2. pyproject.toml
   - Add [tool.pytest.ini_options] markers = [...] declaring the
     `integration` marker that the connectivity test files already
     decorate with. Stops the `PytestUnknownMarkWarning` per-occurrence
     log spam that was bloating performance.md (each unregistered mark
     was emitting four-plus warning blocks per xdist worker per file).

Author: Robbie1977

.github/workflows/performance-test.yml

@@ -47,27 +47,27 @@ jobs:
           curl -v --max-time 10 "http://owl.virtualflybrain.org/kbs/vfb" || echo "Server unreachable"
           
       - name: Run Performance Test
-        # Auto-retry once on failure: cold-start runs hit Neo4j + SOLR with
-        # empty result caches and frequently breach THRESHOLD_VERY_SLOW
-        # purely on first-call latency. The retry runs with the cache the
-        # first attempt warmed; flakes-from-cold-start no longer mask real
-        # regressions. If the second attempt also fails, surface the failure.
-        # IMPORTANT: the retry OVERWRITES performance_test_output.log so that
-        # the downstream "Fail job on test failures" step grades on the
-        # second attempt's output only — otherwise attempt 1's `FAIL:` lines
-        # leak through and turn a green retry red.
+        # Switched from `python -m unittest` (single-threaded) to `pytest -n auto`
+        # so the suite parallelises across the runner's vCPUs. pytest discovers
+        # unittest.TestCase subclasses automatically; no test refactoring needed.
+        # Auto-retry once on failure: cold-start runs hit Neo4j + SOLR with empty
+        # result caches and can breach thresholds on first-call latency. The
+        # retry runs warm. If both fail, surface the failure.
+        # IMPORTANT: the retry OVERWRITES performance_test_output.log so the
+        # downstream "Fail job on test failures" step grades on the second
+        # attempt's output only.
         run: |
           set +e
-          echo "=== Performance test attempt 1/2 ==="
-          python -m unittest src.test.test_query_performance -v 2>&1 | tee performance_test_output.log
+          echo "=== Performance test attempt 1/2 (parallel) ==="
+          pytest -v -n auto src/test/test_query_performance.py 2>&1 | tee performance_test_output.log
           FIRST_EXIT=${PIPESTATUS[0]}
           if [[ $FIRST_EXIT -eq 0 ]]; then
               exit 0
           fi
           echo ""
           echo "=== Attempt 1 failed (exit $FIRST_EXIT). Retrying once with warm cache. ==="
           echo "=== RETRY ATTEMPT (auto-retry on first failure) ===" > performance_test_output.log
-          python -m unittest src.test.test_query_performance -v 2>&1 | tee -a performance_test_output.log
+          pytest -v -n auto src/test/test_query_performance.py 2>&1 | tee -a performance_test_output.log
           exit ${PIPESTATUS[0]}
 
       - name: Run Legacy Performance Test
@@ -123,7 +123,7 @@ jobs:
           # grades on the last attempt only.
           set +e
           echo "=== Connectivity test attempt 1/2 (parallel) ==="
-          pytest -v -n auto \
+          pytest -v -n 8 \
             src/test/test_neuron_neuron_connectivity.py \
             src/test/test_neuron_region_connectivity.py \
             src/test/test_upstream_class_connectivity.py \
@@ -137,7 +137,7 @@ jobs:
           fi
           echo ""
           echo "=== Connectivity attempt 1 failed (exit $FIRST_EXIT). Retrying once with warm cache. ==="
-          pytest -v -n auto \
+          pytest -v -n 8 \
             src/test/test_neuron_neuron_connectivity.py \
             src/test/test_neuron_region_connectivity.py \
             src/test/test_upstream_class_connectivity.py \

pyproject.toml

@@ -3,4 +3,13 @@ requires = [
     "setuptools>=45,<69",
     "wheel"
 ]
-build-backend = "setuptools.build_meta"
+build-backend = "setuptools.build_meta"
+
+[tool.pytest.ini_options]
+# Register custom marks so the connectivity test files' `@pytest.mark.integration`
+# decorations stop logging `PytestUnknownMarkWarning` per occurrence per
+# xdist worker. Each unregistered mark was emitting four-plus warning
+# blocks in performance.md — pure noise.
+markers = [
+    "integration: integration tests that exercise the live VFB upstream (Neo4j, SOLR, owlery). Skip via -m 'not integration'.",
+]