From 33c4e76d916f275e8c94e0e35349bc0c218273e1 Mon Sep 17 00:00:00 2001 From: Joshua Powers Date: Thu, 16 Nov 2023 08:53:56 -0700 Subject: [PATCH] test(inputs.vault): Wait for minimum number of metrics (#14302) Additional support for vault test to ensure we wait for the minimum 50 metrics we are expecting due to the way that vault takes time to start up. --- plugins/inputs/vault/vault_test.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/plugins/inputs/vault/vault_test.go b/plugins/inputs/vault/vault_test.go index fde45c790..0028d13ad 100644 --- a/plugins/inputs/vault/vault_test.go +++ b/plugins/inputs/vault/vault_test.go @@ -238,7 +238,10 @@ func TestIntegration(t *testing.T) { // Collect the metrics and compare var acc testutil.Accumulator - require.NoError(t, plugin.Gather(&acc)) + require.Eventually(t, func() bool { + require.NoError(t, plugin.Gather(&acc)) + return len(acc.GetTelegrafMetrics()) > 50 + }, 5*time.Second, 100*time.Millisecond) actual := acc.GetTelegrafMetrics() testutil.RequireMetricsStructureSubset(t, expected, actual, options...)