aboutsummaryrefslogtreecommitdiff
path: root/test/test_oracle.ml
diff options
context:
space:
mode:
authorLukasz Kasprzak <lukas@labunix.xyz>2026-08-12 10:40:12 +0200
committerLukasz Kasprzak <lukas@labunix.xyz>2026-08-12 10:40:12 +0200
commitac569e859be08320e47909e496d6e5e8f6057da7 (patch)
tree3e8f8fdc4cae5de987e7886e4bfcc9fc0c77e38d /test/test_oracle.ml
parent40e7d5c9fb8ec4f68c1ea4ace3835d987a272fcb (diff)
downloadcolitur-ac569e859be08320e47909e496d6e5e8f6057da7.tar.gz
colitur-ac569e859be08320e47909e496d6e5e8f6057da7.zip
docs+test: small factual corrections (item 7, part 1)
Six independent, small corrections found during the final review: - dune (workspace root): the comment said the stanza used "(:standard)" to preserve dune's default `default` alias target; the stanza actually spells that out explicitly via (alias_rec install). Comment now matches the code. - test_validate.ml's test_easter_extremes asserted `List.length ys = 2` where an identity check was called for -- the comment already named 1598 and 1666, but nothing confirmed extreme_years() found THOSE two rather than some other pair with the right cardinality. Now asserts the identities directly (the project's "cardinality where identity was required" vacuity flavour, per the review). - test_oracle.ml and expected-divergences-missalemeum.sexp both claimed "one entry (M13) is [verdict open]" -- M11 is open too (its own verdict changed from colitur to open in fix round 1); both now say "two entries (M11 and M13)". - expected-divergences-missalemeum.sexp's M2 note attributed `band` to temporal_ef.ml; `band` is precedence_ef.ml's own function. - lib/kernel/precedence.mli documented `dropped`/`admit`'s physical- equality obligation nowhere -- it lived only in one rite's own module (Rite_ef.Precedence_ef.admit's doc comment), but this signature is what an author of the next rite actually reads. Added the obligation here, cross-referencing the EF instance as precedent, not the only source. - README's opam install line omitted sexplib and ppx_sexp_conv (both in dune-project's own depends; `dune build` fails without them for a contributor following the README verbatim) and documented only `colitur easter`, though `temporal` and `day` both exist and are the more useful entry points. Fixed both. No behaviour change: comment/doc/test-assertion corrections only (the easter-extremes fix strengthens an assertion, it does not change what passes). Verified byte-identical `colitur day` output across 1583, 1900, 1902, 2008, 2011, 2026, 2038, 9999. 259/259 tests green.
Diffstat (limited to 'test/test_oracle.ml')
-rw-r--r--test/test_oracle.ml10
1 files changed, 7 insertions, 3 deletions
diff --git a/test/test_oracle.ml b/test/test_oracle.ml
index 152464a..ac8d95d 100644
--- a/test/test_oracle.ml
+++ b/test/test_oracle.ml
@@ -56,9 +56,13 @@
entries; RG 110's inseparable-Peter/Paul commemoration is unimplemented
code, a real feature this task did not build) -- honestly verdicted
[missalemeum] (colitur is short a feature or a row, not right), never
- silently absorbed as if colitur were correct. One entry (M13) is
- [verdict open]: adjudicated as unresolved, not resolved either way --
- the brief's own explicit permission ("say so as an open item") used for
+ silently absorbed as if colitur were correct. TWO entries (M11 and
+ M13) are [verdict open] -- CORRECTED, final fix wave, item 7: this
+ comment previously said "one entry (M13)", missing M11, whose own
+ verdict was changed from [colitur] to [open] in fix round 1 (see M11's
+ own entry below for why) but this summary was never updated to match.
+ Both are adjudicated as unresolved, not resolved either way -- the
+ brief's own explicit permission ("say so as an open item") used for
real, not defaulted past. See the task report for every entry's full
reasoning and primary-source citation. *)