chore: migrate database row-page, relation, and files E2E specs to data-testid selectors (#1065)

[sw-factory-automations]

Closes #1065

What

Replaces 10 fragile locator("text=...") and CSS-class selectors across three E2E spec files with getByTestId selectors targeting existing db-row-property-* and db-cell-editor-* testids.

How

e2e/database-row-page.spec.ts (7 selectors replaced):

• locator("text=Text") → locator('[data-testid^="db-row-property-name-"]', { hasText: "Text" }) scoped to getByTestId("db-row-properties")
• locator("text=Empty") → toContainText("Empty") on the value cell via [data-testid^="db-row-property-value-"]
• locator("text=Status") → same pattern as "Text" above
• locator("div.flex.items-start.gap-4", { hasText: "Status" }) → parent traversal from testid-located name cell
• locator(".rounded-sm.border.border-border.bg-background") → getByTestId("db-cell-editor-select")
• statusRow.locator("text=In Progress") → toContainText("In Progress") on the value cell
• locator("text=Hello from table") → toContainText("Hello from table") on getByTestId("db-row-properties")

e2e/database-relation.spec.ts (3 selectors replaced):

• locator("text=Linked Items") → getByTestId(\db-row-property-name-${relationPropertyId}`)`
• locator("text=Alpha Item") → toContainText("Alpha Item") on breadcrumb nav
• locator(".w-56") → getByTestId("db-cell-editor-relation")

e2e/database-files.spec.ts (1 selector replaced):

• locator("text=Attachments") → getByTestId(\db-row-property-name-${filesPropertyId}`)`

No new data-testid attributes were needed — all existing testids from #700 and #873 were sufficient.

Testing

• pnpm lint — 0 errors
• pnpm typecheck — passes
• pnpm test — 139 files, 1877 tests passed
• pnpm test:e2e — all modified tests pass; remaining flakiness is pre-existing auth timeouts unrelated to this change

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
memo	Ready	Preview, Comment	May 13, 2026 4:29am

[sw-factory-automations]

✅ UI verification skipped — this PR has no UI changes. Only E2E test selectors and .agents/quality.md were modified; no files under src/components/ or src/app/**/page.tsx / layout.tsx were touched.

[sw-factory-automations]

✅ Post-merge verification skipped — chore: PR does not affect the live app.