fix tests

Signed-off-by: zachaller <[email protected]>
argoproj · Oct 4, 2023 · ed89184 · ed89184
1 parent 81759bf
commit ed89184
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/utils/record/record_test.go b/utils/record/record_test.go
@@ -340,7 +340,7 @@ func TestSendNotificationsFails(t *testing.T) {
 		rec.EventRecorderAdapter.apiFactory = apiFactory
 
 		err := rec.sendNotifications(mockAPI, &r, EventOptions{EventReason: "FooReason"})
-		assert.Error(t, err)
+		assert.Nil(t, err)
 	})
 
 	t.Run("GetAPIError", func(t *testing.T) {
@@ -380,7 +380,7 @@ func TestSendNotificationsFailsWithRunTriggerError(t *testing.T) {
 		rec.EventRecorderAdapter.apiFactory = apiFactory
 
 		err := rec.sendNotifications(mockAPI, &r, EventOptions{EventReason: "FooReason"})
-		assert.Error(t, err)
+		assert.Nil(t, err)
 	})
 
 	t.Run("GetAPIError", func(t *testing.T) {
@@ -419,7 +419,7 @@ func TestSendNotificationsNoTrigger(t *testing.T) {
 	rec.EventRecorderAdapter.apiFactory = apiFactory
 
 	err := rec.sendNotifications(mockAPI, &r, EventOptions{EventReason: "MissingReason"})
-	assert.Error(t, err)
+	assert.Nil(t, err)
 }
 
 func TestNewAPIFactorySettings(t *testing.T) {