Skip to content

Commit

Permalink
ci: merge e2e-cypress.yml & e2e-playwright.yml into e2e.yml
Browse files Browse the repository at this point in the history
  • Loading branch information
nsbarsukov committed Oct 3, 2024
1 parent 2393607 commit 39ff5c7
Show file tree
Hide file tree
Showing 4 changed files with 68 additions and 68 deletions.
60 changes: 0 additions & 60 deletions .github/workflows/e2e-cypress.yml

This file was deleted.

70 changes: 66 additions & 4 deletions .github/workflows/e2e-playwright.yml → .github/workflows/e2e.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,12 @@
name: ⚙️ Playwright E2E testing
name: ⚙️ E2E testing
on:
pull_request:

env:
CYPRESS_BASELINE_REPO: 'baseline'
CYPRESS_RESULTS_FOLDER: 'projects/demo-cypress/tests-results'
CYPRESS_BASELINE_SNAPSHOTS: 'projects/demo-cypress/tests-results/snapshots/baseline'

jobs:
build-demo:
if: ${{ !contains(github.head_ref , 'release/') }}
Expand All @@ -25,6 +30,53 @@ jobs:
path: dist/demo
key: ${{ env.CACHE_DIST_KEY }}

cypress:
runs-on: ubuntu-latest
name: Cypress / Component Testing
steps:
- uses: actions/[email protected]
- uses: actions/[email protected]
with:
ref: ${{ github.base_ref }}
path: ${{ env.CYPRESS_BASELINE_REPO }}
- uses: taiga-family/ci/actions/setup/[email protected]
- uses: taiga-family/ci/actions/setup/[email protected]

- name: Share node_modules between current and baseline repos
run: |
cp -r node_modules ${{ env.CYPRESS_BASELINE_REPO }}/node_modules
tree -d -L 1
tree -d ${{ env.CYPRESS_BASELINE_REPO }} -L 1
- name: Run tests for baseline state
run: |
cd ${{ env.CYPRESS_BASELINE_REPO }}
npx nx component-test demo-cypress
cd ../
mkdir -p ${{ env.CYPRESS_BASELINE_SNAPSHOTS }} && cp -r ${{ env.CYPRESS_BASELINE_REPO }}/${{env.CYPRESS_BASELINE_SNAPSHOTS }}/. ${{ env.CYPRESS_BASELINE_SNAPSHOTS }}
rm -r ${{ env.CYPRESS_BASELINE_REPO }}
- name: Run tests for current state
run: npx nx component-test demo-cypress

- name: Prepare diff screenshots
run: |
npm install canvas
npx ts-node ./scripts/visual-testing/combine-cypress-failed-screenshots.ts
- name: Debug output
continue-on-error: true
run: tree ${{ env.CYPRESS_RESULTS_FOLDER }} -L 1

- name: Upload artifacts
uses: actions/[email protected]
with:
path: '${{ env.CYPRESS_RESULTS_FOLDER }}/**/*.diff.png'
name: ${{ env.CYPRESS_SNAPSHOTS_ARTIFACTS_KEY }}
if-no-files-found: ignore
compression-level: 0
retention-days: 1

playwright:
if: ${{ !contains(github.head_ref , 'release/') }}
runs-on: ubuntu-latest
Expand All @@ -34,7 +86,7 @@ jobs:
matrix:
shardIndex: [1, 2, 3, 4, 5, 6, 7, 8, 9]
shardTotal: [9]
name: playwright / (${{ matrix.shardIndex }} of ${{ matrix.shardTotal }})
name: Playwright / (${{ matrix.shardIndex }} of ${{ matrix.shardTotal }})
steps:
- uses: actions/[email protected]
- uses: taiga-family/ci/actions/setup/[email protected]
Expand Down Expand Up @@ -74,6 +126,7 @@ jobs:
- name: Combine images to get diff reports
run: |
npm install canvas
npm view canvas version
npx ts-node ./scripts/visual-testing/combine-playwright-failed-screenshots.ts
- name: Debug output
Expand Down Expand Up @@ -112,13 +165,13 @@ jobs:
result:
if: ${{ !contains(github.head_ref , 'release/') }}
name: E2E result
needs: [playwright]
needs: [playwright, cypress]
runs-on: ubuntu-latest
steps:
- uses: actions/[email protected]
- uses: taiga-family/ci/actions/setup/[email protected]

- name: Download artifacts / ${{ env.PLAYWRIGHT_SNAPSHOTS_ARTIFACTS_KEY }}
- name: Download artifacts for Playwright
continue-on-error: true
uses: actions/[email protected]
with:
Expand All @@ -127,6 +180,15 @@ jobs:
merge-multiple: true
- run: ls -R ./total/playwright || echo "not found"

- name: Download artifacts for Cypress
continue-on-error: true
uses: actions/[email protected]
with:
path: ./total/cypress
pattern: ${{ env.CYPRESS_SNAPSHOTS_ARTIFACTS_KEY }}
merge-multiple: true
- run: ls -R ./total/cypress || echo "not found"

- name: Check if diff-output exists
id: diff_checker
run: |
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
class="fade"
>
<div tuiFade>Very long value in chip</div>
<div>{{ 123456 | tuiAmount: 'RUB' | async }}</div>
<div>{{ 123000 | tuiAmount: 'RUB' | async }}</div>
</tui-chip>

<tui-chip
Expand Down
4 changes: 1 addition & 3 deletions scripts/visual-testing/combine-snapshots.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,11 @@
* (not friendly for our external contributors).
* https://github.com/Automattic/node-canvas/issues/1511
*/
import {createCanvas, loadImage, version} from 'canvas';
import {createCanvas, loadImage} from 'canvas';

export async function combineSnapshots(
imagesPaths: string[],
): Promise<NodeJS.ArrayBufferView> {
console.info('canvas:', version);

const images = await Promise.all(imagesPaths.map(loadImage));
const totalWidth = images.reduce((acc: number, {width}) => acc + width, 0);
const maxHeight = Math.max(...images.map(({height}) => height));
Expand Down

0 comments on commit 39ff5c7

Please sign in to comment.