notifier: in tests use labels.FromStrings

Replacing code which assumes the internal structure of `Labels`.

Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
This commit is contained in:
Bryan Boreham 2022-03-09 22:21:29 +00:00 committed by Julien Pivotto
parent 735914f692
commit ac02cfcb79

View file

@ -243,7 +243,7 @@ func TestCustomDo(t *testing.T) {
func TestExternalLabels(t *testing.T) { func TestExternalLabels(t *testing.T) {
h := NewManager(&Options{ h := NewManager(&Options{
QueueCapacity: 3 * maxBatchSize, QueueCapacity: 3 * maxBatchSize,
ExternalLabels: labels.Labels{{Name: "a", Value: "b"}}, ExternalLabels: labels.FromStrings("a", "b"),
RelabelConfigs: []*relabel.Config{ RelabelConfigs: []*relabel.Config{
{ {
SourceLabels: model.LabelNames{"alertname"}, SourceLabels: model.LabelNames{"alertname"},
@ -570,7 +570,7 @@ func makeInputTargetGroup() *targetgroup.Group {
} }
func TestLabelsToOpenAPILabelSet(t *testing.T) { func TestLabelsToOpenAPILabelSet(t *testing.T) {
require.Equal(t, models.LabelSet{"aaa": "111", "bbb": "222"}, labelsToOpenAPILabelSet(labels.Labels{{Name: "aaa", Value: "111"}, {Name: "bbb", Value: "222"}})) require.Equal(t, models.LabelSet{"aaa": "111", "bbb": "222"}, labelsToOpenAPILabelSet(labels.FromStrings("aaa", "111", "bbb", "222")))
} }
// TestHangingNotifier validates that targets updates happen even when there are // TestHangingNotifier validates that targets updates happen even when there are