From cb8f152089cf81f788b88cda898a0830d62a172a Mon Sep 17 00:00:00 2001
From: Ivy Gesare <83957195+Gesare5@users.noreply.github.com>
Date: Tue, 5 Nov 2024 14:13:41 +0300
Subject: [PATCH] [MM-61094] Fix errcheck issues in
 server/channels/app/app_test.go (#29124)

---
 server/.golangci.yml            | 1 -
 server/channels/app/app_test.go | 6 ++++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/server/.golangci.yml b/server/.golangci.yml
index e6e1c26cc1..a80c6c9fc4 100644
--- a/server/.golangci.yml
+++ b/server/.golangci.yml
@@ -84,7 +84,6 @@ issues:
         channels/api4/team_test.go|\
         channels/api4/user_test.go|\
         channels/api4/websocket_test.go|\
-        channels/app/app_test.go|\
         channels/app/auto_responder_test.go|\
         channels/app/bot_test.go|\
         channels/app/brand.go|\
diff --git a/server/channels/app/app_test.go b/server/channels/app/app_test.go
index adfa82f577..365454956c 100644
--- a/server/channels/app/app_test.go
+++ b/server/channels/app/app_test.go
@@ -85,7 +85,8 @@ func TestDoAdvancedPermissionsMigration(t *testing.T) {
 
 	th.ResetRoleMigration()
 
-	th.App.DoAdvancedPermissionsMigration()
+	err := th.App.DoAdvancedPermissionsMigration()
+	require.NoError(t, err)
 
 	roleNames := []string{
 		"system_user",
@@ -232,7 +233,8 @@ func TestDoAdvancedPermissionsMigration(t *testing.T) {
 	th.App.Srv().SetLicense(model.NewTestLicense())
 
 	// Check the migration doesn't change anything if run again.
-	th.App.DoAdvancedPermissionsMigration()
+	err = th.App.DoAdvancedPermissionsMigration()
+	require.NoError(t, err)
 
 	roles2, err2 := th.App.GetRolesByNames(roleNames)
 	assert.Nil(t, err2)