fix fail test with cover flag

Signed-off-by: Fedor Partanskiy <fedor.partanskiy@atme.com>

Author: pfi79

.github/workflows/verify-build.yml

@@ -37,8 +37,6 @@ jobs:
     name: Unit Tests
     needs: basic-checks
     runs-on: ${{ github.repository == 'hyperledger/fabric' && 'fabric-ubuntu-22.04' || 'ubuntu-22.04' }}
-    env:
-      GOEXPERIMENT: nocoverageredesign
     steps:
       - uses: actions/checkout@v4
         with:

core/handlers/library/registry_plugin_test.go

@@ -45,7 +45,7 @@ func buildPlugin(t *testing.T, dest, pkg string) {
 	require.NoError(t, err, "Could not build plugin: "+string(output))
 }
 
-func TestLoadAuthPlugin(t *testing.T) {
+func TestLoadAuthPluginNoCover(t *testing.T) {
 	if noplugin {
 		t.Skip("plugins disabled")
 	}
@@ -65,7 +65,7 @@ func TestLoadAuthPlugin(t *testing.T) {
 	require.True(t, endorser.invoked, "Expected filter to invoke endorser on invoke")
 }
 
-func TestLoadDecoratorPlugin(t *testing.T) {
+func TestLoadDecoratorPluginNoCover(t *testing.T) {
 	if noplugin {
 		t.Skip("plugins disabled")
 	}
@@ -86,7 +86,7 @@ func TestLoadDecoratorPlugin(t *testing.T) {
 	require.True(t, proto.Equal(decoratedInput, testInput), "Expected chaincode input to remain unchanged")
 }
 
-func TestEndorsementPlugin(t *testing.T) {
+func TestEndorsementPluginNoCover(t *testing.T) {
 	if noplugin {
 		t.Skip("plugins disabled")
 	}
@@ -109,7 +109,7 @@ func TestEndorsementPlugin(t *testing.T) {
 	require.Equal(t, []byte{1, 2, 3}, output)
 }
 
-func TestValidationPlugin(t *testing.T) {
+func TestValidationPluginNoCover(t *testing.T) {
 	if noplugin {
 		t.Skip("plugins disabled")
 	}

scripts/run-unit-tests.sh

@@ -18,6 +18,10 @@ serial_packages=(
     "github.com/hyperledger/fabric/gossip/..."
 )
 
+no_coverage_packages=(
+    "github.com/hyperledger/fabric/core/handlers/library"
+)
+
 # packages which need to be tested with build tag pkcs11
 pkcs11_packages=(
     "github.com/hyperledger/fabric/bccsp/factory"
@@ -132,6 +136,14 @@ serial_test_packages() {
     fi
 }
 
+no_coverage_test_packages() {
+    local filter
+    filter=$(package_filter "${no_coverage_packages[@]}")
+    if [ -n "$filter" ]; then
+        join_by $'\n' "$@" | grep -E "$filter" || true
+    fi
+}
+
 # "go test" the provided packages. Packages that are not present in the serial package list
 # will be tested in parallel
 run_tests() {
@@ -159,7 +171,16 @@ run_tests() {
         local -a parallel
         while IFS= read -r pkg; do parallel+=("$pkg"); done < <(parallel_test_packages "$@")
         if [ "${#parallel[@]}" -ne 0 ]; then
-            go test "${flags[@]}" "${race_flags[@]}" -tags "$GO_TAGS" "${parallel[@]}" -short -timeout=20m
+            go test "${flags[@]}" "${race_flags[@]}" -tags "$GO_TAGS" "${parallel[@]}" -short -timeout=20m -skip=NoCover
+        fi
+
+        local -a no_coverage
+        while IFS= read -r pkg; do no_coverage+=("$pkg"); done < <(no_coverage_test_packages "$@")
+        if [ "${#no_coverage[@]}" -ne 0 ]; then
+            echo "test with no coverage"
+            unset flags[0]
+            flags=("${flags[@]}")
+            go test "${flags[@]}" "${race_flags[@]}" -tags "$GO_TAGS" "${no_coverage[@]}" -short -timeout=20m -run=NoCover
         fi
     }
 }