remove parallel

remove parallel to stabilize ci/cd

.golangci.yml

@@ -17,7 +17,7 @@ linters:
     - cyclop
     - unused
     - usetesting
-    - paralleltest
+    # - paralleltest  # Disabled to prevent race conditions in CI
     - bodyclose      # Check HTTP response bodies are closed
     - gochecknoinits  # Disallow `init` functions
     # - gocritic       # Advanced style/performance checks
@@ -55,9 +55,6 @@ linters-settings:
 
 issues:
   exclude-rules:
-    - path-except: ^test/
-      linters:
-        - paralleltest
     - path: cmd/metricsExample_test/main\.go
       linters:
         - gosec

api/clientSyncRunner/runner_test.go

@@ -26,7 +26,6 @@ type ClientSyncConfig struct {
 }
 
 func TestClientSyncRunner(t *testing.T) {
-	t.Parallel()
 	cfg := &ClientSyncConfig{}
 	test.CreateDB(t, cfg)
 	acfg := test.CreateAWSContainer(t, cfg)

api/docCleanRunner/runner_test.go

@@ -31,7 +31,6 @@ type DocCleanConfig struct {
 }
 
 func TestDocCleanRunner(t *testing.T) {
-	t.Parallel()
 	cfg := &DocCleanConfig{}
 	test.CreateDB(t, cfg)
 	acfg := test.CreateAWSContainer(t, cfg)

api/docInitRunner/runner_test.go

@@ -26,7 +26,6 @@ type DocInitConfig struct {
 }
 
 func TestDocInitRunner(t *testing.T) {
-	t.Parallel()
 	cfg := &DocInitConfig{}
 	test.CreateDB(t, cfg)
 	acfg := test.CreateAWSContainer(t, cfg)

api/docSyncRunner/runner_test.go

@@ -30,7 +30,6 @@ type DocSyncConfig struct {
 }
 
 func TestDocSyncRunner(t *testing.T) {
-	t.Parallel()
 	cfg := &DocSyncConfig{}
 	test.CreateDB(t, cfg)
 	acfg := test.CreateAWSContainer(t, cfg)

api/docTextRunner/runner_test.go

@@ -37,7 +37,6 @@ type DocTextConfig struct {
 }
 
 func TestDocTextRunner(t *testing.T) {
-	t.Parallel()
 	cfg := &DocTextConfig{}
 	test.CreateDB(t, cfg)
 	acfg := test.CreateAWSContainer(t, cfg)

api/queryAPI/client_test.go

@@ -18,7 +18,6 @@ import (
 )
 
 func TestCreateClient(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)
@@ -47,7 +46,6 @@ func TestCreateClient(t *testing.T) {
 }
 
 func TestGetClient(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)
@@ -81,7 +79,6 @@ func TestGetClient(t *testing.T) {
 }
 
 func TestUpdateClient(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)

api/queryAPI/collector_test.go

@@ -15,7 +15,6 @@ import (
 )
 
 func TestSetCollector(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)
@@ -60,7 +59,6 @@ func TestSetCollector(t *testing.T) {
 }
 
 func TestGetCollectorByClientId(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)

api/queryAPI/documents_test.go

@@ -17,7 +17,6 @@ import (
 )
 
 func TestUploadDocument(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)
@@ -57,7 +56,6 @@ func TestUploadDocument(t *testing.T) {
 }
 
 func TestListDocumentsByClientId(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)
@@ -90,7 +88,6 @@ func TestListDocumentsByClientId(t *testing.T) {
 }
 
 func TestGetDocument(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)

api/queryAPI/query_test.go

@@ -41,7 +41,6 @@ import (
 )
 
 func TestCreateQuery(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)
@@ -62,7 +61,6 @@ func TestCreateQuery(t *testing.T) {
 }
 
 func TestListQueries(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)
@@ -91,7 +89,6 @@ func TestListQueries(t *testing.T) {
 }
 
 func TestGetQuery(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)
@@ -116,7 +113,6 @@ func TestGetQuery(t *testing.T) {
 }
 
 func TestUpdateQuery(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)
@@ -230,7 +226,6 @@ func createDocumentWithText(t testing.TB, cfg *ControllerConfig, clientId string
 }
 
 func TestTestQuery(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)

api/queryAPI/status_test.go

@@ -13,7 +13,6 @@ import (
 )
 
 func TestGetClientStatus(t *testing.T) {
-	t.Parallel()
 	cfg := &ControllerConfig{}
 	test.CreateDB(t, cfg)
 	initializeTestConfig(t, cfg)

internal/server/api/listener_test.go

@@ -22,7 +22,6 @@ func TestNewAPI(t *testing.T) {
 	if testing.Short() {
 		t.SkipNow()
 	}
-	t.Parallel()
 	ctx := t.Context()
 
 	cfg := &BaseConfig{}

internal/test/mockserver_test.go

@@ -51,7 +51,6 @@ func TestCreateDetectDocumentTextExpectation(t *testing.T) {
 }
 
 func TestWaitForMockEndpoint(t *testing.T) {
-	t.Parallel()
 	if testing.Short() {
 		t.SkipNow()
 	}

internal/test/objectstore_test.go

@@ -21,7 +21,6 @@ type StoreConfig struct {
 }
 
 func TestCreateBucket(t *testing.T) {
-	t.Parallel()
 	if testing.Short() {
 		t.SkipNow()
 	}

internal/test/queue_test.go

@@ -22,7 +22,6 @@ type TestConfig struct {
 }
 
 func TestCreateQueue(t *testing.T) {
-	t.Parallel()
 	if testing.Short() {
 		t.SkipNow()
 	}
@@ -38,7 +37,6 @@ func TestCreateQueue(t *testing.T) {
 }
 
 func TestAssertMessageWait(t *testing.T) {
-	t.Parallel()
 	if testing.Short() {
 		t.SkipNow()
 	}
@@ -63,7 +61,6 @@ func TestAssertMessageWait(t *testing.T) {
 }
 
 func TestAssertMessageBodyWait(t *testing.T) {
-	t.Parallel()
 	if testing.Short() {
 		t.SkipNow()
 	}
@@ -85,7 +82,6 @@ func TestAssertMessageBodyWait(t *testing.T) {
 }
 
 func TestAssertMessageAttrWait(t *testing.T) {
-	t.Parallel()
 	if testing.Short() {
 		t.SkipNow()
 	}

test/process_test.go

@@ -24,7 +24,6 @@ type Config struct {
 
 func TestProcess(t *testing.T) {
 	t.Run("basic upload", func(t *testing.T) {
-		t.Parallel()
 		cfg := &Config{}
 
 		net, clean := test.CreateFullNetwork(t, t.Context(), cfg)
@@ -95,7 +94,6 @@ func TestProcess(t *testing.T) {
 		assert.EqualExportedValues(t, expectedDoc, *docRes.JSON200)
 	})
 	t.Run("update config", func(t *testing.T) {
-		t.Parallel()
 		cfg := &Config{}
 
 		net, clean := test.CreateFullNetwork(t, t.Context(), cfg)
@@ -176,7 +174,6 @@ func TestProcess(t *testing.T) {
 		assert.EqualExportedValues(t, expectedDoc, *docRes.JSON200)
 	})
 	t.Run("test multiple versions", func(t *testing.T) {
-		t.Parallel()
 		cfg := &Config{}
 
 		net, clean := test.CreateFullNetwork(t, t.Context(), cfg)

test/queryAPI/accessory_test.go

@@ -13,7 +13,6 @@ import (
 )
 
 func TestQueryAPIAccessories(t *testing.T) {
-	t.Parallel()
 	ctx := t.Context()
 
 	cfg := &Config{}

test/queryAPI/client_test.go

@@ -12,7 +12,6 @@ import (
 )
 
 func TestClient(t *testing.T) {
-	t.Parallel()
 	ctx := t.Context()
 
 	cfg := &Config{}

test/queryAPI/collectorservice_test.go

@@ -26,7 +26,6 @@ type Config struct {
 }
 
 func TestCollectorService(t *testing.T) {
-	t.Parallel()
 	ctx := t.Context()
 
 	cfg := &Config{}

test/queryAPI/queryservice_test.go

@@ -15,7 +15,6 @@ import (
 
 func TestQueryAPI(t *testing.T) {
 	t.Run("list no queries", func(t *testing.T) {
-		t.Parallel()
 		cfg := &Config{}
 
 		c, cleanup := test.CreateAPINetworkWithParams(t, cfg, test.QueryAPI, &test.FullDependenciesParams{
@@ -32,7 +31,6 @@ func TestQueryAPI(t *testing.T) {
 		assert.Len(t, queriesRes.JSON200.Queries, 0)
 	})
 	t.Run("create and get query", func(t *testing.T) {
-		t.Parallel()
 		ctx := t.Context()
 
 		cfg := &Config{}
@@ -71,7 +69,6 @@ func TestQueryAPI(t *testing.T) {
 		assert.Equal(t, jcfg, *queriesRes.JSON200.Queries[0].Config)
 	})
 	t.Run("update query", func(t *testing.T) {
-		t.Parallel()
 		ctx := t.Context()
 
 		cfg := &Config{}
@@ -116,7 +113,6 @@ func TestQueryAPI(t *testing.T) {
 		assert.Nil(t, queryRes.JSON200.RequiredQueries)
 	})
 	t.Run("multiple dependent queries", func(t *testing.T) {
-		t.Parallel()
 		ctx := t.Context()
 
 		cfg := &Config{}