Browse Source

metrics: add missing calls to Ticker.Stop in tests (#20866)

ucwong 5 years ago
parent
commit
53e034ce0b
2 changed files with 2 additions and 0 deletions
  1. 1 0
      metrics/meter_test.go
  2. 1 0
      metrics/sample_test.go

+ 1 - 0
metrics/meter_test.go

@@ -26,6 +26,7 @@ func TestMeterDecay(t *testing.T) {
 		ticker: time.NewTicker(time.Millisecond),
 		meters: make(map[*StandardMeter]struct{}),
 	}
+	defer ma.ticker.Stop()
 	m := newStandardMeter()
 	ma.meters[m] = struct{}{}
 	go ma.tick()

+ 1 - 0
metrics/sample_test.go

@@ -346,6 +346,7 @@ func TestUniformSampleConcurrentUpdateCount(t *testing.T) {
 	quit := make(chan struct{})
 	go func() {
 		t := time.NewTicker(10 * time.Millisecond)
+		defer t.Stop()
 		for {
 			select {
 			case <-t.C: