2022-07-14 14:59:59 -05:00
|
|
|
package state
|
|
|
|
|
|
|
|
import (
|
|
|
|
"context"
|
|
|
|
"fmt"
|
|
|
|
"net/url"
|
|
|
|
"testing"
|
|
|
|
|
|
|
|
"github.com/grafana/grafana-plugin-sdk-go/data"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
|
|
|
|
|
|
"github.com/grafana/grafana/pkg/infra/log"
|
|
|
|
"github.com/grafana/grafana/pkg/services/ngalert/eval"
|
|
|
|
"github.com/grafana/grafana/pkg/services/ngalert/models"
|
|
|
|
"github.com/grafana/grafana/pkg/util"
|
|
|
|
)
|
|
|
|
|
|
|
|
func Test_getOrCreate(t *testing.T) {
|
2022-10-06 14:30:12 -05:00
|
|
|
url := &url.URL{
|
2022-07-14 14:59:59 -05:00
|
|
|
Scheme: "http",
|
|
|
|
Host: "localhost:3000",
|
|
|
|
Path: "/test",
|
2022-10-06 14:30:12 -05:00
|
|
|
}
|
|
|
|
l := log.New("test")
|
|
|
|
c := newCache()
|
2022-07-14 14:59:59 -05:00
|
|
|
|
|
|
|
generateRule := models.AlertRuleGen(models.WithNotEmptyLabels(5, "rule-"))
|
|
|
|
|
|
|
|
t.Run("should combine all labels", func(t *testing.T) {
|
|
|
|
rule := generateRule()
|
|
|
|
|
|
|
|
extraLabels := models.GenerateAlertLabels(5, "extra-")
|
|
|
|
result := eval.Result{
|
|
|
|
Instance: models.GenerateAlertLabels(5, "result-"),
|
|
|
|
}
|
2022-10-06 14:30:12 -05:00
|
|
|
state := c.getOrCreate(context.Background(), l, rule, result, extraLabels, url)
|
2022-07-14 14:59:59 -05:00
|
|
|
for key, expected := range extraLabels {
|
|
|
|
require.Equal(t, expected, state.Labels[key])
|
|
|
|
}
|
|
|
|
assert.Len(t, state.Labels, len(extraLabels)+len(rule.Labels)+len(result.Instance))
|
|
|
|
for key, expected := range extraLabels {
|
|
|
|
assert.Equal(t, expected, state.Labels[key])
|
|
|
|
}
|
|
|
|
for key, expected := range rule.Labels {
|
|
|
|
assert.Equal(t, expected, state.Labels[key])
|
|
|
|
}
|
|
|
|
for key, expected := range result.Instance {
|
|
|
|
assert.Equal(t, expected, state.Labels[key])
|
|
|
|
}
|
|
|
|
})
|
|
|
|
t.Run("extra labels should take precedence over rule and result labels", func(t *testing.T) {
|
|
|
|
rule := generateRule()
|
|
|
|
|
|
|
|
extraLabels := models.GenerateAlertLabels(2, "extra-")
|
|
|
|
|
|
|
|
result := eval.Result{
|
|
|
|
Instance: models.GenerateAlertLabels(5, "result-"),
|
|
|
|
}
|
|
|
|
for key := range extraLabels {
|
|
|
|
rule.Labels[key] = "rule-" + util.GenerateShortUID()
|
|
|
|
result.Instance[key] = "result-" + util.GenerateShortUID()
|
|
|
|
}
|
|
|
|
|
2022-10-06 14:30:12 -05:00
|
|
|
state := c.getOrCreate(context.Background(), l, rule, result, extraLabels, url)
|
2022-07-14 14:59:59 -05:00
|
|
|
for key, expected := range extraLabels {
|
|
|
|
require.Equal(t, expected, state.Labels[key])
|
|
|
|
}
|
|
|
|
})
|
|
|
|
t.Run("rule labels should take precedence over result labels", func(t *testing.T) {
|
|
|
|
rule := generateRule()
|
|
|
|
|
|
|
|
extraLabels := models.GenerateAlertLabels(2, "extra-")
|
|
|
|
|
|
|
|
result := eval.Result{
|
|
|
|
Instance: models.GenerateAlertLabels(5, "result-"),
|
|
|
|
}
|
|
|
|
for key := range rule.Labels {
|
|
|
|
result.Instance[key] = "result-" + util.GenerateShortUID()
|
|
|
|
}
|
2022-10-06 14:30:12 -05:00
|
|
|
state := c.getOrCreate(context.Background(), l, rule, result, extraLabels, url)
|
2022-07-14 14:59:59 -05:00
|
|
|
for key, expected := range rule.Labels {
|
|
|
|
require.Equal(t, expected, state.Labels[key])
|
|
|
|
}
|
|
|
|
})
|
|
|
|
t.Run("rule labels should be able to be expanded with result and extra labels", func(t *testing.T) {
|
|
|
|
result := eval.Result{
|
|
|
|
Instance: models.GenerateAlertLabels(5, "result-"),
|
|
|
|
}
|
|
|
|
rule := generateRule()
|
|
|
|
|
|
|
|
extraLabels := models.GenerateAlertLabels(2, "extra-")
|
|
|
|
|
|
|
|
labelTemplates := make(data.Labels)
|
|
|
|
for key := range extraLabels {
|
|
|
|
labelTemplates["rule-"+key] = fmt.Sprintf("{{ with (index .Labels \"%s\") }}{{.}}{{end}}", key)
|
|
|
|
}
|
|
|
|
for key := range result.Instance {
|
|
|
|
labelTemplates["rule-"+key] = fmt.Sprintf("{{ with (index .Labels \"%s\") }}{{.}}{{end}}", key)
|
|
|
|
}
|
|
|
|
rule.Labels = labelTemplates
|
|
|
|
|
2022-10-06 14:30:12 -05:00
|
|
|
state := c.getOrCreate(context.Background(), l, rule, result, extraLabels, url)
|
2022-07-14 14:59:59 -05:00
|
|
|
for key, expected := range extraLabels {
|
|
|
|
assert.Equal(t, expected, state.Labels["rule-"+key])
|
|
|
|
}
|
|
|
|
for key, expected := range result.Instance {
|
|
|
|
assert.Equal(t, expected, state.Labels["rule-"+key])
|
|
|
|
}
|
|
|
|
})
|
2023-03-06 11:08:00 -06:00
|
|
|
|
2022-07-14 14:59:59 -05:00
|
|
|
t.Run("rule annotations should be able to be expanded with result and extra labels", func(t *testing.T) {
|
|
|
|
result := eval.Result{
|
|
|
|
Instance: models.GenerateAlertLabels(5, "result-"),
|
|
|
|
}
|
|
|
|
|
|
|
|
rule := generateRule()
|
|
|
|
|
|
|
|
extraLabels := models.GenerateAlertLabels(2, "extra-")
|
|
|
|
|
|
|
|
annotationTemplates := make(data.Labels)
|
|
|
|
for key := range extraLabels {
|
|
|
|
annotationTemplates["rule-"+key] = fmt.Sprintf("{{ with (index .Labels \"%s\") }}{{.}}{{end}}", key)
|
|
|
|
}
|
|
|
|
for key := range result.Instance {
|
|
|
|
annotationTemplates["rule-"+key] = fmt.Sprintf("{{ with (index .Labels \"%s\") }}{{.}}{{end}}", key)
|
|
|
|
}
|
|
|
|
rule.Annotations = annotationTemplates
|
|
|
|
|
2022-10-06 14:30:12 -05:00
|
|
|
state := c.getOrCreate(context.Background(), l, rule, result, extraLabels, url)
|
2022-07-14 14:59:59 -05:00
|
|
|
for key, expected := range extraLabels {
|
|
|
|
assert.Equal(t, expected, state.Annotations["rule-"+key])
|
|
|
|
}
|
|
|
|
for key, expected := range result.Instance {
|
|
|
|
assert.Equal(t, expected, state.Annotations["rule-"+key])
|
|
|
|
}
|
|
|
|
})
|
2023-03-06 11:08:00 -06:00
|
|
|
|
|
|
|
t.Run("expected Reduce and Math expression values", func(t *testing.T) {
|
|
|
|
result := eval.Result{
|
|
|
|
Instance: models.GenerateAlertLabels(5, "result-"),
|
|
|
|
Values: map[string]eval.NumberValueCapture{
|
|
|
|
"A": eval.NumberValueCapture{Var: "A", Value: util.Pointer(1.0)},
|
|
|
|
"B": eval.NumberValueCapture{Var: "B", Value: util.Pointer(2.0)},
|
|
|
|
},
|
|
|
|
}
|
|
|
|
rule := generateRule()
|
|
|
|
|
|
|
|
state := c.getOrCreate(context.Background(), l, rule, result, nil, url)
|
|
|
|
assert.Equal(t, map[string]float64{"A": 1, "B": 2}, state.Values)
|
|
|
|
})
|
|
|
|
|
|
|
|
t.Run("expected Classic Condition values", func(t *testing.T) {
|
|
|
|
result := eval.Result{
|
|
|
|
Instance: models.GenerateAlertLabels(5, "result-"),
|
|
|
|
Values: map[string]eval.NumberValueCapture{
|
|
|
|
"B0": eval.NumberValueCapture{Var: "B", Value: util.Pointer(1.0)},
|
|
|
|
"B1": eval.NumberValueCapture{Var: "B", Value: util.Pointer(2.0)},
|
|
|
|
},
|
|
|
|
}
|
|
|
|
rule := generateRule()
|
|
|
|
|
|
|
|
state := c.getOrCreate(context.Background(), l, rule, result, nil, url)
|
|
|
|
assert.Equal(t, map[string]float64{"B0": 1, "B1": 2}, state.Values)
|
|
|
|
})
|
2022-07-14 14:59:59 -05:00
|
|
|
}
|
|
|
|
|
|
|
|
func Test_mergeLabels(t *testing.T) {
|
|
|
|
t.Run("merges two maps", func(t *testing.T) {
|
|
|
|
a := models.GenerateAlertLabels(5, "set1-")
|
|
|
|
b := models.GenerateAlertLabels(5, "set2-")
|
|
|
|
|
|
|
|
result := mergeLabels(a, b)
|
|
|
|
require.Len(t, result, len(a)+len(b))
|
|
|
|
for key, val := range a {
|
|
|
|
require.Equal(t, val, result[key])
|
|
|
|
}
|
|
|
|
for key, val := range b {
|
|
|
|
require.Equal(t, val, result[key])
|
|
|
|
}
|
|
|
|
})
|
|
|
|
t.Run("first set take precedence if conflict", func(t *testing.T) {
|
|
|
|
a := models.GenerateAlertLabels(5, "set1-")
|
|
|
|
b := models.GenerateAlertLabels(5, "set2-")
|
|
|
|
c := b.Copy()
|
|
|
|
for key, val := range a {
|
|
|
|
c[key] = "set2-" + val
|
|
|
|
}
|
|
|
|
|
|
|
|
result := mergeLabels(a, c)
|
|
|
|
require.Len(t, result, len(a)+len(b))
|
|
|
|
for key, val := range a {
|
|
|
|
require.Equal(t, val, result[key])
|
|
|
|
}
|
|
|
|
for key, val := range b {
|
|
|
|
require.Equal(t, val, result[key])
|
|
|
|
}
|
|
|
|
})
|
|
|
|
}
|