Skip to content

Commit 4af2602

Browse files
authored
test(spanner): fix continuous test failure (#11629)
* test(spanner): fix continuous test failure * running the test in parallel can inject mock monitoring to other test cases causing flakiness
1 parent 02458a5 commit 4af2602

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

spanner/metrics_test.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@ func TestNewBuiltinMetricsTracerFactory(t *testing.T) {
4141
if testing.Short() {
4242
t.Skip("TestNewBuiltinMetricsTracerFactory tests skipped in -short mode.")
4343
}
44-
t.Parallel()
4544

4645
ctx := context.Background()
4746
clientUID := "test-uid"
@@ -121,7 +120,12 @@ func TestNewBuiltinMetricsTracerFactory(t *testing.T) {
121120
{
122121
desc: "should create a new tracer factory with default meter provider",
123122
runOnlyInEmulator: isEmulatorEnvSet(),
124-
config: ClientConfig{},
123+
config: ClientConfig{
124+
SessionPoolConfig: SessionPoolConfig{
125+
MinOpened: 0,
126+
MaxOpened: 1,
127+
},
128+
},
125129

126130
wantBuiltinEnabled: true,
127131
wantCreateTSCallsCount: 2,

0 commit comments

Comments
 (0)