test(audit): éliminer tous les pytest.raises(Exception) résiduels

Ratchet ``BROAD_RAISES_BASELINE`` Phase 8 : **24 → 0**.

Les 24 sites résiduels acceptés par la baseline temporaire ont
tous été remplacés par leur classe d'exception précise :

**13 sites Pydantic ``ValidationError``** (frozen ConfigDict +
validateurs métier) :

- ``tests/pipeline/test_sprint_a14_s6_spec.py:116`` — PipelineSpec.frozen
- ``tests/domain/test_sprint_a14_s4_artifacts.py:118,146,150``
- ``tests/domain/test_sprint_a14_s4_corpus.py:48,55``
- ``tests/domain/test_sprint_a14_s4_documents.py:85``
- ``tests/domain/test_sprint_a14_s4_provenance_errors.py:43``
- ``tests/domain/test_sprint_a14_s5_evaluation_specs.py:53,66,115,207,259``
- ``tests/app/schemas/test_sprint_a14_s39_run_spec_extended.py`` :
7 sites ``pytest.raises(Exception, match=...)`` — les
``ValueError`` levés dans les validateurs Pydantic sont
re-emballés en ``ValidationError``.

**2 sites ``FrozenInstanceError``** (dataclass frozen) :

- ``tests/adapters/storage/test_sprint_a14_s29_artifact_store.py:108,551``
— ArtifactKey et ArtifactRecord (dataclass frozen=True).

**2 sites ``PicaronesError``** dans pipeline :

- ``tests/pipeline/test_sprint_a14_s28_planner.py:560`` —
``PipelineExecutor.run_plan`` rejette un argument qui n'est pas
un ``ExecutionPlan``.
- ``tests/pipeline/test_sprint_a14_s28_planner.py:626`` —
``PipelineExecutor.__init__`` rejette un ``planner`` qui n'est
pas un ``PipelinePlanner``.

**Test ratchet abaissé à 0**

``BROAD_RAISES_BASELINE = 24 → 0``. Tout nouveau
``pytest.raises(Exception)`` introduit dans une PR future fait
échouer ``test_broad_pytest_raises_below_baseline`` —
**verrouillage strict** désormais en place.

**Bilan**

Suite : 4 789 passed, 16 skipped, 8 deselected, 2 xfailed. Ruff
propre. Chaque test attrape désormais exactement la classe
d'exception attendue — les régressions où une exception
différente serait levée font échouer le test.

tests/adapters/storage/test_sprint_a14_s29_artifact_store.py

@@ -35,6 +35,7 @@ from __future__ import annotations
 
 import json
 import threading
+from dataclasses import FrozenInstanceError
 from pathlib import Path
 
 import pytest
@@ -105,7 +106,7 @@ class TestArtifactKeyDataclass:
 
     def test_frozen(self) -> None:
         k = _basic_key()
-        with pytest.raises(Exception):  # FrozenInstanceError
+        with pytest.raises(FrozenInstanceError):
             k.adapter_name = "different"  # type: ignore[misc]
 
 
@@ -548,5 +549,5 @@ class TestStoredArtifactDataclass:
         sa = StoredArtifact(
             key="k", artifact=_make_artifact(), payload=b"x",
         )
-        with pytest.raises(Exception):  # FrozenInstanceError
+        with pytest.raises(FrozenInstanceError):
             sa.payload = b"y"  # type: ignore[misc]

tests/app/schemas/test_sprint_a14_s39_run_spec_extended.py

@@ -14,6 +14,7 @@ purement additive.
 from __future__ import annotations
 
 import pytest
+from pydantic import ValidationError
 
 from picarones.app.schemas.run_spec import (
     PipelineSpecYaml,
@@ -94,7 +95,7 @@ class TestPreferredTextOutput:
         assert pipe.preferred_text_output == "corrector.corrected_text"
 
     def test_rejects_missing_dot(self) -> None:
-        with pytest.raises(Exception, match="format"):
+        with pytest.raises(ValidationError, match="format"):
             PipelineSpecYaml(
                 name="bad",
                 initial_inputs=(ArtifactType.IMAGE,),
@@ -108,7 +109,7 @@ class TestPreferredTextOutput:
             )
 
     def test_rejects_unknown_step(self) -> None:
-        with pytest.raises(Exception, match="introuvable"):
+        with pytest.raises(ValidationError, match="introuvable"):
             PipelineSpecYaml(
                 name="bad",
                 initial_inputs=(ArtifactType.IMAGE,),
@@ -122,7 +123,7 @@ class TestPreferredTextOutput:
             )
 
     def test_rejects_step_not_producing_type(self) -> None:
-        with pytest.raises(Exception, match="ne produit pas"):
+        with pytest.raises(ValidationError, match="ne produit pas"):
             PipelineSpecYaml(
                 name="bad",
                 initial_inputs=(ArtifactType.IMAGE,),
@@ -137,7 +138,7 @@ class TestPreferredTextOutput:
             )
 
     def test_rejects_unknown_artifact_type(self) -> None:
-        with pytest.raises(Exception, match="output_type"):
+        with pytest.raises(ValidationError, match="output_type"):
             PipelineSpecYaml(
                 name="bad",
                 initial_inputs=(ArtifactType.IMAGE,),
@@ -173,7 +174,7 @@ class TestInputsFromValidation:
 
     def test_initial_step_id_rejects_unknown_initial_input(self) -> None:
         # `__initial__` mais le type n'est pas dans initial_inputs → erreur.
-        with pytest.raises(Exception, match="initial_inputs"):
+        with pytest.raises(ValidationError, match="initial_inputs"):
             PipelineSpecYaml(
                 name="bad",
                 initial_inputs=(ArtifactType.IMAGE,),
@@ -218,7 +219,7 @@ class TestInputsFromValidation:
 
     def test_rejects_forward_reference(self) -> None:
         # Un step ne peut pas référencer un step en aval de lui.
-        with pytest.raises(Exception, match="antérieure"):
+        with pytest.raises(ValidationError, match="antérieure"):
             PipelineSpecYaml(
                 name="bad",
                 initial_inputs=(ArtifactType.IMAGE,),
@@ -241,7 +242,7 @@ class TestInputsFromValidation:
             )
 
     def test_rejects_step_not_producing_referenced_type(self) -> None:
-        with pytest.raises(Exception, match="ne produit pas"):
+        with pytest.raises(ValidationError, match="ne produit pas"):
             PipelineSpecYaml(
                 name="bad",
                 initial_inputs=(ArtifactType.IMAGE,),

tests/architecture/test_no_broad_pytest_raises.py

@@ -72,7 +72,7 @@ def _scan_broad_raises() -> list[tuple[Path, int]]:
 #:
 #: 1. Remplacer ``pytest.raises(Exception)`` par la classe précise.
 #: 2. Baisser :data:`BROAD_RAISES_BASELINE` du même montant.
-BROAD_RAISES_BASELINE = 24
+BROAD_RAISES_BASELINE = 0
 
 
 def test_broad_pytest_raises_below_baseline() -> None:

tests/domain/test_sprint_a14_s4_artifacts.py

@@ -13,6 +13,7 @@ from __future__ import annotations
 import hashlib
 
 import pytest
+from pydantic import ValidationError
 
 from picarones.domain import (
     Artifact,
@@ -115,7 +116,7 @@ class TestArtifactCreation:
 
     def test_content_hash_must_be_64_hex(self) -> None:
         # Trop court
-        with pytest.raises(Exception):  # pydantic ValidationError
+        with pytest.raises(ValidationError):
             Artifact(
                 id="x", document_id="d1", type=ArtifactType.RAW_TEXT,
                 content_hash="abc",
@@ -143,11 +144,11 @@ class TestArtifactCreation:
 class TestArtifactImmutability:
     def test_frozen_blocks_attribute_mutation(self) -> None:
         a = Artifact(id="x", document_id="d1", type=ArtifactType.RAW_TEXT)
-        with pytest.raises(Exception):  # pydantic ValidationError
+        with pytest.raises(ValidationError):
             a.id = "y"  # type: ignore[misc]
 
     def test_extra_fields_rejected(self) -> None:
-        with pytest.raises(Exception):  # pydantic ValidationError
+        with pytest.raises(ValidationError):
             Artifact(  # type: ignore[call-arg]
                 id="x", document_id="d1", type=ArtifactType.RAW_TEXT,
                 bogus_field="oops",

tests/domain/test_sprint_a14_s4_corpus.py

@@ -3,6 +3,7 @@
 from __future__ import annotations
 
 import pytest
+from pydantic import ValidationError
 
 from picarones.domain import ArtifactType, CorpusSpec, CorpusSpecError, DocumentRef, GroundTruthRef
 
@@ -45,14 +46,14 @@ class TestCorpusSpec:
         assert c.metadata["language"] == "fr"
 
     def test_name_validation(self) -> None:
-        with pytest.raises(Exception):  # pydantic ValidationError
+        with pytest.raises(ValidationError):
             CorpusSpec(name="")  # min_length=1
 
 
 class TestCorpusSpecImmutability:
     def test_frozen_blocks_mutation(self) -> None:
         c = CorpusSpec(name="x")
-        with pytest.raises(Exception):
+        with pytest.raises(ValidationError):
             c.name = "y"  # type: ignore[misc]
 
     def test_json_roundtrip_with_multilevel_gt(self) -> None:

tests/domain/test_sprint_a14_s4_documents.py

@@ -81,8 +81,10 @@ class TestMultiLevelGT:
 
 class TestDocumentRefImmutability:
     def test_frozen_blocks_mutation(self) -> None:
+        from pydantic import ValidationError
+
         d = DocumentRef(id="x")
-        with pytest.raises(Exception):
+        with pytest.raises(ValidationError):
             d.id = "y"  # type: ignore[misc]
 
     def test_json_roundtrip(self) -> None:

tests/domain/test_sprint_a14_s4_provenance_errors.py

@@ -39,8 +39,10 @@ class TestProvenanceRecord:
         assert not p1.is_compatible_with(p4)  # parameters_hash diffère
 
     def test_frozen(self) -> None:
+        from pydantic import ValidationError
+
         p = ProvenanceRecord(code_version="1.0.0")
-        with pytest.raises(Exception):
+        with pytest.raises(ValidationError):
             p.code_version = "1.0.1"  # type: ignore[misc]
 
     def test_json_roundtrip(self) -> None:

tests/domain/test_sprint_a14_s5_evaluation_specs.py

@@ -8,6 +8,7 @@ des dataclasses pydantic.
 from __future__ import annotations
 
 import pytest
+from pydantic import ValidationError
 
 from picarones.domain import (
     ArtifactType,
@@ -50,7 +51,7 @@ class TestMetricSpec:
             name="cer",
             input_types=(ArtifactType.RAW_TEXT, ArtifactType.RAW_TEXT),
         )
-        with pytest.raises(Exception):  # pydantic ValidationError
+        with pytest.raises(ValidationError):
             spec.name = "wer"  # type: ignore[misc]
 
     def test_no_callable_field(self) -> None:
@@ -63,7 +64,7 @@ class TestMetricSpec:
         assert not hasattr(spec, "func")
 
     def test_extra_field_rejected(self) -> None:
-        with pytest.raises(Exception):
+        with pytest.raises(ValidationError):
             MetricSpec(  # type: ignore[call-arg]
                 name="cer",
                 input_types=(ArtifactType.RAW_TEXT, ArtifactType.RAW_TEXT),
@@ -112,7 +113,7 @@ class TestProjectionSpec:
             target_type=ArtifactType.RAW_TEXT,
             projector_name="alto_to_text",
         )
-        with pytest.raises(Exception):
+        with pytest.raises(ValidationError):
             p.projector_name = "other"  # type: ignore[misc]
 
     def test_json_roundtrip(self) -> None:
@@ -204,7 +205,7 @@ class TestEvaluationView:
             name="x",
             candidate_types=frozenset({ArtifactType.RAW_TEXT}),
         )
-        with pytest.raises(Exception):
+        with pytest.raises(ValidationError):
             view.name = "y"  # type: ignore[misc]
 
     def test_json_roundtrip(self) -> None:
@@ -256,5 +257,5 @@ class TestEvaluationSpec:
 
     def test_frozen(self) -> None:
         s = EvaluationSpec()
-        with pytest.raises(Exception):
+        with pytest.raises(ValidationError):
             s.views = ()  # type: ignore[misc]

tests/pipeline/test_sprint_a14_s28_planner.py

@@ -557,7 +557,10 @@ class TestPipelineExecutorWithPlanner:
         executor = PipelineExecutor(
             adapter_resolver=lambda n: _OCRStub(),
         )
-        with pytest.raises(Exception, match="ExecutionPlan"):
+        # ``PipelineExecutor.run_plan`` lève ``PicaronesError`` quand
+        # l'argument ``plan`` n'est pas un ``ExecutionPlan``.
+        from picarones.domain.errors import PicaronesError
+        with pytest.raises(PicaronesError, match="ExecutionPlan"):
             executor.run_plan(
                 plan="not a plan",  # type: ignore[arg-type]
                 document=DocumentRef(id="d1"),
@@ -623,7 +626,8 @@ class TestPipelineExecutorWithPlanner:
         assert plan.metric_junctions  # non vide grâce au registry injecté
 
     def test_planner_must_be_pipeline_planner(self) -> None:
-        with pytest.raises(Exception, match="PipelinePlanner"):
+        from picarones.domain.errors import PicaronesError
+        with pytest.raises(PicaronesError, match="PipelinePlanner"):
             PipelineExecutor(
                 adapter_resolver=lambda n: _OCRStub(),
                 planner="not a planner",  # type: ignore[arg-type]

tests/pipeline/test_sprint_a14_s6_spec.py

@@ -112,6 +112,8 @@ class TestPipelineSpec:
         assert s.step_by_id("missing") is None
 
     def test_frozen(self) -> None:
+        from pydantic import ValidationError
+
         s = PipelineSpec(name="x")
-        with pytest.raises(Exception):
+        with pytest.raises(ValidationError):
             s.name = "y"  # type: ignore[misc]