Skip to content

Fix Frontend testing documentation drift - #189

Open
Ekhorn wants to merge 3 commits into
mainfrom
running-all-frontend-tests
Open

Fix Frontend testing documentation drift#189
Ekhorn wants to merge 3 commits into
mainfrom
running-all-frontend-tests

Conversation

@Ekhorn

@Ekhorn Ekhorn commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Update the frontend testing guide

The guide had drifted from ui/ since it was last refreshed, and still documented the npmTestUI Gradle task removed in openremote/openremote#3230.

Changelog

  • Rewrite "Running the test" around npmTest and -p scoping, drop the removed npmTestUI task, and document -Pargs for sharding app tests
  • Split the Playwright config step into the app and component factories (defineAppConfig / defineCtConfig), replacing the stale default import
  • Add the app variant of the npmTest Gradle task, which installs only the test workspaces and forwards Playwright arguments
  • Correct ui/test/app.config.ts to ui/test/app.config.cts
  • Describe the Shared (apps) and CtShared (components) fixture split
  • Use fixtures that exist in the app test boilerplate
  • Drop the no-op worker: 1 from the setup/teardown snippet
  • Add newlines around admonition contents

The worker change is tracked in openremote/openremote#3251 ui/fix-incorrect-worker-setting, so this should land after that fix.

@Ekhorn Ekhorn self-assigned this Aug 27, 2026
@Ekhorn
Ekhorn requested a review from wborn August 27, 2026 08:48
Comment thread docs/developer-guide/160-frontend-testing.md Outdated
@Ekhorn

Ekhorn commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

Decided that we'll get rid of the task entirely. Will update the PR once openremote/openremote#3230 is ready.

@Ekhorn
Ekhorn force-pushed the running-all-frontend-tests branch from 3e15e50 to 2603a23 Compare August 31, 2026 14:21
@Ekhorn Ekhorn changed the title Mention how to run all UI tests Fix Frontend testing documentation drift Aug 31, 2026
@Ekhorn
Ekhorn marked this pull request as ready for review August 31, 2026 15:05
@Ekhorn
Ekhorn requested a review from wborn August 31, 2026 15:07
@wborn
wborn requested a review from MartinaeyNL August 31, 2026 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants