From 37ad26a082851c02497c36e7fce40555b9480e11 Mon Sep 17 00:00:00 2001 From: undergroundwires Date: Sat, 9 Oct 2021 12:39:05 +0100 Subject: [PATCH] Remove integration tests from deployments #90 Integration tests may depend on third parties and can fail from time to time. In some situations failing tests can be acceptable to go forward with deployments. They should not be a requirement that blocks deployments. They may lead to unintended lack of distributed packages as seen in #90. --- .github/workflows/deploy-desktop.yaml | 6 ++---- .github/workflows/deploy-site.yaml | 6 ++---- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/.github/workflows/deploy-desktop.yaml b/.github/workflows/deploy-desktop.yaml index 3a849c4a..2d74b617 100644 --- a/.github/workflows/deploy-desktop.yaml +++ b/.github/workflows/deploy-desktop.yaml @@ -25,10 +25,8 @@ jobs: node-version: 15.x - name: Install dependencies run: npm ci - - name: Run tests - run: | - npm run test:unit - npm run test:integration + - name: Run unit tests + run: npm run test:unit - name: Publish desktop app run: npm run electron:build -- -p always # https://nklayman.github.io/vue-cli-plugin-electron-builder/guide/recipes.html#upload-release-to-github env: diff --git a/.github/workflows/deploy-site.yaml b/.github/workflows/deploy-site.yaml index 6f88fb31..18727358 100644 --- a/.github/workflows/deploy-site.yaml +++ b/.github/workflows/deploy-site.yaml @@ -89,10 +89,8 @@ jobs: run: npm ci working-directory: site - - name: "App: Run tests" - run: | - npm run test:unit - npm run test:integration + name: "App: Run unit tests" + run: npm run test:unit working-directory: site - name: "App: Build"