Skip to content

IBX-11740: Added playwright-browser-tests.yml - #100

Open
pawlakadrian wants to merge 25 commits into
mainfrom
ibx-11740-playwright
Open

IBX-11740: Added playwright-browser-tests.yml#100
pawlakadrian wants to merge 25 commits into
mainfrom
ibx-11740-playwright

Conversation

@pawlakadrian

@pawlakadrian pawlakadrian commented May 27, 2026

Copy link
Copy Markdown
🎫 Issue IBX-11740

Related PRs:

Description:

Adds playwright-browser-tests.yml — a reusable workflow for running Playwright tests across Ibexa packages.

  • Reuses prepare_project_edition.sh from ibexa/ci-scripts (same script browser-tests.yml uses) to set up the app for the given edition
  • Installs cohesivo-playwright from vendor, builds it, then runs npm run test:all
  • APP_EDITION is set from project-edition input — only tests tagged for the given edition are executed
  • No hardcoded credentials — uses secrets.ADMIN_LOGIN/ADMIN_PASSWORD with standard test defaults as fallback
  • Artifacts collected per edition from all playwright-tests/ directories in vendor
  • Slack notifications on failure always, and on success when send-success-notification is enabled.

For QA:

Documentation:

@pawlakadrian
pawlakadrian marked this pull request as draft May 27, 2026 10:57
@pawlakadrian pawlakadrian changed the title IBX-11740: Added playwright-browser-tests.yml [DON'T MERGE - POC] IBX-11740: Added playwright-browser-tests.yml Jun 24, 2026
Comment thread .github/workflows/playwright-browser-tests.yml Outdated
Comment thread .github/workflows/playwright-browser-tests.yml
Comment thread .github/workflows/playwright-browser-tests.yml Outdated
Comment thread .github/workflows/playwright-browser-tests.yml Outdated
Comment thread .github/workflows/playwright-browser-tests.yml Outdated
Comment thread .github/workflows/playwright-browser-tests.yml Outdated
Comment thread .github/workflows/playwright-browser-tests.yml Outdated
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@pawlakadrian
pawlakadrian marked this pull request as ready for review August 4, 2026 11:32
@pawlakadrian pawlakadrian changed the title [DON'T MERGE - POC] IBX-11740: Added playwright-browser-tests.yml IBX-11740: Added playwright-browser-tests.yml Sep 4, 2026
@pawlakadrian
pawlakadrian requested a review from a team September 9, 2026 11:43
Comment thread .github/workflows/playwright-browser-tests.yml Outdated
Comment thread .github/workflows/playwright-browser-tests.yml Outdated
Comment thread .github/workflows/playwright-browser-tests.yml Outdated
Comment thread .github/workflows/playwright-browser-tests.yml
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

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.

4 participants