Remove parallelism from tests for simpler log output.

google · Mar 24, 2019 · a3c2987 · a3c2987
1 parent 671ffac
commit a3c2987
Show file tree

Hide file tree

Showing 10 changed files with 0 additions and 13 deletions.
diff --git a/internal/exporter/json_test.go b/internal/exporter/json_test.go
@@ -92,7 +92,6 @@ func TestHandleJSON(t *testing.T) {
 	for _, tc := range handleJSONTests {
 		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
-			t.Parallel()
 			ms := metrics.NewStore()
 			for _, metric := range tc.metrics {
 				testutil.FatalIfErr(t, ms.Add(metric))

diff --git a/internal/exporter/prometheus_test.go b/internal/exporter/prometheus_test.go
@@ -191,7 +191,6 @@ func TestHandlePrometheus(t *testing.T) {
 	for _, tc := range handlePrometheusTests {
 		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
-			t.Parallel()
 			ms := metrics.NewStore()
 			for _, metric := range tc.metrics {
 				testutil.FatalIfErr(t, ms.Add(metric))

diff --git a/internal/exporter/varz_test.go b/internal/exporter/varz_test.go
@@ -67,7 +67,6 @@ func TestHandleVarz(t *testing.T) {
 	for _, tc := range handleVarzTests {
 		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
-			t.Parallel()
 			ms := metrics.NewStore()
 			for _, metric := range tc.metrics {
 				testutil.FatalIfErr(t, ms.Add(metric))

diff --git a/internal/metrics/metric_test.go b/internal/metrics/metric_test.go
@@ -102,7 +102,6 @@ func TestEmitLabelSet(t *testing.T) {
 	for _, tc := range labelSetTests {
 		tc := tc
 		t.Run(fmt.Sprintf("%v", tc.values), func(t *testing.T) {
-			t.Parallel()
 			m := NewMetric("test", "prog", Gauge, Int, "foo", "bar", "quux")
 			d, _ := m.GetDatum(tc.values...)
 			datum.SetInt(d, 37, ts)

diff --git a/internal/vm/checker/checker_test.go b/internal/vm/checker/checker_test.go
@@ -173,7 +173,6 @@ func TestCheckInvalidPrograms(t *testing.T) {
 	for _, tc := range checkerInvalidPrograms {
 		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
-			t.Parallel()
 			ast, err := parser.Parse(tc.name, strings.NewReader(tc.program))
 			if err != nil {
 				t.Fatal(err)
@@ -369,7 +368,6 @@ func TestCheckValidPrograms(t *testing.T) {
 	for _, tc := range checkerValidPrograms {
 		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
-			t.Parallel()
 			ast, err := parser.Parse(tc.name, strings.NewReader(tc.program))
 			if err != nil {
 				t.Fatal(err)
@@ -417,7 +415,6 @@ func TestCheckTypeExpressions(t *testing.T) {
 	for _, tc := range checkerTypeExpressionTests {
 		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
-			t.Parallel()
 			ast, err := checker.Check(tc.expr)
 			if err != nil {
 				t.Fatalf("check error: %s", err)

diff --git a/internal/vm/codegen/codegen_test.go b/internal/vm/codegen/codegen_test.go
@@ -856,7 +856,6 @@ func TestCodegen(t *testing.T) {
 	for _, tc := range testCodeGenPrograms {
 		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
-			t.Parallel()
 			ast, err := parser.Parse(tc.name, strings.NewReader(tc.source))
 			if err != nil {
 				t.Fatalf("Parse error: %s", err)

diff --git a/internal/vm/loader_test.go b/internal/vm/loader_test.go
@@ -113,7 +113,6 @@ func TestProcessEvents(t *testing.T) {
 	for _, tt := range testProcessEvents {
 		tt := tt
 		t.Run(tt.name, func(t *testing.T) {
-			t.Parallel()
 			w := watcher.NewFakeWatcher()
 			store := metrics.NewStore()
 			lines := make(chan *logline.LogLine)

diff --git a/internal/vm/parser/parser_test.go b/internal/vm/parser/parser_test.go
@@ -332,7 +332,6 @@ func TestParserRoundTrip(t *testing.T) {
 	for _, tc := range parserTests {
 		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
-			t.Parallel()
 			p := newParser(tc.name, strings.NewReader(tc.program))
 			r := mtailParse(p)
 

diff --git a/internal/vm/types/types_test.go b/internal/vm/types/types_test.go
@@ -128,7 +128,6 @@ func TestTypeUnification(t *testing.T) {
 	for _, tc := range typeUnificationTests {
 		tc := tc
 		t.Run(fmt.Sprintf("%s %s", tc.a, tc.b), func(t *testing.T) {
-			t.Parallel()
 			err := Unify(tc.a, tc.b)
 			if err != nil {
 				t.Errorf("%s", err)
@@ -210,7 +209,6 @@ func TestInferCaprefType(t *testing.T) {
 	for _, tc := range inferCaprefTypeTests {
 		tc := tc
 		t.Run(tc.pattern, func(t *testing.T) {
-			t.Parallel()
 			re, err := syntax.Parse(`(`+tc.pattern+`)`, syntax.Perl)
 			if err != nil {
 				t.Fatal(err)

diff --git a/internal/vm/vm_test.go b/internal/vm/vm_test.go
@@ -465,7 +465,6 @@ func TestInstrs(t *testing.T) {
 	for _, tc := range instructions {
 		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
-			t.Parallel()
 			var m []*metrics.Metric
 			m = append(m,
 				metrics.NewMetric("foo", "test", metrics.Counter, metrics.Int),