Merge pull request #227 from hashicorp/syncer-test

Fix test configuration race.
hashicorp · Mar 9, 2020 · 19f8701 · 19f8701
2 parents 40963d9 + 5bfb169
commit 19f8701
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/catalog/to-consul/syncer_test.go b/catalog/to-consul/syncer_test.go
@@ -167,10 +167,11 @@ func TestConsulSyncer_noReapingUntilInitialSync(t *testing.T) {
 		Address: a.HTTPAddr,
 	})
 	require.NoError(t, err)
-	s, closer := testConsulSyncer(client)
-	// Set the sync period to 5ms so we know it will have run at least once
-	// after we wait 100ms.
-	s.SyncPeriod = 5 * time.Millisecond
+	s, closer := testConsulSyncerWithConfig(client, func(s *ConsulSyncer) {
+		// Set the sync period to 5ms so we know it will have run at least once
+		// after we wait 100ms.
+		s.SyncPeriod = 5 * time.Millisecond
+	})
 	defer closer()
 
 	// Create a service directly in Consul. Since it was created on the