Merge pull request #2153 from keboola/hotfix-release-pipeline-skip-pa…

…rticular-tests

hotfix: Pipeline does not test all code, just that one that it is expected to be released

.github/workflows/release-cli.yml

@@ -57,6 +57,7 @@ jobs:
     uses: ./.github/workflows/test-unit.yml
     with:
       without-cache: true
+      package-exception-regex: "./internal/pkg/service/stream|./internal/pkg/service/templates|./internal/pkg/service/appsproxy"
 
   test-e2e-cli:
     name: "E2E: CLI"

.github/workflows/release-service-apps-proxy.yml

@@ -35,6 +35,7 @@ jobs:
     uses: ./.github/workflows/test-unit.yml
     with:
       without-cache: true
+      package-exception-regex: "./internal/pkg/service/stream|./internal/pkg/service/templates|./internal/pkg/service/cli"
   test-k8s-service-apps-proxy:
     name: "K8S: Apps Proxy"
     secrets: inherit

.github/workflows/release-service-stream.yml

@@ -33,6 +33,9 @@ jobs:
     name: "Unit Tests"
     secrets: inherit
     uses: ./.github/workflows/test-unit.yml
+    with:
+      without-cache: true
+      package-exception-regex: "./internal/pkg/service/appsproxy|./internal/pkg/service/templates|./internal/pkg/service/cli"
   test-e2e-service-stream:
     name: "E2E: Stream"
     secrets: inherit

.github/workflows/release-service-templates.yml

@@ -35,6 +35,7 @@ jobs:
     uses: ./.github/workflows/test-unit.yml
     with:
       without-cache: true
+      package-exception-regex: "./internal/pkg/service/appsproxy|./internal/pkg/service/stream|./internal/pkg/service/cli"
   test-e2e-service-templates:
     name: "E2E: Templates"
     secrets: inherit

.github/workflows/test-unit.yml

@@ -7,6 +7,10 @@ on:
         type: boolean
         default: false
         required: false
+      package-exception-regex:
+        type: string
+        default: ""
+        required: false
 
 env:
   TEST_PARALLELISM: 12
@@ -90,7 +94,12 @@ jobs:
           if [ "$RUNNER_OS" == "macOS" ]; then
             export GOFLAGS="-ldflags=-extldflags=-Wl,-ld_classic"
           fi
-          
+
+          # Prevent testing unnecessary code for particular release
+          if [ -n "${{ inputs.package-exception-regex }}" ]; then
+            export TEST_EXCEPT="go list ./internal/pkg/... | grep -vE '${{ inputs.package-exception-regex }}'"
+          fi
+
           # ENVs can affect test caching, make sure to only run tests with expected ENVs
           env -i  \
             TERM="$TERM" \
@@ -111,6 +120,7 @@ jobs:
             TEST_KBC_PROJECTS_LOCK_HOST="$TEST_KBC_PROJECTS_LOCK_HOST" \
             TEST_KBC_PROJECTS_LOCK_PASSWORD="$TEST_KBC_PROJECTS_LOCK_PASSWORD" \
             TEST_KBC_PROJECTS_FILE="$TEST_KBC_PROJECTS_FILE" \
+            TEST_EXCEPT="$TEST_EXCEPT" \
           make tests-unit
         env:
           TEST_KBC_PROJECTS_LOCK_HOST: ${{ vars.TEST_KBC_PROJECTS_LOCK_HOST }}

scripts/tests.sh

@@ -18,6 +18,7 @@ TEST_LOG_FORMAT="${TEST_LOG_FORMAT:=testname}"
 TEST_DETECT_RACE="${TEST_DETECT_RACE:=true}"
 TEST_COVERAGE="${TEST_COVERAGE:=true}"
 TEST_PACKAGE="${TEST_PACKAGE:=./...}"
+TEST_EXCEPT="${TEST_EXCEPT:=}"
 TEST_ARGS="${TEST_ARGS:=}"
 if [[ $TEST_VERBOSE == "true" ]]; then
   TEST_ARGS="$TEST_ARGS -v"
@@ -28,6 +29,9 @@ fi
 if [[ $TEST_COVERAGE == "true" ]]; then
   TEST_ARGS="$TEST_ARGS -coverprofile=/tmp/profile.out"
 fi
+if [[ -n $TEST_EXCEPT ]]; then
+  TEST_PACKAGE="\$($TEST_EXCEPT)"
+fi
 
 # Run tests, sequentially because the API is shared resource
 echo "Running tests ..."