mirror of
https://github.com/hanzoai/team.git
synced 2026-09-30 01:57:35 +00:00
zeekay
8feb24ea1f
The server module named github.com/hanzoai/team/server/v8 — a path that resolves to the retired workspace's repository, not this one — while server/public still published github.com/mattermost/mattermost/server/public. Both are github.com/hanzoteam/server/... now, which is where this code actually lives. That second half is what made the binary unbootable. Upstream's public path stayed claimed by upstream, mattermost-plugin-ai imported it, and the build linked public/model twice; public/plugin's gob.Register then ran twice under one name and panicked at init, before main. Reverting the rename traded the brand for a working binary. Rebranding the plugin alongside it needs no such trade: the agents fork is github.com/hanzoteam/agents/v2 against our public module, so no module named mattermost/mattermost is in the graph at all. Measured: `go list -deps ./cmd/mattermost` names exactly one public/model, and it is ours.
334 lines
10 KiB
Go
334 lines
10 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package app
|
|
|
|
import (
|
|
"net/http"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
|
|
"github.com/hanzoteam/server/server/public/model"
|
|
"github.com/hanzoteam/server/server/v8/channels/testlib"
|
|
)
|
|
|
|
func TestSaveReactionForPost(t *testing.T) {
|
|
mainHelper.Parallel(t)
|
|
th := Setup(t).InitBasic(t)
|
|
|
|
post := th.CreatePost(t, th.BasicChannel)
|
|
reaction1, err := th.App.SaveReactionForPost(th.Context, &model.Reaction{
|
|
UserId: th.BasicUser.Id,
|
|
PostId: post.Id,
|
|
EmojiName: "cry",
|
|
})
|
|
require.NotNil(t, reaction1)
|
|
require.Nil(t, err)
|
|
reaction2, err := th.App.SaveReactionForPost(th.Context, &model.Reaction{
|
|
UserId: th.BasicUser.Id,
|
|
PostId: post.Id,
|
|
EmojiName: "smile",
|
|
})
|
|
require.NotNil(t, reaction2)
|
|
require.Nil(t, err)
|
|
reaction3, err := th.App.SaveReactionForPost(th.Context, &model.Reaction{
|
|
UserId: th.BasicUser.Id,
|
|
PostId: post.Id,
|
|
EmojiName: "rofl",
|
|
})
|
|
require.NotNil(t, reaction3)
|
|
require.Nil(t, err)
|
|
|
|
t.Run("should save reaction with mixed case emoji name", func(t *testing.T) {
|
|
reaction := &model.Reaction{
|
|
UserId: th.BasicUser.Id,
|
|
PostId: post.Id,
|
|
EmojiName: "sMiLe",
|
|
}
|
|
|
|
result, err := th.App.SaveReactionForPost(th.Context, reaction)
|
|
require.Nil(t, err)
|
|
require.NotNil(t, result)
|
|
require.Equal(t, "smile", result.EmojiName)
|
|
})
|
|
|
|
t.Run("should not add reaction if it does not exist on the system", func(t *testing.T) {
|
|
reaction := &model.Reaction{
|
|
UserId: th.BasicUser.Id,
|
|
PostId: th.BasicPost.Id,
|
|
EmojiName: "definitely-not-a-real-emoji",
|
|
}
|
|
|
|
result, err := th.App.SaveReactionForPost(th.Context, reaction)
|
|
require.NotNil(t, err)
|
|
require.Nil(t, result)
|
|
})
|
|
|
|
t.Run("should not add reaction if we are over the limit", func(t *testing.T) {
|
|
var originalLimit *int
|
|
th.UpdateConfig(t, func(cfg *model.Config) {
|
|
originalLimit = cfg.ServiceSettings.UniqueEmojiReactionLimitPerPost
|
|
*cfg.ServiceSettings.UniqueEmojiReactionLimitPerPost = 3
|
|
})
|
|
defer th.UpdateConfig(t, func(cfg *model.Config) {
|
|
cfg.ServiceSettings.UniqueEmojiReactionLimitPerPost = originalLimit
|
|
})
|
|
|
|
reaction := &model.Reaction{
|
|
UserId: th.BasicUser.Id,
|
|
PostId: post.Id,
|
|
EmojiName: "joy",
|
|
}
|
|
|
|
result, err := th.App.SaveReactionForPost(th.Context, reaction)
|
|
require.NotNil(t, err)
|
|
require.Nil(t, result)
|
|
})
|
|
|
|
t.Run("should always add reaction if we are over the limit but the reaction is not unique", func(t *testing.T) {
|
|
user := th.CreateUser(t)
|
|
|
|
var originalLimit *int
|
|
th.UpdateConfig(t, func(cfg *model.Config) {
|
|
originalLimit = cfg.ServiceSettings.UniqueEmojiReactionLimitPerPost
|
|
*cfg.ServiceSettings.UniqueEmojiReactionLimitPerPost = 3
|
|
})
|
|
defer th.UpdateConfig(t, func(cfg *model.Config) {
|
|
cfg.ServiceSettings.UniqueEmojiReactionLimitPerPost = originalLimit
|
|
})
|
|
|
|
reaction := &model.Reaction{
|
|
UserId: user.Id,
|
|
PostId: post.Id,
|
|
EmojiName: "cry",
|
|
}
|
|
|
|
result, err := th.App.SaveReactionForPost(th.Context, reaction)
|
|
require.Nil(t, err)
|
|
require.NotNil(t, result)
|
|
})
|
|
|
|
t.Run("cannot save reaction in restricted DM", func(t *testing.T) {
|
|
th.App.UpdateConfig(func(cfg *model.Config) {
|
|
*cfg.TeamSettings.RestrictDirectMessage = model.DirectMessageTeam
|
|
})
|
|
|
|
// Create a DM channel between two users who don't share a team
|
|
dmChannel := th.CreateDmChannel(t, th.BasicUser2)
|
|
|
|
// Ensure the two users do not share a team
|
|
teams, err := th.App.GetTeamsForUser(th.BasicUser.Id)
|
|
require.Nil(t, err)
|
|
for _, team := range teams {
|
|
teamErr := th.App.RemoveUserFromTeam(th.Context, team.Id, th.BasicUser.Id, th.SystemAdminUser.Id)
|
|
require.Nil(t, teamErr)
|
|
}
|
|
teams, err = th.App.GetTeamsForUser(th.BasicUser2.Id)
|
|
require.Nil(t, err)
|
|
for _, team := range teams {
|
|
teamErr := th.App.RemoveUserFromTeam(th.Context, team.Id, th.BasicUser2.Id, th.SystemAdminUser.Id)
|
|
require.Nil(t, teamErr)
|
|
}
|
|
|
|
// Create separate teams for each user
|
|
team1 := th.CreateTeam(t)
|
|
team2 := th.CreateTeam(t)
|
|
th.LinkUserToTeam(t, th.BasicUser, team1)
|
|
th.LinkUserToTeam(t, th.BasicUser2, team2)
|
|
|
|
// Create a post in the DM channel
|
|
post := &model.Post{
|
|
UserId: th.BasicUser.Id,
|
|
ChannelId: dmChannel.Id,
|
|
Message: "test post",
|
|
}
|
|
post, _, err = th.App.CreatePost(th.Context, post, dmChannel, model.CreatePostFlags{})
|
|
require.Nil(t, err)
|
|
|
|
reaction := &model.Reaction{
|
|
UserId: th.BasicUser.Id,
|
|
PostId: post.Id,
|
|
EmojiName: "smile",
|
|
}
|
|
|
|
_, appErr := th.App.SaveReactionForPost(th.Context, reaction)
|
|
require.NotNil(t, appErr)
|
|
require.Equal(t, "api.reaction.save.restricted_dm.error", appErr.Id)
|
|
require.Equal(t, http.StatusBadRequest, appErr.StatusCode)
|
|
|
|
// Reset config
|
|
th.App.UpdateConfig(func(cfg *model.Config) {
|
|
*cfg.TeamSettings.RestrictDirectMessage = model.DirectMessageAny
|
|
})
|
|
})
|
|
}
|
|
|
|
func TestDeleteReactionForPost(t *testing.T) {
|
|
mainHelper.Parallel(t)
|
|
th := Setup(t).InitBasic(t)
|
|
|
|
post := th.CreatePost(t, th.BasicChannel)
|
|
reaction, err := th.App.SaveReactionForPost(th.Context, &model.Reaction{
|
|
UserId: th.BasicUser.Id,
|
|
PostId: post.Id,
|
|
EmojiName: "smile",
|
|
})
|
|
require.NotNil(t, reaction)
|
|
require.Nil(t, err)
|
|
|
|
t.Run("should delete reaction with mixed case emoji name", func(t *testing.T) {
|
|
reactionToDelete := &model.Reaction{
|
|
UserId: th.BasicUser.Id,
|
|
PostId: post.Id,
|
|
EmojiName: "sMiLe",
|
|
}
|
|
|
|
appErr := th.App.DeleteReactionForPost(th.Context, reactionToDelete)
|
|
require.Nil(t, appErr)
|
|
|
|
reactions, err := th.App.GetReactionsForPost(post.Id)
|
|
require.Nil(t, err)
|
|
require.Empty(t, reactions)
|
|
})
|
|
}
|
|
|
|
func TestDeleteReactionForPostWithRestrictedDM(t *testing.T) {
|
|
mainHelper.Parallel(t)
|
|
t.Run("cannot delete reaction in restricted DM", func(t *testing.T) {
|
|
mainHelper.Parallel(t)
|
|
th := Setup(t).InitBasic(t)
|
|
|
|
th.App.UpdateConfig(func(cfg *model.Config) {
|
|
*cfg.TeamSettings.RestrictDirectMessage = model.DirectMessageTeam
|
|
})
|
|
|
|
// Create a DM channel between two users who don't share a team
|
|
dmChannel := th.CreateDmChannel(t, th.BasicUser2)
|
|
|
|
// Ensure the two users do not share a team
|
|
teams, err := th.App.GetTeamsForUser(th.BasicUser.Id)
|
|
require.Nil(t, err)
|
|
for _, team := range teams {
|
|
teamErr := th.App.RemoveUserFromTeam(th.Context, team.Id, th.BasicUser.Id, th.SystemAdminUser.Id)
|
|
require.Nil(t, teamErr)
|
|
}
|
|
teams, err = th.App.GetTeamsForUser(th.BasicUser2.Id)
|
|
require.Nil(t, err)
|
|
for _, team := range teams {
|
|
teamErr := th.App.RemoveUserFromTeam(th.Context, team.Id, th.BasicUser2.Id, th.SystemAdminUser.Id)
|
|
require.Nil(t, teamErr)
|
|
}
|
|
|
|
// Create separate teams for each user
|
|
team1 := th.CreateTeam(t)
|
|
team2 := th.CreateTeam(t)
|
|
th.LinkUserToTeam(t, th.BasicUser, team1)
|
|
th.LinkUserToTeam(t, th.BasicUser2, team2)
|
|
|
|
// Create a post in the DM channel
|
|
post := &model.Post{
|
|
UserId: th.BasicUser.Id,
|
|
ChannelId: dmChannel.Id,
|
|
Message: "test post",
|
|
}
|
|
post, _, err = th.App.CreatePost(th.Context, post, dmChannel, model.CreatePostFlags{})
|
|
require.Nil(t, err)
|
|
|
|
reaction := &model.Reaction{
|
|
UserId: th.BasicUser.Id,
|
|
PostId: post.Id,
|
|
EmojiName: "smile",
|
|
}
|
|
|
|
appErr := th.App.DeleteReactionForPost(th.Context, reaction)
|
|
require.NotNil(t, appErr)
|
|
require.Equal(t, "api.reaction.delete.restricted_dm.error", appErr.Id)
|
|
require.Equal(t, http.StatusBadRequest, appErr.StatusCode)
|
|
|
|
// Reset config
|
|
th.App.UpdateConfig(func(cfg *model.Config) {
|
|
*cfg.TeamSettings.RestrictDirectMessage = model.DirectMessageAny
|
|
})
|
|
})
|
|
}
|
|
|
|
func TestSharedChannelSyncForReactionActions(t *testing.T) {
|
|
mainHelper.Parallel(t)
|
|
t.Run("adding a reaction in a shared channel performs a content sync when sync service is running on that node", func(t *testing.T) {
|
|
th := setupSharedChannels(t).InitBasic(t)
|
|
|
|
sharedChannelService := NewMockSharedChannelService(th.Server.GetSharedChannelSyncService())
|
|
th.Server.SetSharedChannelSyncService(sharedChannelService)
|
|
testCluster := &testlib.FakeClusterInterface{}
|
|
th.Server.Platform().SetCluster(testCluster)
|
|
|
|
user := th.BasicUser
|
|
|
|
channel := th.CreateChannel(t, th.BasicTeam, WithShared(true))
|
|
|
|
post, _, err := th.App.CreatePost(th.Context, &model.Post{
|
|
UserId: user.Id,
|
|
ChannelId: channel.Id,
|
|
Message: "Hello folks",
|
|
}, channel, model.CreatePostFlags{SetOnline: true})
|
|
require.Nil(t, err, "Creating a post should not error")
|
|
|
|
reaction := &model.Reaction{
|
|
UserId: user.Id,
|
|
PostId: post.Id,
|
|
EmojiName: "+1",
|
|
}
|
|
|
|
_, err = th.App.SaveReactionForPost(th.Context, reaction)
|
|
require.Nil(t, err, "Adding a reaction should not error")
|
|
assert.Len(t, sharedChannelService.channelNotifications, 2)
|
|
assert.Equal(t, channel.Id, sharedChannelService.channelNotifications[0])
|
|
assert.Equal(t, channel.Id, sharedChannelService.channelNotifications[1])
|
|
})
|
|
|
|
t.Run("removing a reaction in a shared channel performs a content sync when sync service is running on that node", func(t *testing.T) {
|
|
th := setupSharedChannels(t).InitBasic(t)
|
|
|
|
sharedChannelService := NewMockSharedChannelService(th.Server.GetSharedChannelSyncService())
|
|
th.Server.SetSharedChannelSyncService(sharedChannelService)
|
|
testCluster := &testlib.FakeClusterInterface{}
|
|
th.Server.Platform().SetCluster(testCluster)
|
|
|
|
user := th.BasicUser
|
|
|
|
channel := th.CreateChannel(t, th.BasicTeam, WithShared(true))
|
|
|
|
post, _, err := th.App.CreatePost(th.Context, &model.Post{
|
|
UserId: user.Id,
|
|
ChannelId: channel.Id,
|
|
Message: "Hello folks",
|
|
}, channel, model.CreatePostFlags{SetOnline: true})
|
|
require.Nil(t, err, "Creating a post should not error")
|
|
|
|
reaction := &model.Reaction{
|
|
UserId: user.Id,
|
|
PostId: post.Id,
|
|
EmojiName: "+1",
|
|
}
|
|
|
|
err = th.App.DeleteReactionForPost(th.Context, reaction)
|
|
require.Nil(t, err, "Adding a reaction should not error")
|
|
assert.Len(t, sharedChannelService.channelNotifications, 2)
|
|
assert.Equal(t, channel.Id, sharedChannelService.channelNotifications[0])
|
|
assert.Equal(t, channel.Id, sharedChannelService.channelNotifications[1])
|
|
})
|
|
}
|
|
|
|
func (th *TestHelper) UpdateConfig(tb testing.TB, f func(*model.Config)) {
|
|
if th.ConfigStore.IsReadOnly() {
|
|
return
|
|
}
|
|
old := th.ConfigStore.Get()
|
|
updated := old.Clone()
|
|
f(updated)
|
|
_, _, err := th.ConfigStore.Set(updated)
|
|
require.NoError(tb, err)
|
|
}
|