diff --git a/.github/workflows/cypress.yml b/.github/workflows/cypress.yml index 6bd9a2eb7..af110bec1 100644 --- a/.github/workflows/cypress.yml +++ b/.github/workflows/cypress.yml @@ -4,12 +4,19 @@ on: pull_request: push: branches: + - main - master - stable* env: - APP_NAME: viewer - BRANCH: ${{ github.base_ref }} + # Adjust APP_NAME if your repository name is different + APP_NAME: ${{ github.event.repository.name }} + + # This represents the server branch to checkout. + # Usually it's the base branch of the PR, but for pushes it's the branch itself. + # e.g. 'main', 'stable27' or 'feature/my-feature + # n.b. server will use head_ref, as we want to test the PR branch. + BRANCH: ${{ github.base_ref || github.ref_name }} jobs: init: @@ -20,10 +27,20 @@ jobs: steps: - name: Checkout app - uses: actions/checkout@c85c95e3d7251135ab7dc9ce3241c5835cc595a9 # v3.5.3 + uses: actions/checkout@8e5e7e5ab8b370d6c329ec480221332ada57f0ab # v3.5.2 + + - name: Check composer.json + id: check_composer + uses: andstor/file-existence-action@20b4d2e596410855db8f9ca21e96fbe18e12930b # v2 + with: + files: "composer.json" + + - name: Install composer dependencies + if: steps.check_composer.outputs.files_exists == 'true' + run: composer install --no-dev - name: Read package.json node and npm engines version - uses: skjnldsv/read-package-engines-version-actions@8205673bab74a63eb9b8093402fd9e0e018663a1 # v2.2 + uses: skjnldsv/read-package-engines-version-actions@0ce2ed60f6df073a62a77c0a4958dd0fc68e32e7 # v2.1 id: versions with: fallbackNode: "^14" @@ -32,19 +49,18 @@ jobs: - name: Set up node ${{ steps.versions.outputs.nodeVersion }} uses: actions/setup-node@64ed1c7eab4cce3362f8c340dee64e5eaeef8f7c # v3.6.0 with: - cache: 'npm' node-version: ${{ steps.versions.outputs.nodeVersion }} - name: Set up npm ${{ steps.versions.outputs.npmVersion }} run: npm i -g npm@"${{ steps.versions.outputs.npmVersion }}" - - name: Install dependencies & build app + - name: Install node dependencies & build app run: | npm ci TESTING=true npm run build --if-present - name: Save context - uses: actions/cache@88522ab9f39a2ea568f7027eddc7d8d8bc9d59c8 # v3.3.1 + uses: buildjet/cache/save@e376f15c6ec6dc595375c78633174c7e5f92dc0e # v3 with: key: cypress-context-${{ github.run_id }} path: ./ @@ -56,14 +72,15 @@ jobs: strategy: fail-fast: false matrix: - # run multiple copies of the current job in parallel + # Run multiple copies of the current job in parallel + # Please increase the number or runners as your tests suite grows containers: [1, 2, 3, 4, 5, 6, 7, 8, 9, 10] name: runner ${{ matrix.containers }} steps: - name: Restore context - uses: actions/cache@88522ab9f39a2ea568f7027eddc7d8d8bc9d59c8 # v3.3.1 + uses: buildjet/cache/restore@e376f15c6ec6dc595375c78633174c7e5f92dc0e # v3 with: fail-on-cache-miss: true key: cypress-context-${{ github.run_id }} @@ -72,17 +89,19 @@ jobs: - name: Set up node ${{ needs.init.outputs.nodeVersion }} uses: actions/setup-node@64ed1c7eab4cce3362f8c340dee64e5eaeef8f7c # v3.6.0 with: - cache: 'npm' node-version: ${{ needs.init.outputs.nodeVersion }} - name: Set up npm ${{ needs.init.outputs.npmVersion }} run: npm i -g npm@"${{ needs.init.outputs.npmVersion }}" - - name: Run E2E cypress tests + - name: Run ${{ matrix.containers == 'component' && 'component' || 'E2E' }} cypress tests uses: cypress-io/github-action@db1693016f23ccf9043f4b2428f9b04e5d502a73 # v5.8.1 with: record: true parallel: true + # cypress run type + component: ${{ matrix.containers == 'component' }} + group: Run ${{ matrix.containers == 'component' && 'component' || 'E2E' }} # cypress env ci-build-id: ${{ github.sha }}-${{ github.run_number }} tag: ${{ github.event_name }} @@ -100,9 +119,20 @@ jobs: uses: actions/upload-artifact@0b7f8abb1508181956e8e162db84b466c27e18ce # v3.1.2 if: always() with: - name: snapshots + name: snapshots_${{ matrix.containers }} path: cypress/snapshots + - name: Extract NC logs + if: failure() && matrix.containers != 'component' + run: docker logs nextcloud-cypress-tests-${{ env.APP_NAME }} > nextcloud.log + + - name: Upload NC logs + uses: actions/upload-artifact@0b7f8abb1508181956e8e162db84b466c27e18ce # v3.1.2 + if: failure() && matrix.containers != 'component' + with: + name: nc_logs_${{ matrix.containers }} + path: nextcloud.log + summary: runs-on: ubuntu-latest needs: [init, cypress] diff --git a/cypress.config.ts b/cypress.config.ts index 4fd7a320a..8ac4efa77 100644 --- a/cypress.config.ts +++ b/cypress.config.ts @@ -22,8 +22,8 @@ export default defineConfig({ // Needed to trigger `after:run` events with cypress open experimentalInteractiveRunEvents: true, - // faster video processing - videoCompression: false, + // Faster processing, video is broken on GH actions anyway + video: false, // Visual regression testing env: { @@ -65,7 +65,9 @@ export default defineConfig({ // Remove container after run on('after:run', () => { - stopNextcloud() + if (!process.env.CI) { + stopNextcloud() + } }) // Before the browser launches diff --git a/cypress/dockerNode.ts b/cypress/dockerNode.ts index af1d4b428..de2a02819 100644 --- a/cypress/dockerNode.ts +++ b/cypress/dockerNode.ts @@ -156,7 +156,7 @@ export const stopNextcloud = async function() { * @param container the container to get the ip from */ export const getContainerIP = async function( - container: Container = docker.getContainer(CONTAINER_NAME) + container: Container = docker.getContainer(CONTAINER_NAME), ): Promise { let ip = '' let tries = 0 @@ -195,7 +195,7 @@ export const waitOnNextcloud = async function(ip: string) { const runExec = async function( container: Docker.Container, command: string[], - verbose = false + verbose = false, ) { const exec = await container.exec({ Cmd: command,