diff --git a/.github/workflows/cypress.yml b/.github/workflows/cypress.yml index 34d2b8206..6d0ef6099 100644 --- a/.github/workflows/cypress.yml +++ b/.github/workflows/cypress.yml @@ -25,11 +25,20 @@ env: BRANCH: ${{ github.base_ref || github.ref_name }} jobs: - init: + cypress: runs-on: ubuntu-latest - outputs: - nodeVersion: ${{ steps.versions.outputs.nodeVersion }} - npmVersion: ${{ steps.versions.outputs.npmVersion }} + + strategy: + fail-fast: false + matrix: + # Run multiple copies of the current job in parallel + # Please increase the number or runners as your tests suite grows (0 based index for e2e tests) + containers: [0, 1, 2, 3, 4, 5, 6, 7] + # Hack as strategy.job-total includes the component and GitHub does not allow math expressions + # Always align this number with the total of e2e runners (max. index + 1) + total-containers: [8] + + name: runner ${{ matrix.containers }} env: PUPPETEER_SKIP_DOWNLOAD: true @@ -58,8 +67,8 @@ jobs: uses: skjnldsv/read-package-engines-version-actions@06d6baf7d8f41934ab630e97d9e6c0bc9c9ac5e4 # v3 id: versions with: - fallbackNode: "^20" - fallbackNpm: "^10" + fallbackNode: '^20' + fallbackNpm: '^10' - name: Set up node ${{ steps.versions.outputs.nodeVersion }} uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # v4.0.2 @@ -71,56 +80,18 @@ jobs: - name: Install node dependencies & build app run: | - npm ci + # We'll install cypress in the cypress job + CYPRESS_INSTALL_BINARY=0 npm ci npm run build --if-present - - name: Save context - uses: buildjet/cache/save@e376f15c6ec6dc595375c78633174c7e5f92dc0e # v3 - with: - key: cypress-context-${{ github.run_id }} - path: ./ - - cypress: - runs-on: ubuntu-latest - needs: init - - strategy: - fail-fast: false - matrix: - # Run multiple copies of the current job in parallel - # Please increase the number or runners as your tests suite grows (0 based index for e2e tests) - containers: [0, 1, 2, 3, 4, 5, 6, 7] - # Hack as strategy.job-total includes the component and GitHub does not allow math expressions - # Always aling this number with the total of e2e runners (max. index + 1) - total-containers: [8] - - name: runner ${{ matrix.containers }} - - steps: - - name: Disabled on forks - if: ${{ github.event.pull_request.head.repo.full_name != github.repository }} - run: | - echo 'Can not approve PRs from forks' - exit 1 - - - name: Restore context - uses: buildjet/cache/restore@e376f15c6ec6dc595375c78633174c7e5f92dc0e # v3 - with: - fail-on-cache-miss: true - key: cypress-context-${{ github.run_id }} - path: ./ - - - name: Set up node ${{ needs.init.outputs.nodeVersion }} - uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # v4.0.2 - with: - node-version: ${{ needs.init.outputs.nodeVersion }} - - - name: Set up npm ${{ needs.init.outputs.npmVersion }} - run: npm i -g 'npm@${{ needs.init.outputs.npmVersion }}' + - name: Install cypress + run: ./node_modules/cypress/bin/cypress install - name: Run ${{ matrix.containers == 'component' && 'component' || 'E2E' }} cypress tests - uses: cypress-io/github-action@8d3918616d8ac34caa2b49afc8b408b6a872a6f5 # v6.7.1 + uses: cypress-io/github-action@496e7dc0edc421a9de8a36a31c793340e00c61bf # v6.7.5 with: + # We already installed the dependencies in the init job + install: false # cypress run type component: ${{ matrix.containers == 'component' }} # Do not add Cypress record key config as this conflicts with cypress-split @@ -144,7 +115,7 @@ jobs: - name: Extract NC logs if: failure() && matrix.containers != 'component' - run: docker logs nextcloud-cypress-tests-${{ env.APP_NAME }} > nextcloud.log + run: docker logs nextcloud-cypress-tests_${{ env.APP_NAME }} > nextcloud.log - name: Upload NC logs uses: actions/upload-artifact@0b2256b8c012f0828dc542b3febcab082c67f72b # v4.3.4 @@ -155,7 +126,7 @@ jobs: summary: runs-on: ubuntu-latest-low - needs: [init, cypress] + needs: [cypress] if: always() @@ -163,4 +134,4 @@ jobs: steps: - name: Summary status - run: if ${{ needs.init.result != 'success' || ( needs.cypress.result != 'success' && needs.cypress.result != 'skipped' ) }}; then exit 1; fi + run: if ${{ needs.cypress.result != 'success' && needs.cypress.result != 'skipped' }}; then exit 1; fi diff --git a/package-lock.json b/package-lock.json index ad6c9cfa5..f318b1f21 100644 --- a/package-lock.json +++ b/package-lock.json @@ -36,7 +36,7 @@ }, "devDependencies": { "@nextcloud/browserslist-config": "^3.0.1", - "@nextcloud/cypress": "^1.0.0-beta.8", + "@nextcloud/cypress": "^1.0.0-beta.9", "@nextcloud/eslint-config": "^8.4.1", "@nextcloud/sharing": "^0.2.3", "@nextcloud/stylelint-config": "^3.0.1", @@ -1681,42 +1681,21 @@ } }, "node_modules/@nextcloud/cypress": { - "version": "1.0.0-beta.8", - "resolved": "https://registry.npmjs.org/@nextcloud/cypress/-/cypress-1.0.0-beta.8.tgz", - "integrity": "sha512-VK3RyXj2HJISx0wUEs1yBBJ3fkDX5C1CAM92Px4Brf9IPz5YfCZAICvxjM8HIXPACR0yb5MUCtGz7E9hv1H/wA==", + "version": "1.0.0-beta.9", + "resolved": "https://registry.npmjs.org/@nextcloud/cypress/-/cypress-1.0.0-beta.9.tgz", + "integrity": "sha512-i43skZoCkrr5H1L1PsYddzz/SjDlSNcTCrEcMJFRwoHiJo5giUNCz0b2lKEQBMtudP8J+BGfG+AIFA0LJ/oD2A==", "dev": true, - "license": "AGPL-3.0-or-later", "dependencies": { "dockerode": "^4.0.2", "fast-xml-parser": "^4.3.6", - "wait-on": "^7.2.0" + "wait-on": "^8.0.0" }, "engines": { "node": "^20.0.0", "npm": "^10.0.0" }, "peerDependencies": { - "cypress": "^13.2.0" - } - }, - "node_modules/@nextcloud/cypress/node_modules/wait-on": { - "version": "7.2.0", - "resolved": "https://registry.npmjs.org/wait-on/-/wait-on-7.2.0.tgz", - "integrity": "sha512-wCQcHkRazgjG5XoAq9jbTMLpNIjoSlZslrJ2+N9MxDsGEv1HnFoVjOCexL0ESva7Y9cu350j+DWADdk54s4AFQ==", - "dev": true, - "license": "MIT", - "dependencies": { - "axios": "^1.6.1", - "joi": "^17.11.0", - "lodash": "^4.17.21", - "minimist": "^1.2.8", - "rxjs": "^7.8.1" - }, - "bin": { - "wait-on": "bin/wait-on" - }, - "engines": { - "node": ">=12.0.0" + "cypress": "^13.9.0" } }, "node_modules/@nextcloud/dialogs": { diff --git a/package.json b/package.json index 22c367e05..fb5df6bec 100644 --- a/package.json +++ b/package.json @@ -74,7 +74,7 @@ }, "devDependencies": { "@nextcloud/browserslist-config": "^3.0.1", - "@nextcloud/cypress": "^1.0.0-beta.8", + "@nextcloud/cypress": "^1.0.0-beta.9", "@nextcloud/eslint-config": "^8.4.1", "@nextcloud/sharing": "^0.2.3", "@nextcloud/stylelint-config": "^3.0.1",