feat: t.Parallel()

Author: FoseFx

.golangci.yaml

@@ -35,7 +35,6 @@ linters:
     - godox # TODO
     - nestif # TODO
     - nlreturn # TODO
-    - paralleltest # TODO
     - protogetter # TODO
     - revive # TODO
     - stylecheck # TODO

libs/common/auth/auth_test.go

@@ -7,6 +7,8 @@ import (
 )
 
 func TestIDTokenClaims_AsExpected(t *testing.T) {
+	t.Parallel()
+
 	tests := []struct {
 		name          string
 		claims        IDTokenClaims
@@ -95,6 +97,8 @@ func TestIDTokenClaims_AsExpected(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
+			t.Parallel()
+
 			err := tt.claims.AsExpected()
 			if tt.expectedError {
 				require.Error(t, err)

libs/common/hwgrpc/hwgrpc_test.go

@@ -21,6 +21,7 @@ func arrayEq(t *testing.T, expected []language.Tag, parsed []language.Tag) {
 }
 
 func TestParseLocales(t *testing.T) {
+	t.Parallel()
 	const mdnExample = "fr-CH, fr;q=0.9, en;q=0.8, de;q=0.7, *;q=0.5"
 	expected := []language.Tag{language.MustParse("fr-CH"), language.French, language.English, language.German}
 	parsed, ok := hwgrpc.ParseLocales(mdnExample)
@@ -32,6 +33,7 @@ func TestParseLocales(t *testing.T) {
 }
 
 func TestParseLocalesReordered(t *testing.T) {
+	t.Parallel()
 	const mdnExample = "fr-CH, de;q=0.7, en;q=0.8, fr;q=0.9, *;q=0.5"
 	expected := []language.Tag{language.MustParse("fr-CH"), language.French, language.English, language.German}
 	parsed, ok := hwgrpc.ParseLocales(mdnExample)
@@ -43,6 +45,7 @@ func TestParseLocalesReordered(t *testing.T) {
 }
 
 func TestParseLocalesSimple(t *testing.T) {
+	t.Parallel()
 	const mdnExample = "de"
 	expected := []language.Tag{language.German}
 	parsed, ok := hwgrpc.ParseLocales(mdnExample)

libs/common/hwgrpc/locale_interceptor_test.go

@@ -12,6 +12,8 @@ import (
 )
 
 func TestLocaleInterceptor(t *testing.T) {
+	t.Parallel()
+
 	testCases := map[string][]string{
 		"de":     {"de"},
 		"en":     {"en"},
@@ -22,8 +24,11 @@ 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()
+
 				ctx := context.Background()
 
 				md := metadata.New(map[string]string{

libs/common/hwgrpc/panic_interceptor_test.go

@@ -37,6 +37,7 @@ type RecoverySuite struct {
 }
 
 func TestPanicRecoverInterceptor(t *testing.T) {
+	t.Parallel()
 	telemetry.SetupMetrics(context.Background(), nil)
 	s := &RecoverySuite{
 		InterceptorTestSuite: &testpb.InterceptorTestSuite{

libs/common/test/grpc_test.go

@@ -12,5 +12,6 @@ import (
 // - AuthenticatedUserMetadata accesses the AuthenticatedUserClaim map incorrectly
 // - AuthenticatedUserClaim map is not JSON-able
 func TestAuthenticatedUserMetadataDoesNotCrash(t *testing.T) {
+	t.Parallel()
 	_ = AuthenticatedUserMetadata(uuid.NewString())
 }

libs/hwauthz/spicedb/spicedb_test.go

@@ -52,6 +52,7 @@ func TestMain(m *testing.M) {
 }
 
 func TestBulkCheck(t *testing.T) {
+	t.Parallel()
 	ctx := context.Background()
 	client := NewSpiceDBAuthZ()
 
@@ -102,6 +103,8 @@ func TestBulkCheck(t *testing.T) {
 }
 
 func TestLookupResources(t *testing.T) {
+	t.Parallel()
+
 	ctx := context.Background()
 	client := NewSpiceDBAuthZ()
 
@@ -134,6 +137,8 @@ func TestLookupResources(t *testing.T) {
 }
 
 func TestDeleteObject(t *testing.T) {
+	t.Parallel()
+
 	// client
 	ctx := context.Background()
 	client := NewSpiceDBAuthZ()

libs/hwdb/helper_test.go

@@ -8,13 +8,19 @@ import (
 )
 
 func TestPbToTimestamp(t *testing.T) {
+	t.Parallel()
+
 	t.Run("src = nil", func(t *testing.T) {
+		t.Parallel()
+
 		if hwdb.PbToTimestamp(nil).Valid {
 			t.Error()
 		}
 	})
 
 	t.Run("src not nil", func(t *testing.T) {
+		t.Parallel()
+
 		src := timestamppb.Timestamp{
 			Seconds: 0,
 			Nanos:   0,

libs/hwes/aggregate_test.go

@@ -8,6 +8,8 @@ import (
 )
 
 func TestResolveAggregateIDAndTypeFromStreamID(t *testing.T) {
+	t.Parallel()
+
 	testCases := []struct {
 		streamID              string
 		expectedError         bool

libs/hwes/event_test.go

@@ -11,6 +11,8 @@ import (
 )
 
 func TestEventWithUserID(t *testing.T) {
+	t.Parallel()
+
 	ctx := context.Background()
 	u := uuid.New()
 	e := hwes.Event{}
@@ -25,6 +27,8 @@ func TestEventWithUserID(t *testing.T) {
 }
 
 func TestEventWithOrganizationID(t *testing.T) {
+	t.Parallel()
+
 	ctx := context.Background()
 	u := uuid.New()
 	e := hwes.Event{}