let's ignore some

Author: FoseFx

libs/common/grpc.go

@@ -121,7 +121,10 @@ func DefaultStreamInterceptors(metrics *prometheusGrpcProvider.ServerMetrics) []
 func DefaultServerOptions() []grpc.ServerOption {
 	// register new metrics collector with prometheus
 	metrics := prometheusGrpcProvider.NewServerMetrics()
-	telemetry.PrometheusRegistry().MustRegister(metrics)
+
+	if !hwutil.IsRunningInGoTests() {
+		telemetry.PrometheusRegistry().MustRegister(metrics)
+	}
 
 	unaryInterceptorChain := grpc.ChainUnaryInterceptor(DefaultUnaryInterceptors(metrics)...)
 	streamInterceptorChain := grpc.ChainStreamInterceptor(DefaultStreamInterceptors(metrics)...)

libs/common/hwgrpc/locale_interceptor_test.go

@@ -24,7 +24,6 @@ func TestLocaleInterceptor(t *testing.T) {
 	}
 
 	for acceptLanguageHeader, expectedLocalesStrings := range testCases {
-		t.Parallel()
 		t.Run(fmt.Sprintf("Test localeInterceptor with accept-language header of '%s'", acceptLanguageHeader),
 			func(t *testing.T) {
 				t.Parallel()

libs/hwutil/env.go

@@ -1,6 +1,7 @@
 package hwutil
 
 import (
+	"flag"
 	"os"
 
 	"github.com/rs/zerolog/log"
@@ -29,3 +30,7 @@ func HasEnv(key string) bool {
 	_, found := os.LookupEnv(key)
 	return found
 }
+
+func IsRunningInGoTests() bool {
+	return flag.Lookup("test.v") != nil
+}

services/property-svc/internal/property-view/api/grpc_test.go

@@ -71,9 +71,8 @@ func setup() (
 	return ctx, client, as, dbMock, teardown
 }
 
+//nolint:paralleltest
 func TestPropertyViewGrpcService_UpdatePropertyViewRule_Validation(t *testing.T) {
-	t.Parallel()
-
 	ctx, client, _, dbMock, teardown := setup()
 	defer teardown()
 
@@ -243,9 +242,8 @@ func TestPropertyViewGrpcService_UpdatePropertyViewRule_AllEmptyNoEffect(t *test
 	as.ExpectToBeEmpty(t)
 }
 
+//nolint:paralleltest
 func TestPropertyViewGrpcService_UpdatePropertyViewRule_TaskPropertyMatcher_GreenPath_Created(t *testing.T) {
-	t.Parallel()
-
 	ctx, client, as, dbMock, teardown := setup()
 	defer teardown()
 
@@ -313,9 +311,8 @@ func TestPropertyViewGrpcService_UpdatePropertyViewRule_TaskPropertyMatcher_Gree
 	})
 }
 
+//nolint:paralleltest
 func TestPropertyViewGrpcService_UpdatePropertyViewRule_PatientPropertyMatcher_GreenPath_Created(t *testing.T) {
-	t.Parallel()
-
 	ctx, client, as, dbMock, teardown := setup()
 	defer teardown()
 
@@ -383,9 +380,8 @@ func TestPropertyViewGrpcService_UpdatePropertyViewRule_PatientPropertyMatcher_G
 	})
 }
 
+//nolint:paralleltest
 func TestPropertyViewGrpcService_UpdatePropertyViewRule_TaskPropertyMatcher_GreenPath_Updated(t *testing.T) {
-	t.Parallel()
-
 	ctx, client, as, dbMock, teardown := setup()
 	defer teardown()
 
@@ -482,9 +478,8 @@ func TestPropertyViewGrpcService_UpdatePropertyViewRule_TaskPropertyMatcher_Gree
 	})
 }
 
+//nolint:paralleltest
 func TestPropertyViewGrpcService_UpdatePropertyViewRule_PatientPropertyMatcher_GreenPath_Updated(t *testing.T) {
-	t.Parallel()
-
 	ctx, client, as, dbMock, teardown := setup()
 	defer teardown()
 

services/property-svc/internal/property-view/projections/property_rules_postgres/property_rules_postgres_test.go

@@ -52,9 +52,8 @@ func setup() (ctx context.Context, projection *Projection, dbMock pgxmock.PgxPoo
 	return ctx, projection, dbMock, teardown
 }
 
+//nolint:paralleltest
 func TestPropertyViewPropertyRulesProjection_Create_TaskPropertyMatcher_GreenPath(t *testing.T) {
-	t.Parallel()
-
 	ctx, projection, dbMock, teardown := setup()
 	defer teardown()
 

services/property-svc/stories/GetPropertieBySubjectType_test.go

@@ -17,9 +17,9 @@ import (
 // TestGetPropertiesBySubjectType:
 //   - Create Properties
 //   - Check GetPropertiesBySubjectType
+//
+//nolint:paralleltest
 func TestGetProperties(t *testing.T) {
-	t.Parallel()
-
 	propertyClient := propertyServiceClient()
 
 	ctx := context.Background()

services/tasks-svc/internal/task/api/grpc_test.go

@@ -252,9 +252,8 @@ func TestTaskGrpcService_CreateSubtask_Validation(t *testing.T) {
 	require.NoError(t, err, codes.InvalidArgument, "rejects fully valid request")
 }
 
+//nolint:paralleltest
 func TestTaskGrpcService_UpdateSubtask_Validation(t *testing.T) {
-	t.Parallel()
-
 	ctx, client, teardown := setup()
 	defer teardown()