fix: pr review comments from EJ #1239
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: Run tests | |
| concurrency: | |
| group: ${{ github.workflow }}-${{ github.ref }} | |
| cancel-in-progress: true | |
| on: | |
| - push | |
| - workflow_call | |
| jobs: | |
| test-suites: | |
| runs-on: ubuntu-latest | |
| # Stop the occasional rogue instance before the 6h GitHub limit | |
| timeout-minutes: 15 | |
| permissions: | |
| contents: write | |
| steps: | |
| - name: Checkout | |
| uses: actions/checkout@v4 | |
| - name: Use Node.js 20 | |
| uses: actions/setup-node@v4 | |
| with: | |
| node-version: 20 | |
| - name: Install everything | |
| run: npm install | |
| #- name: Always test with the latest browserslist db | |
| # run: | | |
| # npx update-browserslist-db@latest | |
| #- name: Commit browserslist db changes on dev branches | |
| # if: ${{ github.ref_type == 'branch' && ! github.ref_protected }} | |
| # run: | | |
| # if git diff --exit-code package-lock.json; then | |
| # echo "The browserslist db is up to date." | |
| # else | |
| # echo "The browserslist db is out of date." | |
| # git config --global user.email "github-actions[bot]@users.noreply.github.com" | |
| # git config --global user.name "github-actions[bot]" | |
| # git add package-lock.json | |
| # git commit -m "chore: update browserslist db" | |
| # git push | |
| # fi | |
| - name: Ng test for studio-web | |
| run: | | |
| npx nx build web-component | |
| npx nx test:once studio-web | |
| - name: Cypress run for web-component | |
| uses: cypress-io/github-action@v6 | |
| with: | |
| install: false | |
| start: | | |
| npx nx serve web-component | |
| npx nx serve-test-data web-component | |
| wait-on: sleep 15 # there is no reliable URL to wait for... | |
| command: npx nx test:once web-component | |
| - name: Check that i18n and l10n are up to date | |
| run: | | |
| npx nx extract-i18n studio-web | |
| if diff -w <(git show HEAD:packages/studio-web/src/i18n/messages.json | sort) <(sort < packages/studio-web/src/i18n/messages.json); then echo OK: The i18n database is up to date.; else echo ERROR: The i18n database is out of date.; npx nx check-l10n studio-web || echo ERROR: The l10n databases are also out of date.; false; fi | |
| if npx nx check-l10n studio-web; then echo OK: The l10n databases are up to date.; else echo ERROR: An l10n database is out of date.; false; fi | |
| - name: make sure the WP plugin zip file is in sync | |
| run: | | |
| cd packages/web-component/wordpress-plugin | |
| unzip -d extract read-along-web-app-loader.zip | |
| if ! diff -qr read-along-web-app-loader extract/read-along-web-app-loader; then \ | |
| echo The zipped WordPress plugin packages/web-component/wordpress-plugin/read-along-web-app-loader.zip is out of date.; \ | |
| false; \ | |
| fi | |
| - name: make sure bundling works | |
| shell: bash | |
| run: | | |
| npx nx bundle web-component | |
| git status | |
| git diff --word-diff=porcelain --word-diff-regex=... --color | perl -ple 's/^(\x1b[^ -+]{0,6})? (.{81,})$/$1 . " " . substr($2, 0, 40) . " [... " . (length($2)-80) . " bytes ...] " . substr($2, -40)/ex' | |
| playwright-tests: | |
| name: Run Playwright test-suites | |
| timeout-minutes: 60 | |
| runs-on: ubuntu-latest | |
| strategy: | |
| fail-fast: false | |
| matrix: | |
| shardIndex: [1, 2, 3, 4] | |
| shardTotal: [4] | |
| steps: | |
| - uses: actions/checkout@v4 | |
| - uses: actions/setup-node@v4 | |
| with: | |
| node-version: lts/* | |
| - name: Install and run the back-end API, needed for end-to-end testing | |
| run: | | |
| git clone https://github.com/ReadAlongs/Studio | |
| cd Studio | |
| pip install -e . -r requirements.api.txt | |
| ./run-web-api.sh & | |
| # wait for the API to be up | |
| curl --retry 20 --retry-delay 1 --retry-all-errors http://localhost:8000/api/v1/langs | |
| - name: Install everything | |
| run: npm install | |
| - name: Install dependencies | |
| run: npm ci | |
| - name: Run studio-web in the background | |
| run: | | |
| npx nx build web-component | |
| npx nx run-many --targets=serve,serve-fr,serve-es --projects=web-component,studio-web --parallel 6 & | |
| # wait for the studio web to be up | |
| sleep 100 | |
| curl --retry 20 --retry-delay 30 --retry-all-errors http://localhost:4200 | |
| - name: Run Playwright tests for studio-web | |
| run: | | |
| npx playwright install --with-deps chromium | |
| npx nx e2e studio-web --shard=${{ matrix.shardIndex }}/${{ matrix.shardTotal }} | |
| - name: Upload blob report to GitHub Actions Artifacts | |
| if: ${{ !cancelled() }} | |
| uses: actions/upload-artifact@v4 | |
| with: | |
| name: blob-report-${{ matrix.shardIndex }} | |
| path: packages/studio-web/blob-report | |
| retention-days: 1 | |
| merge-reports: | |
| # Merge reports after playwright-tests, even if some shards have failed | |
| if: ${{ !cancelled() }} | |
| needs: [playwright-tests] | |
| name: "Merge playwright reports" | |
| runs-on: ubuntu-latest | |
| steps: | |
| - uses: actions/checkout@v4 | |
| - uses: actions/setup-node@v4 | |
| with: | |
| node-version: lts/* | |
| - name: Install everything | |
| run: npm install | |
| - name: Install dependencies | |
| run: npm ci | |
| - name: Install playwright | |
| run: npx playwright install | |
| - name: Download blob reports from GitHub Actions Artifacts | |
| uses: actions/download-artifact@v4 | |
| with: | |
| path: all-blob-reports | |
| pattern: blob-report-* | |
| merge-multiple: true | |
| - name: Merge into HTML Report | |
| run: npx playwright merge-reports --reporter=html,github ./all-blob-reports | |
| - name: Upload HTML report | |
| uses: actions/upload-artifact@v4 | |
| with: | |
| name: html-report--attempt-${{ github.run_attempt }} | |
| path: playwright-report | |
| retention-days: 5 |