tests updates

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()
 

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]))
 	}
 }
 

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)

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")