The remaining Playwright failures were all in interactions that depend on
the Blazor Server circuit being responsive in CI:
- Settings page access now navigates to /Settings/General directly instead
of expanding the flaky nested Settings SubMenu flyout.
- Drop SettingsMenu_ShowsAllExpectedSubItems; the submenu-expansion is the
unreliable interaction the bUnit migration exists to replace, and the
Settings pages are now covered by bUnit component tests.
- Remove the GameImport UI tests (and unused GamesPage/OpenRCT2.lcx). The
ChunkUploader modal never renders under the in-process test host, so they
failed in every CI run; the Games list render is already covered by
AdminNavigationTests.GamesPage_ShowsEmptyTable.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace the SignalR-circuit-race-prone Playwright component tests for
GameEdit, Settings, metadata, profile, roles and users with in-process
bUnit component tests. Keep a thin Playwright smoke layer for the true
E2E paths (login, admin navigation/routing, first-time setup, game
import) that bUnit cannot cover.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The DataTable component issues relational queries (AsSplitQuery, Include,
and a translated punctuation-stripping search expression) that the EF
InMemory provider cannot translate. On CI this surfaced as a native stack
overflow in CountAsync that crashed the in-process server and cascaded
into widespread timeouts. A file-based SQLite database supports these
queries and concurrent connections, stabilizing the test run.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Target net10.0 to match the now-net10 server project (and fix the
Playwright browser-install path in the PR workflow accordingly).
- Dismiss the persistent "Import Ready" notification (added with main's
background UploadTracker) before clicking Import; its Duration=0 toast
overlays and intercepts the modal's Import button.
- Update General settings assertions: "Database Provider" moved to the new
dedicated /Settings/Database page, so assert on "Use SSL" instead.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy OpenRCT2.lcx into TestData/ and reference via AppContext.BaseDirectory
so tests don't depend on a user-specific Downloads path.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Replace process-based server management (ServerManager) with ASP.NET Core's
WebApplicationFactory<Program> for proper integration testing:
- Add UITestApplicationFactory with dual-host pattern (real Kestrel + dummy TestServer)
to work around .NET 9's hard-cast to TestServer in WebApplicationFactory
- Use in-memory EF Core database with dynamic port binding (IPAddress.Loopback:0)
- Stub IVersionProvider and IGitHubService to avoid external dependencies
- Programmatically create admin user in ConfiguredServerFixture via service layer
- Remove ServerManager.cs (no longer needed)
- Skip CompleteWizardAndLogin test (requires real DB file I/O)
- 18 tests pass, 1 skipped, ~28s runtime (down from ~101s)
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
