Merge pull request #5016 from camilamacedo86/fix-unit-test-ci

🌱 (fix): Remove duplication to avoid flakes in the ci and keep it more optimise

Author: k8s-ci-robot

.github/workflows/coverage.yml

@@ -0,0 +1,31 @@
+name: Coverage
+
+on:
+  push:
+    paths-ignore: ['**/*.md']
+  pull_request:
+    paths-ignore: ['**/*.md']
+
+jobs:
+  coverage:
+    runs-on: ubuntu-latest
+    if: (github.event_name == 'push' || github.event.pull_request.head.repo.full_name != github.repository)
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v5
+
+      - name: Setup Go
+        uses: actions/setup-go@v5
+        with:
+          go-version-file: go.mod
+
+      - name: Remove pre-installed kustomize
+        run: sudo rm -f /usr/local/bin/kustomize
+
+      - name: Run tests with coverage
+        run: make test-coverage
+
+      - name: Upload coverage to Coveralls
+        uses: shogo82148/actions-goveralls@v1
+        with:
+          path-to-profile: coverage-all.out

.github/workflows/cross-platform-tests.yml

@@ -0,0 +1,43 @@
+name: Cross-Platform Tests
+
+# Trigger the workflow on pull requests and direct pushes to any branch
+on:
+  push:
+    paths-ignore:
+      - '**/*.md'
+  pull_request:
+    paths-ignore:
+      - '**/*.md'
+
+jobs:
+  test:
+    name: ${{ matrix.os }}
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os:
+          - ubuntu-latest
+          - macos-latest
+    # Pull requests from the same repository won't trigger this checks as they were already triggered by the push
+    if: (github.event_name == 'push' || github.event.pull_request.head.repo.full_name != github.repository)
+    steps:
+      - name: Clone the code
+        uses: actions/checkout@v5
+      - name: Setup Go
+        uses: actions/setup-go@v5
+        with:
+          go-version-file: go.mod
+      # This step is needed as the following one tries to remove
+      # kustomize for each test but has no permission to do so
+      - name: Remove pre-installed kustomize
+        run: sudo rm -f /usr/local/bin/kustomize
+
+      - name: Unit Tests
+        run: make test-unit
+
+      - name: Run Testdata
+        run: make test-testdata
+
+      - name: Run Integration Tests
+        run: make test-integration
+

.github/workflows/lint.yml

@@ -34,3 +34,10 @@ jobs:
       - uses: actions/checkout@v5
       - name: Run yamllint make target
         run: make yamllint
+
+  license:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v5
+      - name: Run license check
+        run: make test-license

.github/workflows/test-e2e-book.yml renamed to .github/workflows/test-book.yml

@@ -47,6 +47,10 @@ jobs:
       - name: Create kind cluster
         run: kind create cluster
 
+      - name: Running make test for ${{ matrix.folder }}
+        working-directory: ${{ matrix.folder }}
+        run: make test
+
       - name: Running make test-e2e for ${{ matrix.folder }}
         working-directory: ${{ matrix.folder }}
         run: make test-e2e

.github/workflows/unit-tests.yml

@@ -144,7 +144,7 @@ go-apidiff:
 ##@ Tests
 
 .PHONY: test
-test: test-unit test-integration test-testdata test-book test-license ## Run the unit and integration tests (used in the CI)
+test: test-unit test-integration test-features test-testdata test-book test-license ## Run the unit and integration tests (used in the CI)
 
 .PHONY: test-unit
 TEST_PKGS := ./pkg/... ./test/e2e/utils/...
@@ -156,10 +156,13 @@ test-coverage: ## Run unit tests creating the output to report coverage
 	- rm -rf *.out  # Remove all coverage files if exists
 	go test -race -failfast -tags=integration -coverprofile=coverage-all.out -coverpkg="./pkg/cli/...,./pkg/config/...,./pkg/internal/...,./pkg/machinery/...,./pkg/model/...,./pkg/plugin/...,./pkg/plugins/golang" $(TEST_PKGS)
 
+.PHONY: test-features
+test-features: ## Run the integration tests
+	./test/features.sh
+
 .PHONY: test-integration
 test-integration: ## Run the integration tests
 	./test/integration.sh
-	./test/features.sh
 
 .PHONY: check-testdata
 check-testdata: ## Run the script to ensure that the testdata is updated