kernel(validate): resolution invariants

Widen Validate.run to take the rite's sanctoral layer alongside the rite
itself (Calendar.year needs both), and add five checks over the fully
resolved liturgical year, on top of the existing temporal-only pass:

- observed: a day's observed celebration never also appears among that
  same day's own commemorations/omissions.
- lost: no sanctoral entry is silently dropped. Per slug, the number of
  times it is actually sighted (observed + commemorations + omitted,
  summed over the year) must never fall below the number of times its
  own Date_spec resolves within the year's span -- also fires if
  resolving the year raises at all, the most total form of loss.
- duplicated: the same per-slug count must never exceed the number of
  Date_spec resolutions either. Deliberately NOT "no slug appears
  twice": a fixed date can legitimately resolve twice in the ~20% of
  liturgical years whose 371-day span reaches it on both ends (30
  November/St Andrew is the worked example in validate.mli).
- unconverged: no day's omitted reason indicates Calendar's placement
  pass hit its round guard before reaching a fixed point.
- admission: the rite's own rules.admit is a fixed point on what it
  already admitted -- the rite-agnostic form of "the admission limit
  was not exceeded" available without embedding a rite's own numeric
  caps (RG 111's, for EF) into kernel code.

Each check has a dedicated negative fixture in the synthetic rite
(test_validate.ml), hand-traced against Calendar's actual resolution
mechanics before writing the assertion, and verified to fail for the
right reason against the code before this change. One pair
(unconverged/duplicated) is not fully independent: hitting the round
guard genuinely also trips duplicated, a real consequence of Calendar's
own accounting once a candidate is simultaneously sighted at its
permanent natural date and wherever the last placement round left it --
documented in guard_rules's own comment, not papered over.

test_validate.ml's ef_rite/run now use the real Rite_ef.context and the
real bootstrapped data/ef layer (Precedence_ef and the sanctoral
bootstrap did not exist when this scaffolding was first written) rather
than the earlier placeholder rules. Validate is clean across the whole
1583..9999 domain against real EF data except the one already-documented
year-9999 truncation case (test_year_9999_does_not_raise).

1 files changed, 284 insertions, 23 deletions

diff --git a/test/test_validate.ml b/test/test_validate.ml
index df8c99c..b2d4f34 100644
--- a/test/test_validate.ml
+++ b/test/test_validate.ml
@@ -1,24 +1,40 @@
 module Val = Colitur_kernel.Validate
 module Rite = Colitur_kernel.Rite
 module P = Colitur_kernel.Precedence
+module Layer = Colitur_kernel.Layer
+module Overlay = Colitur_kernel.Overlay
 module V = Rite_ef.Vocab_ef
 module T = Rite_ef.Temporal_ef
 
-(* Plan 3's real EF precedence rules (Precedence_ef, Tasks 7-11) don't exist
-   yet -- Validate.run doesn't read [rules] or [transfer_target] at all
-   (nothing does before Task 5's Calendar and Task 6's placement pass), so a
-   placeholder is enough to assemble a well-typed Rite.t here. *)
-let ef_rules : (V.season, V.rank) P.rules =
-  { P.band = (fun _ _ -> 0);
-    disposition = (fun ~winner:_ ~loser:_ -> P.Omit);
-    admit = (fun ~observed:_ _ -> []) }
+(* Task 12 widens Validate.run to take a resolved layer -- Precedence_ef and
+   Calendar (Tasks 5-11) now exist, so the REAL EF rite (Rite_ef.context) and
+   its REAL bootstrapped data replace the earlier placeholder rules/layer-less
+   Rite.t this module used before Plan 3's resolution engine was built.
+   Relative to this test's own build directory (_build/default/test/), same
+   convention test_rite_ef.ml already uses -- test/dune declares both as deps
+   of the (test ...) stanza. *)
+let sanctoral_path = "../data/ef/sanctoral.sexp"
+let adjustments_path = "../data/ef/adjustments.sexp"
 
-let ef_rite : (V.season, V.rank) Rite.t =
-  { Rite.id = T.id; vocab = V.vocab; year_start = T.year_start; temporal = T.temporal;
-    anchors = T.anchors; rules = ef_rules; season_runs = V.seasons;
-    transfer_target = (fun _ origin _ -> origin) }
+(* Loaded once at module init, not per call: [run] below is called by every
+   test and by the 200-sample property, and Calendar.year's own resolution
+   cost already dominates -- there is no reason to also re-parse a 322-entry
+   sexp file on every one of those calls. *)
+let real_ef_layer =
+  match Layer.load V.rank_of_sexp sanctoral_path with
+  | Error e -> failwith (Printf.sprintf "%s: failed to load: %s" sanctoral_path e)
+  | Ok layer -> (
+      match Overlay.load V.rank_of_sexp adjustments_path with
+      | Error e -> failwith (Printf.sprintf "%s: failed to load: %s" adjustments_path e)
+      | Ok overlay ->
+          let layer, diagnostics = Overlay.apply layer overlay in
+          if diagnostics <> [] then
+            failwith
+              (Printf.sprintf "unexpected overlay diagnostics: %s"
+                 (String.concat "; " (List.map Overlay.diagnostic_to_string diagnostics)));
+          layer)
 
-let run year = Val.run ef_rite ~year
+let run year = Val.run Rite_ef.context real_ef_layer ~year
 
 let check_year year =
   match run year with
@@ -94,6 +110,8 @@ module Synthetic = struct
   module Temporal = Colitur_kernel.Temporal
   module P = Colitur_kernel.Precedence
   module Rite = Colitur_kernel.Rite
+  module Layer = Colitur_kernel.Layer
+  module Date_spec = Colitur_kernel.Date_spec
 
   type season = A | B
   type rank = R1 | R2
@@ -122,8 +140,12 @@ module Synthetic = struct
   let vocab_collapsed_ranks = { vocab with Vocab.rank_to_string = (fun _ -> "same") }
   let vocab_collapsed_seasons = { vocab with Vocab.season_to_string = (fun _ -> "same") }
 
-  (* Precedence_ef doesn't exist yet (Tasks 7-11); Validate.run never reads
-     [rules], so a placeholder is enough to assemble a well-typed Rite.t. *)
+  (* The placeholder ruleset every TEMPORAL-only fixture below still uses:
+     paired with the default empty [layer] (see [rite] below), there is never
+     a sanctoral candidate for these three functions to be called against, so
+     what they return is moot for those tests -- only the Task 12 resolution
+     fixtures further down override [rules] (and supply a non-empty
+     [layer]), each with its own small, deliberately-shaped ruleset. *)
   let rules : (season, rank) P.rules =
     { P.band = (fun _ _ -> 0);
       disposition = (fun ~winner:_ ~loser:_ -> P.Omit);
@@ -173,14 +195,25 @@ module Synthetic = struct
      this synthetic rite too, not only in EF. *)
   let anchors _y = [ (Slug.to_string (good target).Temporal.office.Cel.slug, target) ]
 
-  let rite ?(vocab = vocab) ?(anchors = fun _ -> []) ?(season_runs = [ A; B ]) temporal
-      : (season, rank) Rite.t =
+  (* Task 12: [rite] now also takes [rules]/[transfer_target] (defaulting to
+     the placeholder above and to "stand still", respectively -- harmless
+     defaults against the default empty [layer], since nothing ever contests
+     the temporal office there) so the resolution fixtures further down can
+     override them without duplicating every other field. *)
+  let rite ?(vocab = vocab) ?(anchors = fun _ -> []) ?(season_runs = [ A; B ]) ?(rules = rules)
+      ?(transfer_target = fun _ origin _ -> origin) temporal : (season, rank) Rite.t =
     { Rite.id = "synthetic"; vocab; year_start; temporal; anchors; rules; season_runs;
-      (* Validate.run doesn't read this either (see [ef_rules] above). *)
-      transfer_target = (fun _ origin _ -> origin) }
+      transfer_target }
 
-  let run ?vocab ?anchors ?season_runs temporal =
-    Val.run (rite ?vocab ?anchors ?season_runs temporal) ~year:2026
+  (* Empty by default: every check built before Task 12 exercises the
+     TEMPORAL-only pass, where an empty layer is exactly the fixture that
+     leaves it unable to affect anything ([Precedence.resolve] against no
+     sanctoral candidates always just observes the temporal office
+     unchallenged). Task 12's own resolution fixtures pass their own. *)
+  let empty_layer = Layer.empty ~id:"synthetic-empty" ~name:"empty"
+
+  let run ?vocab ?anchors ?season_runs ?rules ?transfer_target ?(layer = empty_layer) temporal =
+    Val.run (rite ?vocab ?anchors ?season_runs ?rules ?transfer_target temporal) layer ~year:2026
 
   let has_check check (fs : Val.failure list) = List.exists (fun f -> f.Val.check = check) fs
 
@@ -210,6 +243,180 @@ module Synthetic = struct
 
   let rite_with_two_runs : (season, rank) Rite.t =
     rite ~season_runs:[ A; B; A; B ] two_run_temporal
+
+  (* ---- Task 12: resolution-level fixtures ----
+
+     Everything above only ever drives the TEMPORAL-only pass: [run]'s
+     default [layer] is empty, so [Precedence.resolve] never has a sanctoral
+     candidate to contest against the temporal office, and [rules]/
+     [transfer_target] are never meaningfully exercised. These five fixtures
+     instead give [Calendar.year] real work -- a non-empty [layer] plus a
+     small, deliberately-shaped [rules] (and, for two of them,
+     [transfer_target]) -- each built so its OWN check label fires. Four of
+     the five fire in clean isolation (the other four Task 12 labels stay
+     silent); the fifth (["unconverged"]) genuinely also fires
+     ["duplicated"] alongside it, a real consequence of Calendar's own round-
+     guard accounting, not a fixture design flaw -- see guard_rules's own
+     comment. Every fixture's isolation (or lack of it) was verified by
+     hand-tracing [Calendar]'s resolution mechanics BEFORE writing its
+     assertion (see the task report), not inferred from what the assertion
+     happens to require -- the tests below check that trace against the
+     actual engine output, one fixture at a time. [good] is reused,
+     unchanged, as every fixture's [temporal]: only [rules]/[layer]/
+     [transfer_target] vary, so the temporal-only checks (already proven
+     clean against [good] by [test_synthetic_baseline_is_clean]) cannot be
+     what fires here. *)
+
+  let mk_entry ~month ~day ~slug ~rank =
+    { Layer.date = (match Date_spec.fixed ~month ~day with Ok d -> d | Error e -> failwith e);
+      cel = Cel.make ~slug:(Slug.of_string_exn slug) ~rank ~colour:Colour.White
+              ~layer:"synthetic-sanctoral" () }
+
+  let task12_checks = [ "observed"; "lost"; "duplicated"; "unconverged"; "admission" ]
+
+  (* The Task-12-owned subset of a failure list's own check labels, as a
+     sorted, de-duplicated set -- what each isolation assertion below
+     compares against, so a fixture that (by mistake) also trips an
+     unrelated Task-12 check shows up as a wrong set, not a silently-passing
+     [has_check]. *)
+  let fired_task12_checks (fs : Val.failure list) =
+    List.filter_map (fun f -> if List.mem f.Val.check task12_checks then Some f.Val.check else None) fs
+    |> List.sort_uniq compare
+
+  (* "duplicated": a single ordinary sanctoral entry, always losing to the
+     temporal office (band: Temporal 0 < Sanctoral 10, unconditionally) and
+     always Commemorate-disposed. [admit]'s bug is exactly the shape
+     precedence_ef.mli's own [admit] contract warns against ("a value taken
+     unchanged from comms, never rebuilt"): it REBUILDS every admitted pair
+     via a record update, allocating a fresh, structurally-identical-but-
+     physically-distinct candidate. Precedence.resolve's own [dropped]
+     computation tells an admitted candidate from a dropped one by PHYSICAL
+     equality, so the rebuild defeats it -- the one candidate ends up counted
+     as both admitted (in [commemorations]) and dropped (in [omitted],
+     "admission limit reached"): two sightings for one Date_spec
+     resolution. *)
+  let dup_entry = mk_entry ~month:5 ~day:5 ~slug:"dup-target" ~rank:R2
+  let dup_layer = Layer.of_entries ~id:"dup" ~name:"dup" [ dup_entry ]
+
+  let dup_rules : (season, rank) P.rules =
+    { P.band = (fun _ c -> match c.P.origin with P.Temporal -> 0 | P.Sanctoral -> 10);
+      disposition = (fun ~winner:_ ~loser:_ -> P.Commemorate P.Ordinary);
+      admit = (fun ~observed:_ cs -> List.map (fun (c, p) -> ({ c with P.origin = c.P.origin }, p)) cs) }
+
+  (* "unconverged": two entries collide on one date (6 June), both beating
+     the temporal office and tied with each other, so slug decides:
+     "guard-aaa" wins the day outright every time, "guard-zzz" is always the
+     loser there and always Transfer-disposed. Paired with a
+     [transfer_target] that always answers with the impeded day itself --
+     never strictly forward -- this is the exact non-terminating shape
+     test_calendar.ml's own
+     [test_transfer_guard_records_failure_instead_of_looping] proves hits
+     Calendar's round guard: "guard-zzz", re-injected into 6 June every
+     round, can never win it (it always loses the tie to the natural
+     "guard-aaa" copy already sitting there), so [deferred] never empties.
+
+     GENUINE FINDING (see the task report): this ALSO fires "duplicated", not
+     "unconverged" alone. Once the guard is hit, [build_day]'s final resolve
+     at 6 June sees "guard-zzz" TWICE -- once as the permanent natural entry
+     (which never stops losing there) and once as whatever the last round's
+     [injected] state still holds for it -- and [unresolved] is evaluated per
+     CANDIDATE OBJECT, not per slug, so BOTH copies land in [omitted] with
+     the unconverged reason. Nothing is lost (both copies carry a recorded
+     reason), but the slug is sighted twice against one Date_spec resolution,
+     which is exactly what "duplicated" is for. The same double-recording is
+     latent in test_calendar.ml's own guard fixture too (day_winner/eclipsed
+     at 20 Dec, structurally identical) -- untested there only because that
+     test uses [List.exists], not a count. Calendar's round guard is
+     documented as "nothing in the 1962 calendar is expected to trigger"
+     (calendar.ml), so this is a latent accounting quirk in an unreachable
+     path, not a live bug, and calendar.ml is out of this task's file list --
+     reported, not fixed here. *)
+  let guard_winner_entry = mk_entry ~month:6 ~day:6 ~slug:"guard-aaa" ~rank:R1
+  let guard_loser_entry = mk_entry ~month:6 ~day:6 ~slug:"guard-zzz" ~rank:R1
+  let guard_layer = Layer.of_entries ~id:"guard" ~name:"guard" [ guard_winner_entry; guard_loser_entry ]
+
+  let guard_rules : (season, rank) P.rules =
+    { P.band = (fun _ c -> match c.P.origin with P.Temporal -> 50 | P.Sanctoral -> 10);
+      disposition =
+        (fun ~winner:_ ~loser ->
+          match loser.P.cel.Cel.rank with R1 -> P.Transfer | R2 -> P.Commemorate P.Ordinary);
+      admit = (fun ~observed:_ cs -> cs) }
+
+  let guard_transfer_target (_ : rank P.candidate) (origin : D.t) (_ : D.t -> rank Cel.t) = origin
+
+  (* "admission": three entries collide on one date (9 September), all
+     losing to the temporal office (band: Temporal 0 < Sanctoral 10) and all
+     Commemorate-disposed -- a genuine 3-candidate offer to [admit]. The bug:
+     cap 2 when the offer's length is ODD, cap 1 when EVEN -- a length-keyed
+     rule with no liturgical meaning, chosen as the simplest function that is
+     NOT idempotent on its own output (offer 3, admit 2; re-offer those same
+     2, admit only 1) while staying idempotent -- and so invisible -- on
+     every OTHER shape this suite exercises (never offered exactly 2 or 3
+     candidates elsewhere), including its own clean 3-candidate day. *)
+  let adm_a_entry = mk_entry ~month:9 ~day:9 ~slug:"adm-a" ~rank:R2
+  let adm_b_entry = mk_entry ~month:9 ~day:9 ~slug:"adm-b" ~rank:R2
+  let adm_c_entry = mk_entry ~month:9 ~day:9 ~slug:"adm-c" ~rank:R2
+  let adm_layer = Layer.of_entries ~id:"adm" ~name:"adm" [ adm_a_entry; adm_b_entry; adm_c_entry ]
+
+  let adm_compare_slug (c1, _) (c2, _) = Slug.compare c1.P.cel.Cel.slug c2.P.cel.Cel.slug
+
+  let rec adm_take n = function
+    | [] -> []
+    | x :: xs -> if n <= 0 then [] else x :: adm_take (n - 1) xs
+
+  let adm_rules : (season, rank) P.rules =
+    { P.band = (fun _ c -> match c.P.origin with P.Temporal -> 0 | P.Sanctoral -> 10);
+      disposition = (fun ~winner:_ ~loser:_ -> P.Commemorate P.Ordinary);
+      admit =
+        (fun ~observed:_ cs ->
+          let sorted = List.stable_sort adm_compare_slug cs in
+          if List.length sorted mod 2 = 1 then adm_take 2 sorted else adm_take 1 sorted) }
+
+  (* "observed": two DIFFERENT layer entries sharing one slug -- a realistic
+     data mistake (a renamed or duplicated entry), not prevented by
+     [Layer.t]'s own type. [collide_a] (1 Feb, rank R1) always loses on its
+     OWN date: [band] makes the temporal office win there specifically (5,
+     beating R1's 10) and lose everywhere else (50), so [collide_a] is always
+     Transfer-disposed at 1 Feb. Its constant [transfer_target] sends it to
+     10 Feb -- [collide_b]'s own home date -- where [collide_a]'s rank R1
+     (band 10) now beats both the temporal office (50, since the date is no
+     longer 1 Feb) and [collide_b]'s own rank R2 (band 90): the ARRIVING
+     [collide_a] wins 10 Feb outright, and [collide_b] -- same slug as the
+     new winner -- is Commemorate-disposed (R2) right alongside it. One day
+     ends up reporting the same slug as both its observed celebration and one
+     of its own commemorations. *)
+  let collide_d1 = match D.make ~year:2026 ~month:2 ~day:1 with Ok d -> d | Error e -> failwith e
+  let collide_a_entry = mk_entry ~month:2 ~day:1 ~slug:"collide-x" ~rank:R1
+  let collide_b_entry = mk_entry ~month:2 ~day:10 ~slug:"collide-x" ~rank:R2
+  let collide_layer = Layer.of_entries ~id:"collide" ~name:"collide" [ collide_a_entry; collide_b_entry ]
+
+  let collide_rules : (season, rank) P.rules =
+    { P.band =
+        (fun ctx c ->
+          match c.P.origin with
+          | P.Temporal -> if D.compare ctx.P.date collide_d1 = 0 then 5 else 50
+          | P.Sanctoral -> ( match c.P.cel.Cel.rank with R1 -> 10 | R2 -> 90));
+      disposition =
+        (fun ~winner:_ ~loser ->
+          match loser.P.cel.Cel.rank with R1 -> P.Transfer | R2 -> P.Commemorate P.Ordinary);
+      admit = (fun ~observed:_ cs -> cs) }
+
+  let collide_d2 = match D.make ~year:2026 ~month:2 ~day:10 with Ok d -> d | Error e -> failwith e
+  let collide_transfer_target (_ : rank P.candidate) (_ : D.t) (_ : D.t -> rank Cel.t) = collide_d2
+
+  (* A genuinely resolved, well-behaved day (one ordinary sanctoral entry,
+     cleanly losing and commemorated, nothing transferred) -- proving the
+     five checks stay silent against REAL resolution machinery, not merely
+     against the default empty [layer] every fixture above this section
+     uses. Without this, "no check fires" would only ever have been shown
+     for a layer with nothing in it. *)
+  let clean_sanctoral_entry = mk_entry ~month:8 ~day:8 ~slug:"clean-saint" ~rank:R2
+  let clean_sanctoral_layer = Layer.of_entries ~id:"clean" ~name:"clean" [ clean_sanctoral_entry ]
+
+  let clean_sanctoral_rules : (season, rank) P.rules =
+    { P.band = (fun _ c -> match c.P.origin with P.Temporal -> 0 | P.Sanctoral -> 10);
+      disposition = (fun ~winner:_ ~loser:_ -> P.Commemorate P.Ordinary);
+      admit = (fun ~observed:_ cs -> cs) }
 end
 
 open Synthetic
@@ -227,7 +434,7 @@ let test_synthetic_baseline_is_clean () =
 let test_two_run_season_is_accepted () =
   let r = Synthetic.rite_with_two_runs in
   Alcotest.(check (list string)) "no failures" []
-    (List.map Val.failure_to_string (Val.run r ~year:2026))
+    (List.map Val.failure_to_string (Val.run r Synthetic.empty_layer ~year:2026))
 
 let test_coverage_fires () =
   let temporal d = if D.compare d target = 0 then failwith "boom" else good d in
@@ -332,6 +539,53 @@ let test_vocab_season_injectivity_fires () =
   Alcotest.(check bool) "vocab check fires when season_to_string collapses two seasons to one string" true
     (has_check "vocab" (run ~vocab:vocab_collapsed_seasons good))
 
+(* ---- Task 12: resolution invariants ----
+
+   Each test below asserts that exactly one of the five new check labels
+   fires for its own dedicated fixture (Synthetic's own comments carry the
+   hand-traced mechanics) -- not merely "at least this one", so a fixture
+   that turns out to also trip an unrelated Task 12 check would fail loudly
+   here rather than reading as accidental corroboration. *)
+
+let test_lost_fires_on_resolution_exception () =
+  (* Reuses [test_coverage_fires]'s own broken [temporal]: [Calendar.year]
+     calls [rite.temporal] with no exception guard of its own (unlike the
+     temporal-only pass above, which wraps every call), so the same raise
+     that trips "coverage" also makes resolution itself raise -- the most
+     total form of "silently lost" there is, per validate.mli. *)
+  let temporal d = if D.compare d target = 0 then failwith "boom" else good d in
+  Alcotest.(check (list string)) "only the lost check fires" [ "lost" ] (fired_task12_checks (run temporal))
+
+let test_duplicated_fires () =
+  Alcotest.(check (list string)) "only the duplicated check fires" [ "duplicated" ]
+    (fired_task12_checks (run ~layer:dup_layer ~rules:dup_rules good))
+
+let test_unconverged_fires () =
+  (* Also asserts "duplicated" fires alongside it -- see guard_rules's own
+     comment for why that is the genuine, hand-verified consequence of
+     hitting the round guard here, not an isolation failure. *)
+  Alcotest.(check (list string)) "unconverged fires, and duplicated alongside it"
+    [ "duplicated"; "unconverged" ]
+    (fired_task12_checks
+       (run ~layer:guard_layer ~rules:guard_rules ~transfer_target:guard_transfer_target good))
+
+let test_admission_fires () =
+  Alcotest.(check (list string)) "only the admission check fires" [ "admission" ]
+    (fired_task12_checks (run ~layer:adm_layer ~rules:adm_rules good))
+
+let test_observed_fires () =
+  Alcotest.(check (list string)) "only the observed check fires" [ "observed" ]
+    (fired_task12_checks
+       (run ~layer:collide_layer ~rules:collide_rules ~transfer_target:collide_transfer_target good))
+
+(* The positive counterpart: a genuinely resolved, well-behaved day (real
+   sanctoral entry, real contest, real commemoration) must report none of the
+   five checks -- proven against actual resolution machinery, not only
+   against every OTHER fixture's default empty layer. *)
+let test_resolution_checks_clean_on_a_well_behaved_layer () =
+  Alcotest.(check (list string)) "none of the five checks fire" []
+    (fired_task12_checks (run ~layer:clean_sanctoral_layer ~rules:clean_sanctoral_rules good))
+
 let suite =
   ( "Validate",
     [ Alcotest.test_case "landmark years" `Quick test_landmark_years;
@@ -349,5 +603,12 @@ let suite =
       Alcotest.test_case "anchor clean" `Quick test_anchor_clean;
       Alcotest.test_case "anchor fires" `Quick test_anchor_fires;
       Alcotest.test_case "vocab rank injectivity fires" `Quick test_vocab_rank_injectivity_fires;
-      Alcotest.test_case "vocab season injectivity fires" `Quick test_vocab_season_injectivity_fires ]
+      Alcotest.test_case "vocab season injectivity fires" `Quick test_vocab_season_injectivity_fires;
+      Alcotest.test_case "lost fires on resolution exception" `Quick test_lost_fires_on_resolution_exception;
+      Alcotest.test_case "duplicated fires" `Quick test_duplicated_fires;
+      Alcotest.test_case "unconverged fires" `Quick test_unconverged_fires;
+      Alcotest.test_case "admission fires" `Quick test_admission_fires;
+      Alcotest.test_case "observed fires" `Quick test_observed_fires;
+      Alcotest.test_case "resolution checks clean on a well-behaved layer" `Quick
+        test_resolution_checks_clean_on_a_well_behaved_layer ]
     @ List.map QCheck_alcotest.to_alcotest [ prop_invariants ] )