Skip to content

Mention how to run all UI tests - #189

Draft
Ekhorn wants to merge 1 commit into
mainfrom
running-all-frontend-tests
Draft

Mention how to run all UI tests#189
Ekhorn wants to merge 1 commit into
mainfrom
running-all-frontend-tests

Conversation

@Ekhorn

@Ekhorn Ekhorn commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@Ekhorn Ekhorn self-assigned this Aug 27, 2026
@Ekhorn
Ekhorn requested a review from wborn August 27, 2026 08:48
Comment on lines +219 to +221
::::note
Avoid using the `npmTestUI` task without specifying a path, or you'll open a Playwright window for each project.
::::

@wborn wborn Aug 27, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://docusaurus.io/docs/markdown-features/admonitions

Suggested change
::::note
Avoid using the `npmTestUI` task without specifying a path, or you'll open a Playwright window for each project.
::::
:::note
Avoid using the `npmTestUI` task without specifying a path, or you'll open a Playwright window for each project.
:::

@wborn wborn Aug 27, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extra new lines prevent issues if we start using prettier as Markdown formatter:

https://docusaurus.io/docs/markdown-features/admonitions#usage-with-prettier

@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.

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