aboutsummaryrefslogtreecommitdiff
path: root/internal/readings/partids_test.go
diff options
context:
space:
mode:
authorLukasz Kasprzak <lukas@labunix.xyz>2026-08-05 14:10:34 +0200
committerLukasz Kasprzak <lukas@labunix.xyz>2026-08-05 14:10:34 +0200
commiteb4a02205f5b3b972c44b854d5becbbd0b522981 (patch)
tree6bf2014c441e71dfeb8a06d17c07355c57c51035 /internal/readings/partids_test.go
parent197ce6980daa1c3773eed1548f54861d616d35e4 (diff)
downloadlectio-eb4a02205f5b3b972c44b854d5becbbd0b522981.tar.gz
lectio-eb4a02205f5b3b972c44b854d5becbbd0b522981.zip
readings: PartIDs(new) stops declaring aklamacja, which the engine never emits
caldata.go:42 parses only first/psalm/second/gospel out of the lectionary data, so no OF reading ever carries Part == "acclamation". PartIDs("new") listed aklamacja anyway, so the app built a checkbox from it that filters an ID that never appears -- a dead control, same defect class the app previously shipped for the whole 1962 form. ofPartOrder stays the full five-ID set: it also drives render.LocalizeHeading's label matching, where a scraped heading can still read "Aklamacja" even though this engine's own readings never produce that section. PartIDs now draws from a new, narrower ofEmittedPartOrder instead. Rewrote TestPartLabelsMatchesWhatTheEngineEmits's OF half: it compared PartLabels("of") to a hand-copied duplicate of ofPartOrder, asserting a declaration against itself, which cannot fail for this class of bug. It now sweeps Days over calendar year 2026 and asserts PartLabels("of") matches the observed part-ID set exactly (0.5s). Confirmed red against the pre-fix code, green after.
Diffstat (limited to 'internal/readings/partids_test.go')
-rw-r--r--internal/readings/partids_test.go2
1 files changed, 1 insertions, 1 deletions
diff --git a/internal/readings/partids_test.go b/internal/readings/partids_test.go
index 1ba9373..fdbd816 100644
--- a/internal/readings/partids_test.go
+++ b/internal/readings/partids_test.go
@@ -35,7 +35,7 @@ func TestDayInfoCarriesRank(t *testing.T) {
// them instead, seven of the nine 1962 IDs were wrong and the epistle's
// checkbox did nothing.
func TestPartIDs(t *testing.T) {
- wantNew := []string{"pierwsze_czytanie", "psalm", "drugie_czytanie", "aklamacja", "ewangelia"}
+ wantNew := []string{"pierwsze_czytanie", "psalm", "drugie_czytanie", "ewangelia"}
wantOld := []string{"epistola", "evangelium"}
if got := PartIDs("new"); !equalSlice(got, wantNew) {
t.Errorf("PartIDs(new) = %v, want %v", got, wantNew)