amadan
Answer the review (Claude Fable 5.1)
- A test pins the layout's Onest link to estilo's versioned prefix (not needed where an equivalent already exists). - Slides: the credits page test expects Onest; a committed .pyc removed and __pycache__/ ignored. Calendar, Memoria: "'s" in writetheme's comment. Fichas: estilo is a direct requirement (go mod tidy).
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 file changed,
+24
−0
internal/callstest/fontlink_test.go+24 −0
CI passed — run details
diff --git a/internal/callstest/fontlink_test.go b/internal/callstest/fontlink_test.go| new file mode 100644 |
| index 0000000..358bb10 |
| --- /dev/null |
| +++ b/internal/callstest/fontlink_test.go |
| @@ -0,0 +1,24 @@ |
| +package callstest |
| + |
| +import ( |
| + "bytes" |
| + "os" |
| + "path/filepath" |
| + "testing" |
| + |
| + "amadan.net/oficina/estilo.git/theme" |
| +) |
| + |
| +// The layout names Onest's stylesheet by a literal path; this keeps that |
| +// literal equal to estilo's versioned prefix, so an estilo font move |
| +// fails here rather than every page quietly falling back to the system |
| +// face. |
| +func TestLayoutLinksEstilosFonts(t *testing.T) { |
| + layout, err := os.ReadFile(filepath.Join("..", "calls", "templates", "layout.html")) |
| + if err != nil { |
| + t.Fatal(err) |
| + } |
| + if want := `href="` + theme.FontPrefix + `fonts.css"`; !bytes.Contains(layout, []byte(want)) { |
| + t.Errorf("layout.html does not link %s", want) |
| + } |
| +} |