Skip to content

Commit a6aead8

Browse files
Fixing linter and applying PR feedback
1 parent 047e895 commit a6aead8

3 files changed

Lines changed: 5 additions & 1 deletion

File tree

server/subscribe.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ const (
4141
CommentVisibility = "commentVisibility"
4242
TeamFilter = "teamField"
4343
CommentVisibilityGroupType = "group"
44-
maxDMGMChannelMembers = 200
44+
maxDMGMChannelMembers = 200
4545
)
4646

4747
type FieldFilter struct {

server/subscribe_test.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1807,6 +1807,7 @@ func TestRemoveSubscriptionsForChannelIndexDrift(t *testing.T) {
18071807

18081808
err = p.removeSubscriptionsForChannel(testInstance1.GetID(), dmChannelID)
18091809
assert.NoError(t, err)
1810+
api.AssertExpectations(t)
18101811
}
18111812

18121813
func TestRemoveSubscriptionsForChannel(t *testing.T) {
@@ -2001,6 +2002,8 @@ func TestCleanupDMSubscriptionsOnDisconnect(t *testing.T) {
20012002
}), mock.AnythingOfType("model.PluginKVSetOptions")).Return(true, nil)
20022003

20032004
p.cleanupDMSubscriptionsOnDisconnect(testInstance1.GetID(), mattermostUserID)
2005+
2006+
api.AssertExpectations(t)
20042007
})
20052008

20062009
t.Run("keeps GM subscription when another member is still connected", func(t *testing.T) {

server/webhook_worker_test.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,7 @@ func TestWebhookWorkerDeliveryGuard(t *testing.T) {
9393
require.NoError(t, err)
9494

9595
api.AssertNotCalled(t, "CreatePost", mock.Anything)
96+
api.AssertExpectations(t)
9697
})
9798

9899
t.Run("delivers to DM when a member is still connected", func(t *testing.T) {

0 commit comments

Comments
 (0)