mudhorn/prometheus
1
0
Fork 0
mirror of https://github.com/prometheus/prometheus.git synced 2024-09-19 23:37:31 -07:00
Code Issues Actions 7 Packages Projects Releases Wiki Activity

tsdb tests: fix mockIndex implementation

Browse source 
Signed-off-by: Nicolás Pazos <npazosmendez@gmail.com>
This commit is contained in:
Nicolás Pazos 2024-09-17 13:56:36 -03:00
parent 6c5e2f8fb9
commit db5a6790bb
1 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
tsdb/querier_test.go
View file

@ -20,6 +20,7 @@ import (
"math" "math"
"math/rand" "math/rand"
"path/filepath" "path/filepath"
"slices"
"sort" "sort"
"strconv" "strconv"
"sync" "sync"
@ -2277,12 +2278,16 @@ func (m mockIndex) LabelValues(_ context.Context, name string, matchers ...*labe
} }
for _, series := range m.series { for _, series := range m.series {
matches := true
for _, matcher := range matchers { for _, matcher := range matchers {
if matcher.Matches(series.l.Get(matcher.Name)) { matches = matches && matcher.Matches(series.l.Get(matcher.Name))
// TODO(colega): shouldn't we check all the matchers before adding this to the values? if !matches {
values = append(values, series.l.Get(name)) break
} }
} }
if matches && !slices.Contains(values, series.l.Get(name)) {
values = append(values, series.l.Get(name))
}
} }
return values, nil return values, nil
@ -2382,7 +2387,7 @@ func (m mockIndex) LabelNames(_ context.Context, matchers ...*labels.Matcher) ([
for _, series := range m.series { for _, series := range m.series {
matches := true matches := true
for _, matcher := range matchers { for _, matcher := range matchers {
matches = matches || matcher.Matches(series.l.Get(matcher.Name)) matches = matches && matcher.Matches(series.l.Get(matcher.Name))
if !matches { if !matches {
break break
} }