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>
- 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>
Replace non-waiting Assert.True(await locator.IsVisibleAsync()) checks with
auto-retrying Assertions.Expect(locator).ToBeVisibleAsync()/ToBeHiddenAsync()
to remove flakiness against Blazor's async rendering. Table-row/field page
objects (Metadata/Roles/Users/Profile) now expose ILocator helpers instead of
Task<bool>, so deletion checks wait for the element to disappear.
Also: rename ScreenshotHelper.CaptureIfFailedAsync to CaptureAsync (xUnit v2
cannot expose the test outcome to DisposeAsync, so it always captures the final
page state), drop the unused TestConstants.ServerPort/BaseUrl now that ports
bind dynamically, and fix IClassFixture doc comments to ICollectionFixture.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add ScreenshotHelper that captures full-page screenshots for every test
- Screenshots are named with the full test display name for easy correlation
- Save to SCREENSHOT_DIR env var (CI) or bin/Screenshots/ (local)
- Add ITestOutputHelper to all 10 test classes for test name extraction
- Screenshots are included in the existing ui-test-results artifact upload
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- GameEditTests now use a game seeded via the service layer in
ConfiguredServerFixture instead of importing via the UI.
This eliminates the Blazor InputFile compatibility issue on Linux CI.
- GamesPage.ImportGameAsync uses SetInputFilesAsync with retry and
manual change event dispatch for Blazor InputFile compatibility.
- Add dorny/test-reporter to display test results in PR summary.
- Add checks/pull-requests write permissions for test-reporter.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Refactor FirstTimeSetupTests to use FreshServerFixture via ICollectionFixture
instead of creating its own UITestApplicationFactory per test
- Skip _realHost.Dispose() in UITestApplicationFactory since the DI container's
deep dependency chain causes an uncatchable native StackOverflowException
- All 62 tests pass with exit code 0 (61 pass, 1 skip)
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.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>
- Switch all test classes from IClassFixture<ConfiguredServerFixture> to
[Collection("Server")] with ICollectionFixture so a single server instance
is shared across all test classes (avoids static DatabaseContext.Provider conflicts)
- Put FirstTimeSetupTests in its own collection since it needs an unconfigured server
- Remove Hangfire hosted services in UITestApplicationFactory to prevent
stack overflow during process shutdown (deep DI disposal chain)
- Add graceful shutdown handling with timeout in DisposeAsync
- All 54 tests pass, 1 skipped, clean exit code 0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Create UsersPage page object with navigate, search, user visibility,
roles check, and delete methods
- Add UserManagementTests with 5 tests: admin user visibility, search,
roles display, new user registration, and user deletion
- Fix GamesPage.cs syntax error (unchecked is a C# keyword)
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Create SettingsPage page object with navigation helpers for all settings
sub-pages and SettingsTests with 9 tests covering General form elements,
Users list, Roles list, Authentication, Archives, Media, Beacon, Updates,
and Appearance settings pages.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add Playwright-based UI tests for the Roles management page including:
- RolesPage page object with navigation, add, delete, and visibility methods
- Tests verifying Administrator role visibility
- Tests for adding and deleting custom roles
- Test confirming Administrator role cannot be deleted
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>
