Skip to content

Run playwright tests via compose setup, locally and in CI#8247

Open
pabzm wants to merge 7 commits into
mainfrom
playwright-compose
Open

Run playwright tests via compose setup, locally and in CI#8247
pabzm wants to merge 7 commits into
mainfrom
playwright-compose

Conversation

@pabzm

@pabzm pabzm commented Jun 3, 2026

Copy link
Copy Markdown
Member

The playwright tests now are running via a docker compose setup, which can be executed via ./php/tests/run.sh locally, and also gets called from the github workflow.

  • The PR was tested and verified that it works locally
  • The PR was completely or partially created with AI

@szaimen szaimen added 2. developing Work in progress enhancement New feature or request labels Jun 8, 2026
@szaimen szaimen added this to the next milestone Jun 8, 2026
pabzm added 6 commits June 16, 2026 09:01
Signed-off-by: Pablo Zmdl <pablo@nextcloud.com>
They now are running via a docker compose setup, which can be executed via
./php/tests/run.sh locally, and also gets called from the github workflow.

Signed-off-by: Pablo Zmdl <pablo@nextcloud.com>
Signed-off-by: Pablo Zmdl <pablo@nextcloud.com>
Signed-off-by: Pablo Zmdl <pablo@nextcloud.com>
Signed-off-by: Pablo Zmdl <pablo@nextcloud.com>
Signed-off-by: Pablo Zmdl <pablo@nextcloud.com>
@pabzm pabzm force-pushed the playwright-compose branch from a0cfaae to 07bd5d5 Compare June 16, 2026 07:08
@pabzm

pabzm commented Jun 16, 2026

Copy link
Copy Markdown
Member Author

Rebased to resolve conflicts

@pabzm pabzm requested a review from szaimen June 16, 2026 07:08
@szaimen

szaimen commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Playwright seems to fail unfortunately :/

Signed-off-by: Pablo Zmdl <pablo@nextcloud.com>
@pabzm

pabzm commented Jun 16, 2026

Copy link
Copy Markdown
Member Author

@szaimen It works now, please have another look

@szaimen szaimen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Jun 17, 2026
@szaimen

szaimen commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Thanks a lot for the PR @pabzm :)

However I would postpone my review of this PR until we merged #7995 as this will conflict with this PR here and I would like to merge the desec PR as son as possible. Hope that is fine for you? :)

@pabzm

pabzm commented Jun 22, 2026

Copy link
Copy Markdown
Member Author

I'd be happy to not postpone it too long, since I already had to resolve conflicts twice. but for desec it's ok.

@szaimen

szaimen commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

I'd be happy to not postpone it too long, since I already had to resolve conflicts twice. but for desec it's ok.

All right! Will try to review it after merging desec. Thanks for all your effort!

@szaimen

szaimen commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

This conflicts now after #7995 unfortunately but I can finally review it after another rebase :)

@szaimen szaimen added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing Work in progress enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants