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.
208 lines
5.5 KiB
Go
208 lines
5.5 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package platform
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/hanzoteam/server/server/public/model"
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
const broadcastTest = "test_broadcast_hook"
|
|
|
|
type testBroadcastHook struct{}
|
|
|
|
func (h *testBroadcastHook) Process(msg *HookedWebSocketEvent, webConn *WebConn, args map[string]any) error {
|
|
if args["makes_changes"].(bool) {
|
|
changesMade, _ := msg.Get("changes_made").(int)
|
|
msg.Add("changes_made", changesMade+1)
|
|
}
|
|
|
|
return nil
|
|
}
|
|
|
|
func TestRunBroadcastHooks(t *testing.T) {
|
|
mainHelper.Parallel(t)
|
|
hub := &Hub{
|
|
broadcastHooks: map[string]BroadcastHook{
|
|
broadcastTest: &testBroadcastHook{},
|
|
},
|
|
}
|
|
webConn := &WebConn{}
|
|
|
|
t.Run("should not allocate a new object when no hooks are passed", func(t *testing.T) {
|
|
event := model.NewWebSocketEvent(model.WebsocketEventPosted, "", "", "", nil, "")
|
|
|
|
result := hub.runBroadcastHooks(event, webConn, nil, nil)
|
|
|
|
assert.Same(t, event, result)
|
|
})
|
|
|
|
t.Run("should not allocate a new object when a hook is not making changes", func(t *testing.T) {
|
|
event := model.NewWebSocketEvent(model.WebsocketEventPosted, "", "", "", nil, "")
|
|
|
|
hookIDs := []string{
|
|
broadcastTest,
|
|
}
|
|
hookArgs := []map[string]any{
|
|
{
|
|
"makes_changes": false,
|
|
},
|
|
}
|
|
|
|
result := hub.runBroadcastHooks(event, webConn, hookIDs, hookArgs)
|
|
|
|
assert.Same(t, event, result)
|
|
})
|
|
|
|
t.Run("should allocate a new object and remove when a hook makes changes", func(t *testing.T) {
|
|
event := model.NewWebSocketEvent(model.WebsocketEventPosted, "", "", "", nil, "")
|
|
|
|
hookIDs := []string{
|
|
broadcastTest,
|
|
}
|
|
hookArgs := []map[string]any{
|
|
{
|
|
"makes_changes": true,
|
|
},
|
|
}
|
|
|
|
result := hub.runBroadcastHooks(event, webConn, hookIDs, hookArgs)
|
|
|
|
assert.NotSame(t, event, result)
|
|
model.AssertNotSameMap(t, event.GetData(), result.GetData())
|
|
assert.Equal(t, map[string]any{}, event.GetData())
|
|
assert.Equal(t, result.GetData(), map[string]any{
|
|
"changes_made": 1,
|
|
})
|
|
})
|
|
|
|
t.Run("should not allocate a new object when multiple hooks are not making changes", func(t *testing.T) {
|
|
event := model.NewWebSocketEvent(model.WebsocketEventPosted, "", "", "", nil, "")
|
|
|
|
hookIDs := []string{
|
|
broadcastTest,
|
|
broadcastTest,
|
|
broadcastTest,
|
|
}
|
|
hookArgs := []map[string]any{
|
|
{
|
|
"makes_changes": false,
|
|
},
|
|
{
|
|
"makes_changes": false,
|
|
},
|
|
{
|
|
"makes_changes": false,
|
|
},
|
|
}
|
|
|
|
result := hub.runBroadcastHooks(event, webConn, hookIDs, hookArgs)
|
|
|
|
assert.Same(t, event, result)
|
|
})
|
|
|
|
t.Run("should be able to make changes from only one of make hooks", func(t *testing.T) {
|
|
event := model.NewWebSocketEvent(model.WebsocketEventPosted, "", "", "", nil, "")
|
|
|
|
var hookIDs []string
|
|
var hookArgs []map[string]any
|
|
for i := range 10 {
|
|
hookIDs = append(hookIDs, broadcastTest)
|
|
hookArgs = append(hookArgs, map[string]any{
|
|
"makes_changes": i == 6,
|
|
})
|
|
}
|
|
|
|
result := hub.runBroadcastHooks(event, webConn, hookIDs, hookArgs)
|
|
|
|
assert.NotSame(t, event, result)
|
|
model.AssertNotSameMap(t, event.GetData(), result.GetData())
|
|
assert.Equal(t, event.GetData(), map[string]any{})
|
|
assert.Equal(t, result.GetData(), map[string]any{
|
|
"changes_made": 1,
|
|
})
|
|
})
|
|
|
|
t.Run("should be able to make changes from multiple hooks", func(t *testing.T) {
|
|
event := model.NewWebSocketEvent(model.WebsocketEventPosted, "", "", "", nil, "")
|
|
|
|
var hookIDs []string
|
|
var hookArgs []map[string]any
|
|
for range 10 {
|
|
hookIDs = append(hookIDs, broadcastTest)
|
|
hookArgs = append(hookArgs, map[string]any{
|
|
"makes_changes": true,
|
|
})
|
|
}
|
|
|
|
result := hub.runBroadcastHooks(event, webConn, hookIDs, hookArgs)
|
|
|
|
assert.NotSame(t, event, result)
|
|
model.AssertNotSameMap(t, event.GetData(), result.GetData())
|
|
assert.Equal(t, event.GetData(), map[string]any{})
|
|
assert.Equal(t, result.GetData(), map[string]any{
|
|
"changes_made": 10,
|
|
})
|
|
})
|
|
|
|
t.Run("should not remove precomputed JSON when a hook doesn't make changes", func(t *testing.T) {
|
|
event := model.NewWebSocketEvent(model.WebsocketEventPosted, "", "", "", nil, "")
|
|
event = event.PrecomputeJSON()
|
|
|
|
// Ensure that the event has precomputed JSON because changes aren't included when ToJSON is called again
|
|
originalJSON, _ := event.ToJSON()
|
|
event.Add("data", 1234)
|
|
eventJSON, _ := event.ToJSON()
|
|
require.Equal(t, string(originalJSON), string(eventJSON))
|
|
|
|
hookIDs := []string{
|
|
broadcastTest,
|
|
}
|
|
hookArgs := []map[string]any{
|
|
{
|
|
"makes_changes": false,
|
|
},
|
|
}
|
|
|
|
result := hub.runBroadcastHooks(event, webConn, hookIDs, hookArgs)
|
|
|
|
eventJSON, _ = event.ToJSON()
|
|
assert.Equal(t, string(originalJSON), string(eventJSON))
|
|
|
|
resultJSON, _ := result.ToJSON()
|
|
assert.Equal(t, originalJSON, resultJSON)
|
|
})
|
|
|
|
t.Run("should remove precomputed JSON when a hook makes changes", func(t *testing.T) {
|
|
event := model.NewWebSocketEvent(model.WebsocketEventPosted, "", "", "", nil, "")
|
|
event = event.PrecomputeJSON()
|
|
|
|
// Ensure that the event has precomputed JSON because changes aren't included when ToJSON is called again
|
|
originalJSON, _ := event.ToJSON()
|
|
event.Add("data", 1234)
|
|
eventJSON, _ := event.ToJSON()
|
|
require.Equal(t, originalJSON, eventJSON)
|
|
|
|
hookIDs := []string{
|
|
broadcastTest,
|
|
}
|
|
hookArgs := []map[string]any{
|
|
{
|
|
"makes_changes": true,
|
|
},
|
|
}
|
|
|
|
result := hub.runBroadcastHooks(event, webConn, hookIDs, hookArgs)
|
|
|
|
eventJSON, _ = event.ToJSON()
|
|
assert.Equal(t, string(originalJSON), string(eventJSON))
|
|
|
|
resultJSON, _ := result.ToJSON()
|
|
assert.NotEqual(t, originalJSON, resultJSON)
|
|
})
|
|
}
|