diff --git a/architecture_test.go b/architecture_test.go index 7254b63..9404f9e 100644 --- a/architecture_test.go +++ b/architecture_test.go @@ -70,23 +70,6 @@ func TestStoreCreateGameDoesNotUseNameDerivedRunMetadata(t *testing.T) { }) } -func TestPDFExportPackageDoesNotContainMarkdownParseEntrypoints(t *testing.T) { - assertFilesDoNotContain(t, "pdfexport", []string{ - "func ParseMarkdown(", - "func ParseFile(", - "func ParseFiles(", - "func lookupMarkdownBodyParser(", - }) -} - -func TestGamePackageDoesNotExposeLegacyPrintFacade(t *testing.T) { - if _, err := os.Stat(filepath.Join("game", "print_adapter.go")); err == nil { - t.Fatal("game/print_adapter.go should not exist") - } else if !os.IsNotExist(err) { - t.Fatalf("stat game/print_adapter.go: %v", err) - } -} - func assertPackageDoesNotImport(t *testing.T, dir string, forbidden []string) { t.Helper() diff --git a/nurikabe/nurikabe_test.go b/nurikabe/nurikabe_test.go index eb71d8a..660a85a 100644 --- a/nurikabe/nurikabe_test.go +++ b/nurikabe/nurikabe_test.go @@ -68,13 +68,13 @@ func TestGridStringAndParse(t *testing.T) { } } -func TestParseGridLegacyRequiredLand(t *testing.T) { +func TestParseGridMapsRequiredLandMarkerToIsland(t *testing.T) { g, err := parseGrid("?!\n~o", 2, 2) if err != nil { t.Fatalf("parseGrid error: %v", err) } if g[0][1] != islandCell { - t.Fatalf("legacy required-land marker should load as island, got %q", rune(g[0][1])) + t.Fatalf("required-land marker should load as island, got %q", rune(g[0][1])) } } diff --git a/store/store_test.go b/store/store_test.go index f2bde54..ee773b6 100644 --- a/store/store_test.go +++ b/store/store_test.go @@ -100,7 +100,7 @@ func TestOpen(t *testing.T) { } }) - t.Run("upgrades legacy databases and backfills metadata", func(t *testing.T) { + t.Run("upgrades databases missing run metadata and backfills it", func(t *testing.T) { dbPath := filepath.Join(t.TempDir(), "legacy.db") raw, err := sql.Open("sqlite", dbPath) diff --git a/ui/ui_test.go b/ui/ui_test.go index 1e29aca..5cfeaff 100644 --- a/ui/ui_test.go +++ b/ui/ui_test.go @@ -44,7 +44,7 @@ func TestMenuItemActionID(t *testing.T) { } }) - t.Run("falls back to title for legacy callers", func(t *testing.T) { + t.Run("falls back to title when action is unset", func(t *testing.T) { item := MenuItem{ItemTitle: "Generate"} if got := item.ActionID(); got != "Generate" { t.Fatalf("ActionID() = %q, want %q", got, "Generate")