From c02b489302808063267f68230d51e63c2013947e Mon Sep 17 00:00:00 2001 From: Hector Date: Wed, 9 Oct 2019 21:43:13 +0200 Subject: [PATCH] Mm 19006 (#12616) * Refactor to use structured logging * Properly formatted with gofmt * migrate tests to use testify * fix: swap expected and actual --- app/status_test.go | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/app/status_test.go b/app/status_test.go index 0a56c8158e..e805d548fb 100644 --- a/app/status_test.go +++ b/app/status_test.go @@ -7,6 +7,7 @@ import ( "testing" "github.com/mattermost/mattermost-server/model" + "github.com/stretchr/testify/require" ) func TestSaveStatus(t *testing.T) { @@ -30,11 +31,8 @@ func TestSaveStatus(t *testing.T) { th.App.SaveAndBroadcastStatus(status) after, err := th.App.GetStatus(user.Id) - if err != nil { - t.Fatalf("failed to get status after save: %v", err) - } else if after.Status != statusString { - t.Fatalf("failed to save status, got %v, expected %v", after.Status, statusString) - } + require.Nil(t, err, "failed to get status after save: %v", err) + require.Equal(t, statusString, after.Status, "failed to save status, got %v, expected %v", after.Status, statusString) }) } }