Chore: Rewrite models alert test to standard library (#30021)

pkg/models/alert_test.go

@@ -4,41 +4,38 @@ import (
 	"testing"
 	"testing"
 
 
 	"github.com/grafana/grafana/pkg/components/simplejson"
 	"github.com/grafana/grafana/pkg/components/simplejson"
-	. "github.com/smartystreets/goconvey/convey"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 )
 )
 
 
-func TestAlertingModelTest(t *testing.T) {
-	Convey("Testing Alerting model", t, func() {
-		json1, err := simplejson.NewJson([]byte(`{ "field": "value" }`))
-		So(err, ShouldBeNil)
-		json2, err := simplejson.NewJson([]byte(`{ "field": "value" }`))
-		So(err, ShouldBeNil)
+func TestAlert_ContainsUpdates(t *testing.T) {
+	settings, err := simplejson.NewJson([]byte(`{ "field": "value" }`))
+	require.NoError(t, err)
 
 
-		rule1 := &Alert{
-			Settings: json1,
-			Name:     "Namn",
+	alert1 := &Alert{
+		Settings: settings,
+		Name:     "Name",
 		Message:  "Message",
 		Message:  "Message",
 	}
 	}
 
 
-		rule2 := &Alert{
-			Settings: json2,
-			Name:     "Namn",
+	alert2 := &Alert{
+		Settings: settings,
+		Name:     "Name",
 		Message:  "Message",
 		Message:  "Message",
 	}
 	}
 
 
-		Convey("Testing AlertRule equals", func() {
-			So(rule1.ContainsUpdates(rule2), ShouldBeFalse)
-		})
+	assert.False(t, alert1.ContainsUpdates(alert2))
 
 
-		Convey("Changing the expression should contain update", func() {
-			json2, err := simplejson.NewJson([]byte(`{ "field": "newValue" }`))
-			So(err, ShouldBeNil)
-			rule1.Settings = json2
-			So(rule1.ContainsUpdates(rule2), ShouldBeTrue)
-		})
+	settingsUpdated, err := simplejson.NewJson([]byte(`{ "field": "newValue" }`))
+	require.NoError(t, err)
 
 
-		Convey("Should parse alertRule tags correctly", func() {
-			json2, err := simplejson.NewJson([]byte(`{
+	alert2.Settings = settingsUpdated
+
+	assert.True(t, alert1.ContainsUpdates(alert2))
+}
+
+func TestAlert_GetTagsFromSettings(t *testing.T) {
+	settings, err := simplejson.NewJson([]byte(`{
 		"field": "value",
 		"field": "value",
 		"alertRuleTags": {
 		"alertRuleTags": {
 			"foo": "bar",
 			"foo": "bar",
@@ -46,19 +43,20 @@ func TestAlertingModelTest(t *testing.T) {
 			"tagMap": { "mapValue": "value" }
 			"tagMap": { "mapValue": "value" }
 		}
 		}
 	}`))
 	}`))
-			So(err, ShouldBeNil)
-			rule1.Settings = json2
+	require.NoError(t, err)
+
+	alert := &Alert{
+		Settings: settings,
+		Name:     "Name",
+		Message:  "Message",
+	}
+
 	expectedTags := []*Tag{
 	expectedTags := []*Tag{
 		{Id: 0, Key: "foo", Value: "bar"},
 		{Id: 0, Key: "foo", Value: "bar"},
 		{Id: 0, Key: "waldo", Value: "fred"},
 		{Id: 0, Key: "waldo", Value: "fred"},
 		{Id: 0, Key: "tagMap", Value: ""},
 		{Id: 0, Key: "tagMap", Value: ""},
 	}
 	}
-			actualTags := rule1.GetTagsFromSettings()
+	actualTags := alert.GetTagsFromSettings()
 
 
-			So(len(actualTags), ShouldEqual, len(expectedTags))
-			for _, tag := range expectedTags {
-				So(ContainsTag(actualTags, tag), ShouldBeTrue)
-			}
-		})
-	})
+	assert.ElementsMatch(t, actualTags, expectedTags)
 }
 }