Skip to content

[stable34] test(playwright): don't set webServer.url parameter - #8976

Merged
mejo- merged 1 commit into
stable34from
backport/8968/stable34
Aug 5, 2026
Merged

[stable34] test(playwright): don't set webServer.url parameter#8976
mejo- merged 1 commit into
stable34from
backport/8968/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Aug 5, 2026

Copy link
Copy Markdown

Backport of #8968

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot
backportbot Bot requested review from mejo- and silverkszlo August 5, 2026 08:22
@backportbot backportbot Bot added tests If you write them we ♥ you 3. to review labels Aug 5, 2026
@backportbot backportbot Bot added this to the Nextcloud 34.0.1 milestone Aug 5, 2026
We want Playwright to wait for the `webServer.wait.stdout` condition. If
`webServer.url` is set, it takes precedence.

Set `reuseExistingServer: false` and let `start-nextcloud-server.mjs`
detect if the webserver is already running.

With these changes, Playwright waits for Nextcloud to be ready before
running the tests, but at the same time can be started several times in
a row with a running Docker container without running into conflicts.

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: OpenCode:claude-fable-5
@mejo-
mejo- force-pushed the backport/8968/stable34 branch from 200a5da to 14e6bf9 Compare August 5, 2026 08:24
@mejo-
mejo- marked this pull request as ready for review August 5, 2026 08:25
@mejo-
mejo- requested a review from max-nextcloud as a code owner August 5, 2026 08:25
@mejo-
mejo- enabled auto-merge August 5, 2026 08:25
@mejo-
mejo- merged commit f80a8fd into stable34 Aug 5, 2026
66 checks passed
@mejo-
mejo- deleted the backport/8968/stable34 branch August 5, 2026 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review tests If you write them we ♥ you

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant