From 9f905304653f619bb8ff76d5c40c914cf80d4b12 Mon Sep 17 00:00:00 2001 From: Richard Connon Date: Thu, 5 Jul 2018 15:40:24 +0100 Subject: [PATCH] Check error on triggers with non-existent app test (#1112) --- api/datastore/datastoretest/test.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/api/datastore/datastoretest/test.go b/api/datastore/datastoretest/test.go index 631f874f2..87538405a 100644 --- a/api/datastore/datastoretest/test.go +++ b/api/datastore/datastoretest/test.go @@ -1234,6 +1234,9 @@ func RunTriggersTest(t *testing.T, dsf DataStoreFunc, rp ResourceProvider) { t.Run("non-existant app", func(t *testing.T) { nonMatchingFilter := &models.TriggerFilter{AppID: "notexist"} triggers, err := ds.GetTriggers(ctx, nonMatchingFilter) + if err != nil { + t.Fatalf("expecting no error, got: %s", err) + } if len(triggers.Items) != 0 && err == nil { t.Fatalf("expected empty trigger list and no error, but got list [%v] and err %s", triggers.Items, err) }