ci(reviewer-bot): finalize migrated test configuration

Author: PLeVasseur

.github/workflows/reviewer-bot-tests.yml

@@ -20,7 +20,7 @@ on:
   workflow_dispatch:
 
 jobs:
-  reviewer-bot-tests:
+  reviewer-bot-unit:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repository
@@ -32,11 +32,11 @@ jobs:
       - name: Install dependencies
         run: uv sync --dev
 
-      - name: Run migrated reviewer bot tests with targeted coverage
+      - name: Run reviewer bot unit and integration tests with targeted coverage
         run: >-
           mkdir -p artifacts/coverage &&
           COVERAGE_FILE=artifacts/coverage/.coverage
-          uv run pytest tests --import-mode=importlib
+          uv run pytest tests/unit/reviewer_bot tests/integration/reviewer_bot
           --cov=scripts.reviewer_bot
           --cov=scripts.reviewer_bot_lib
           --cov-branch
@@ -66,3 +66,18 @@ jobs:
                   f"reviewer-bot branch coverage below floor: {branches:.2f}% < 48%"
               )
           PY
+
+  reviewer-bot-contract:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@v6
+
+      - name: Install dependencies
+        run: uv sync --dev
+
+      - name: Run reviewer bot contract tests
+        run: uv run pytest tests/contract/reviewer_bot

pyproject.toml

@@ -57,6 +57,30 @@ lint.ignore = [
 [tool.ruff.lint.flake8-comprehensions]
 allow-dict-calls-with-keyword-arguments = true
 
+[tool.pytest.ini_options]
+testpaths = ["tests"]
+addopts = [
+    "--import-mode=importlib",
+    "-ra",
+    "--strict-config",
+    "--strict-markers",
+]
+markers = [
+    "integration: filesystem, subprocess, multi-module, or boundary tests",
+    "contract: workflow, CLI, adapter, or artifact contract tests",
+]
+filterwarnings = ["error"]
+xfail_strict = true
+
+[tool.coverage.run]
+branch = true
+relative_files = true
+source = ["scripts.reviewer_bot", "scripts.reviewer_bot_lib"]
+
+[tool.coverage.report]
+show_missing = true
+skip_empty = true
+
 [dependency-groups]
 dev = [
     "pytest>=8.3.0",