From e3f6b169271f26ca424f4a072b330d6842b6daf0 Mon Sep 17 00:00:00 2001 From: Arya Khochare <91268931+Aryakoste@users.noreply.github.com> Date: Mon, 4 Nov 2024 18:19:02 +0530 Subject: [PATCH] errcheck issues fixed (#29065) --- server/.golangci.yml | 1 - server/channels/app/opengraph_test.go | 6 ++++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/server/.golangci.yml b/server/.golangci.yml index 19db2f0ce8..23f9bed316 100644 --- a/server/.golangci.yml +++ b/server/.golangci.yml @@ -112,7 +112,6 @@ issues: channels/app/notification_push.go|\ channels/app/notification_test.go|\ channels/app/onboarding_test.go|\ - channels/app/opengraph_test.go|\ channels/app/permissions.go|\ channels/app/permissions_test.go|\ channels/app/platform/busy_test.go|\ diff --git a/server/channels/app/opengraph_test.go b/server/channels/app/opengraph_test.go index 3f42f60af1..fc15698f36 100644 --- a/server/channels/app/opengraph_test.go +++ b/server/channels/app/opengraph_test.go @@ -28,14 +28,16 @@ func BenchmarkForceHTMLEncodingToUTF8(b *testing.B) { r := forceHTMLEncodingToUTF8(strings.NewReader(HTML), ContentType) og := opengraph.NewOpenGraph() - og.ProcessHTML(r) + err := og.ProcessHTML(r) + require.NoError(b, err) } }) b.Run("without converting", func(b *testing.B) { for i := 0; i < b.N; i++ { og := opengraph.NewOpenGraph() - og.ProcessHTML(strings.NewReader(HTML)) + err := og.ProcessHTML(strings.NewReader(HTML)) + require.NoError(b, err) } }) }