From 748f8227e364c87697f549787888cf1cbafd5d7d Mon Sep 17 00:00:00 2001 From: Ben Schumacher Date: Fri, 11 Apr 2025 10:42:16 +0200 Subject: [PATCH] Fix errcheck error in plugin_test.go by properly handling error from th.App.ch.RemovePlugin (#30608) --- server/.golangci.yml | 1 - server/channels/app/plugin_test.go | 3 ++- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/server/.golangci.yml b/server/.golangci.yml index 05041807ef..2f3c7ee32a 100644 --- a/server/.golangci.yml +++ b/server/.golangci.yml @@ -103,7 +103,6 @@ issues: channels/app/platform/session.go|\ channels/app/platform/status.go|\ channels/app/platform/web_hub_test.go|\ - channels/app/plugin_test.go|\ channels/app/post_helpers_test.go|\ channels/app/post_test.go|\ channels/app/slack.go|\ diff --git a/server/channels/app/plugin_test.go b/server/channels/app/plugin_test.go index dba66a1dda..e3f9eb214f 100644 --- a/server/channels/app/plugin_test.go +++ b/server/channels/app/plugin_test.go @@ -1355,7 +1355,8 @@ func TestProcessPrepackagedPlugins(t *testing.T) { require.Len(t, transitionalPlugins, 1) expectPrepackagedPlugin(t, "testplugin", "0.0.1", transitionalPlugins[0]) - th.App.ch.RemovePlugin("testplugin") + appErr := th.App.ch.RemovePlugin("testplugin") + require.Nil(t, appErr) pluginStatus, err := env.Statuses() require.NoError(t, err)