From db5a6790bb9fc982c845ee13231cdb7917d2d5d4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicol=C3=A1s=20Pazos?= Date: Tue, 17 Sep 2024 13:56:36 -0300 Subject: [PATCH] tsdb tests: fix `mockIndex` implementation MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Nicolás Pazos --- tsdb/querier_test.go | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/tsdb/querier_test.go b/tsdb/querier_test.go index 0821b2b37..3214c2d37 100644 --- a/tsdb/querier_test.go +++ b/tsdb/querier_test.go @@ -20,6 +20,7 @@ import ( "math" "math/rand" "path/filepath" + "slices" "sort" "strconv" "sync" @@ -2277,12 +2278,16 @@ func (m mockIndex) LabelValues(_ context.Context, name string, matchers ...*labe } for _, series := range m.series { + matches := true for _, matcher := range matchers { - if matcher.Matches(series.l.Get(matcher.Name)) { - // TODO(colega): shouldn't we check all the matchers before adding this to the values? - values = append(values, series.l.Get(name)) + matches = matches && matcher.Matches(series.l.Get(matcher.Name)) + if !matches { + break } } + if matches && !slices.Contains(values, series.l.Get(name)) { + values = append(values, series.l.Get(name)) + } } return values, nil @@ -2382,7 +2387,7 @@ func (m mockIndex) LabelNames(_ context.Context, matchers ...*labels.Matcher) ([ for _, series := range m.series { matches := true for _, matcher := range matchers { - matches = matches || matcher.Matches(series.l.Get(matcher.Name)) + matches = matches && matcher.Matches(series.l.Get(matcher.Name)) if !matches { break }