Automated merge

* upstream/main:
  Include integration tests in code coverage calculation (istio-ecosystem#427)
  Improve failure messages for supportedversion unit tests (istio-ecosystem#442)

.github/workflows/coverage.yaml

@@ -10,12 +10,12 @@ jobs:
 
     - name: Run coverage
       run: |
-        make test.unit \
+        make test \
           -e COVERAGE=true
 
     - name: Upload to Codecov
       uses: codecov/codecov-action@v4
       with:
-        file: out/coverage.out
+        files: out/coverage-unit.out,out/coverage-integration.out
         token: ${{ secrets.CODECOV_TOKEN }}
-        fail_ci_if_error: true
+        fail_ci_if_error: true

Makefile.core.mk

@@ -71,8 +71,8 @@ ENVTEST_K8S_VERSION ?= 1.29.0
 # Set DOCKER_BUILD_FLAGS to specify flags to pass to 'docker build', default to empty. Example: --platform=linux/arm64
 DOCKER_BUILD_FLAGS ?= "--platform=$(TARGET_OS)/$(TARGET_ARCH)"
 
-GOTEST_FLAGS := $(if $(VERBOSE),-v) $(if $(COVERAGE),-coverprofile=$(REPO_ROOT)/out/coverage.out)
-GINKGO_FLAGS := $(if $(VERBOSE),-v) $(if $(CI),--no-color)
+GOTEST_FLAGS := $(if $(VERBOSE),-v) $(if $(COVERAGE),-coverprofile=$(REPO_ROOT)/out/coverage-unit.out)
+GINKGO_FLAGS := $(if $(VERBOSE),-v) $(if $(CI),--no-color) $(if $(COVERAGE),-coverprofile=coverage-integration.out -coverpkg=./... --output-dir=out)
 
 # CHANNELS define the bundle channels used in the bundle.
 # Add a new line here if you would like to change its default config. (E.g CHANNELS = "candidate,fast,stable")

pkg/test/util/supportedversion/supportedversion_test.go

@@ -22,11 +22,11 @@ import (
 
 func TestInit(t *testing.T) {
 	// no need to call init(), since it's called automatically
-	assert.True(t, len(List) > 0, "List should not be empty")
-	assert.True(t, len(Map) > 0, "M should not be empty")
-	assert.True(t, Default != "", "Default should not be empty")
-	assert.True(t, Old != "", "Default should not be empty")
-	assert.True(t, New != "", "Default should not be empty")
+	assert.True(t, len(List) > 0, "supportedversion.List should not be empty")
+	assert.True(t, len(Map) > 0, "supportedversion.Map should not be empty")
+	assert.True(t, Default != "", "supportedversion.Default should not be empty")
+	assert.True(t, Old != "", "supportedversion.Old should not be empty")
+	assert.True(t, New != "", "supportedversion.New should not be empty")
 
 	assert.Equal(t, len(List), len(Map), "Map should be same size as List")
 	for _, vi := range List {