Merge pull request #15903 from prometheus/beorn7/promql2
Some checks failed
buf.build / lint and publish (push) Has been cancelled
CI / Go tests (push) Has been cancelled
CI / More Go tests (push) Has been cancelled
CI / Go tests with previous Go version (push) Has been cancelled
CI / UI tests (push) Has been cancelled
CI / Go tests on Windows (push) Has been cancelled
CI / Mixins tests (push) Has been cancelled
CI / Build Prometheus for common architectures (0) (push) Has been cancelled
CI / Build Prometheus for common architectures (1) (push) Has been cancelled
CI / Build Prometheus for common architectures (2) (push) Has been cancelled
CI / Build Prometheus for all architectures (0) (push) Has been cancelled
CI / Build Prometheus for all architectures (1) (push) Has been cancelled
CI / Build Prometheus for all architectures (10) (push) Has been cancelled
CI / Build Prometheus for all architectures (11) (push) Has been cancelled
CI / Build Prometheus for all architectures (2) (push) Has been cancelled
CI / Build Prometheus for all architectures (3) (push) Has been cancelled
CI / Build Prometheus for all architectures (4) (push) Has been cancelled
CI / Build Prometheus for all architectures (5) (push) Has been cancelled
CI / Build Prometheus for all architectures (6) (push) Has been cancelled
CI / Build Prometheus for all architectures (7) (push) Has been cancelled
CI / Build Prometheus for all architectures (8) (push) Has been cancelled
CI / Build Prometheus for all architectures (9) (push) Has been cancelled
CI / Check generated parser (push) Has been cancelled
CI / golangci-lint (push) Has been cancelled
CI / fuzzing (push) Has been cancelled
CI / codeql (push) Has been cancelled
Scorecards supply-chain security / Scorecards analysis (push) Has been cancelled
CI / Report status of build Prometheus for all architectures (push) Has been cancelled
CI / Publish main branch artifacts (push) Has been cancelled
CI / Publish release artefacts (push) Has been cancelled
CI / Publish UI on npm Registry (push) Has been cancelled

promqltest: Small formatting improvement for native histograms
This commit is contained in:
Björn Rabenstein 2025-01-29 18:43:38 +01:00 committed by GitHub
commit 3389cdf957
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 3 additions and 3 deletions

View file

@ -800,7 +800,7 @@ func (ev *evalCmd) compareResult(result parser.Value) error {
fp := v.Metric.Hash() fp := v.Metric.Hash()
if _, ok := ev.metrics[fp]; !ok { if _, ok := ev.metrics[fp]; !ok {
if v.H != nil { if v.H != nil {
return fmt.Errorf("unexpected metric %s in result, has value %v", v.Metric, v.H) return fmt.Errorf("unexpected metric %s in result, has value %s", v.Metric, HistogramTestExpression(v.H))
} }
return fmt.Errorf("unexpected metric %s in result, has value %v", v.Metric, v.F) return fmt.Errorf("unexpected metric %s in result, has value %v", v.Metric, v.F)
@ -838,7 +838,7 @@ func (ev *evalCmd) compareResult(result parser.Value) error {
} }
exp0 := ev.expected[0].vals[0] exp0 := ev.expected[0].vals[0]
if exp0.Histogram != nil { if exp0.Histogram != nil {
return fmt.Errorf("expected histogram %v but got %s", exp0.Histogram.TestExpression(), val.String()) return fmt.Errorf("expected histogram %s but got %s", exp0.Histogram.TestExpression(), val.String())
} }
if !almost.Equal(exp0.Value, val.V, defaultEpsilon) { if !almost.Equal(exp0.Value, val.V, defaultEpsilon) {
return fmt.Errorf("expected scalar %v but got %v", exp0.Value, val.V) return fmt.Errorf("expected scalar %v but got %v", exp0.Value, val.V)

View file

@ -247,7 +247,7 @@ load 5m
eval instant at 0m testmetric eval instant at 0m testmetric
`, `,
expectedError: `error in eval testmetric (line 5): unexpected metric {__name__="testmetric"} in result, has value {count:0, sum:0}`, expectedError: `error in eval testmetric (line 5): unexpected metric {__name__="testmetric"} in result, has value {{}}`,
}, },
"instant query, but result is missing a series": { "instant query, but result is missing a series": {
input: testData + ` input: testData + `