Ignore tparallel where it doesnt make sense

Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com>
juanfont · Dec 1, 2022 · 6f4c6c1 · 6f4c6c1
1 parent eb072a1
commit 6f4c6c1
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/integration/general_test.go b/integration/general_test.go
@@ -122,6 +122,10 @@ func TestPingAllByHostname(t *testing.T) {
 	}
 }
 
+// If subtests are parallel, then they will start before setup is run.
+// This might mean we approach setup slightly wrong, but for now, ignore
+// the linter
+// nolint:tparallel
 func TestTaildrop(t *testing.T) {
 	IntegrationSkip(t)
 	t.Parallel()

diff --git a/integration/scenario_test.go b/integration/scenario_test.go
@@ -21,6 +21,10 @@ func IntegrationSkip(t *testing.T) {
 	}
 }
 
+// If subtests are parallel, then they will start before setup is run.
+// This might mean we approach setup slightly wrong, but for now, ignore
+// the linter
+// nolint:tparallel
 func TestHeadscale(t *testing.T) {
 	IntegrationSkip(t)
 	t.Parallel()
@@ -70,6 +74,10 @@ func TestHeadscale(t *testing.T) {
 	}
 }
 
+// If subtests are parallel, then they will start before setup is run.
+// This might mean we approach setup slightly wrong, but for now, ignore
+// the linter
+// nolint:tparallel
 func TestCreateTailscale(t *testing.T) {
 	IntegrationSkip(t)
 	t.Parallel()
@@ -104,6 +112,10 @@ func TestCreateTailscale(t *testing.T) {
 	}
 }
 
+// If subtests are parallel, then they will start before setup is run.
+// This might mean we approach setup slightly wrong, but for now, ignore
+// the linter
+// nolint:tparallel
 func TestTailscaleNodesJoiningHeadcale(t *testing.T) {
 	IntegrationSkip(t)
 	t.Parallel()