Skip to content

test(playwright): fix local test server config for SQLite runs - #61218

Merged
CarlSchwan merged 3 commits into
masterfrom
fix/playwright-local-test-server-config
Jun 12, 2026
Merged

test(playwright): fix local test server config for SQLite runs#61218
CarlSchwan merged 3 commits into
masterfrom
fix/playwright-local-test-server-config

Conversation

@pringelmann

@pringelmann pringelmann commented Jun 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

Two server-config fixes in so the Playwright suite runs reliably against the bundled SQLite container.

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Tests (unit, integration, api and/or acceptance) are included
  • Screenshots before/after for front-end changes
  • Documentation (manuals or wiki) has been updated or is not required
  • Backports requested where applicable (ex: critical bugfixes)
  • Labels added where applicable (ex: bug/enhancement, 3. to review, feature component)
  • Milestone added for target branch/version (ex: 32.x for stable32)

@pringelmann pringelmann added this to the Nextcloud 35 milestone Jun 11, 2026
@pringelmann pringelmann self-assigned this Jun 11, 2026
@pringelmann
pringelmann requested a review from a team as a code owner June 11, 2026 18:51
@pringelmann
pringelmann requested review from nfebe, sorbaugh and susnux and removed request for a team June 11, 2026 18:51
@pringelmann pringelmann added 3. to review Waiting for reviews tests Related to tests CI labels Jun 11, 2026
susnux
susnux reviewed Jun 11, 2026
View reviewed changes
Comment thread tests/playwright/start-nextcloud-server.js Outdated
susnux
susnux reviewed Jun 11, 2026
View reviewed changes
Comment thread tests/playwright/start-nextcloud-server.js Outdated
@pringelmann
pringelmann force-pushed the fix/playwright-local-test-server-config branch from 4de851f to 304e531 Compare June 11, 2026 21:07
@pringelmann
test(playwright): fix local test server config for SQLite runs
b0e823f 
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
@pringelmann
fix: remove uneccessary dbconfig change
5137f30 
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
@pringelmann
fix: remove sql line
a51d520 
Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
@pringelmann
pringelmann force-pushed the fix/playwright-local-test-server-config branch from bd59f44 to a51d520 Compare June 11, 2026 21:37
@susnux
susnux requested review from CarlSchwan and artonge June 11, 2026 23:10
@CarlSchwan
CarlSchwan merged commit d71abde into master Jun 12, 2026
239 of 245 checks passed
@CarlSchwan
CarlSchwan deleted the fix/playwright-local-test-server-config branch June 12, 2026 09:49
@nextcloud-bot nextcloud-bot mentioned this pull request Aug 11, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@susnux susnux susnux approved these changes
@CarlSchwan CarlSchwan CarlSchwan approved these changes
@nfebe nfebe Awaiting requested review from nfebe nfebe is a code owner automatically assigned from nextcloud/server-frontend
@sorbaugh sorbaugh Awaiting requested review from sorbaugh sorbaugh is a code owner automatically assigned from nextcloud/server-frontend
@artonge artonge Awaiting requested review from artonge

Assignees

@pringelmann pringelmann

Labels

3. to review Waiting for reviews CI tests Related to tests

Projects

None yet

Milestone

Nextcloud 35

Development

Successfully merging this pull request may close these issues.

3 participants

@pringelmann @susnux @CarlSchwan