[DON'T MERGE - POC] IBX-11740: Added playwright-browser-tests.yml#100
[DON'T MERGE - POC] IBX-11740: Added playwright-browser-tests.yml#100pawlakadrian wants to merge 22 commits into
Conversation
pawlakadrian
changed the title
|
| type: string | ||
| default: "main" | ||
| cohesivo-version: | ||
| description: "Composer version constraint for ibexa/cohesivo-playwright. Empty = same as the project version (branch alias), so 4.6/5.0/6.0 branches get the matching library version." |
There was a problem hiding this comment.
PW wont be available on 4.6
| required: false | ||
|
|
||
| env: | ||
| APP_ENV: behat |
There was a problem hiding this comment.
Does env 'behat' do anything for playwright setup?
| name: Run first phase of tests setup | ||
| run: | | ||
| cd ${HOME}/build/project | ||
| docker compose --env-file=.env exec -T --user www-data app sh -c "vendor/bin/ibexabehat ${{ inputs.test-setup-phase-1 }}" |
There was a problem hiding this comment.
'vendor/bin/ibexabehat' is this step necessary for PW?
| name: Run second phase of tests setup | ||
| run: | | ||
| cd ${HOME}/build/project | ||
| docker compose --env-file=.env exec -T --user www-data app sh -c "vendor/bin/ibexabehat ${{ inputs.test-setup-phase-2 }}" |
There was a problem hiding this comment.
'vendor/bin/ibexabehat' is this step necessary for PW?
| /home/runner/build/project/vendor/ibexa/**/playwright-tests/playwright-report/ | ||
| /home/runner/build/project/vendor/ibexa/**/playwright-tests/test-results/ | ||
| /home/runner/build/project/vendor/ibexa/cohesivo-playwright/playwright-report/ | ||
| /home/runner/build/project/vendor/ibexa/cohesivo-playwright/test-results/ |
There was a problem hiding this comment.
Is it necessary to upload report to that many paths?
| required: false | ||
| type: string | ||
| default: "http://localhost:8080" | ||
| test-setup-phase-1: |
There was a problem hiding this comment.
Do PW tests require behat setup to run first? Behat will be discontinued in future versions, so we cant rely on it.
| if: always() | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: playwright-report-${{ inputs.project-edition }}-${{ matrix.offset }} |
There was a problem hiding this comment.
Ive got a report with a name: 'playwright-report-commerce-0'. Report name should be somewhat distinguishable (otherwise it will be painful to use) - .ie with a date/time of creating the regression, or branch name 🤔 (it is up for discussion i think)
2 participants
Related PRs:
Description:
Adds playwright-browser-tests.yml — a reusable workflow for running Playwright tests across Ibexa packages.
For QA:
Documentation: