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.
194 lines
6.1 KiB
Go
194 lines
6.1 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package api4
|
|
|
|
import (
|
|
"context"
|
|
"encoding/json"
|
|
"fmt"
|
|
"net/http"
|
|
"sort"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
|
|
"github.com/hanzoteam/server/server/public/model"
|
|
)
|
|
|
|
func TestGetSharedChannelRemotes(t *testing.T) {
|
|
th := setupForSharedChannels(t).InitBasic(t)
|
|
|
|
// Create remote clusters
|
|
remote1 := &model.RemoteCluster{
|
|
Name: "remote1",
|
|
DisplayName: "Remote Cluster 1",
|
|
SiteURL: "http://example.com",
|
|
CreatorId: th.BasicUser.Id,
|
|
Token: model.NewId(),
|
|
LastPingAt: model.GetMillis(),
|
|
}
|
|
remote1, appErr := th.App.AddRemoteCluster(remote1)
|
|
require.Nil(t, appErr)
|
|
|
|
remote2 := &model.RemoteCluster{
|
|
Name: "remote2",
|
|
DisplayName: "Remote Cluster 2",
|
|
SiteURL: "http://example.org",
|
|
CreatorId: th.BasicUser.Id,
|
|
Token: model.NewId(),
|
|
LastPingAt: model.GetMillis(),
|
|
}
|
|
remote2, appErr = th.App.AddRemoteCluster(remote2)
|
|
require.Nil(t, appErr)
|
|
|
|
// Create shared channel
|
|
channel1 := th.CreateChannelWithClientAndTeam(t, th.Client, model.ChannelTypeOpen, th.BasicTeam.Id)
|
|
sc1 := &model.SharedChannel{
|
|
ChannelId: channel1.Id,
|
|
TeamId: th.BasicTeam.Id,
|
|
Home: true,
|
|
ReadOnly: false,
|
|
ShareName: channel1.Name,
|
|
ShareDisplayName: channel1.DisplayName,
|
|
SharePurpose: channel1.Purpose,
|
|
ShareHeader: channel1.Header,
|
|
CreatorId: th.BasicUser.Id,
|
|
}
|
|
|
|
_, sErr := th.App.ShareChannel(th.Context, sc1)
|
|
require.NoError(t, sErr)
|
|
|
|
// Add remotes to channel1
|
|
scr1 := &model.SharedChannelRemote{
|
|
ChannelId: sc1.ChannelId,
|
|
RemoteId: remote1.RemoteId,
|
|
CreatorId: th.BasicUser.Id,
|
|
IsInviteAccepted: true,
|
|
IsInviteConfirmed: true,
|
|
}
|
|
_, sErr = th.App.SaveSharedChannelRemote(scr1)
|
|
require.NoError(t, sErr)
|
|
|
|
scr2 := &model.SharedChannelRemote{
|
|
ChannelId: sc1.ChannelId,
|
|
RemoteId: remote2.RemoteId,
|
|
CreatorId: th.BasicUser.Id,
|
|
IsInviteAccepted: true,
|
|
IsInviteConfirmed: true,
|
|
}
|
|
_, sErr = th.App.SaveSharedChannelRemote(scr2)
|
|
require.NoError(t, sErr)
|
|
|
|
// Test the API endpoint
|
|
url := fmt.Sprintf("/sharedchannels/%s/remotes", channel1.Id)
|
|
resp, err := th.Client.DoAPIGet(context.Background(), url, "")
|
|
require.NoError(t, err)
|
|
require.Equal(t, http.StatusOK, resp.StatusCode)
|
|
|
|
var result []*model.RemoteClusterInfo
|
|
err = json.NewDecoder(resp.Body).Decode(&result)
|
|
require.NoError(t, err)
|
|
|
|
// Verify response
|
|
require.NotNil(t, result)
|
|
require.Len(t, result, 2)
|
|
|
|
// Sort remote infos by display name for consistent testing
|
|
sort.Slice(result, func(i, j int) bool {
|
|
return result[i].DisplayName < result[j].DisplayName
|
|
})
|
|
|
|
// Verify the RemoteClusterInfo objects contain the expected data
|
|
assert.Equal(t, remote1.DisplayName, result[0].DisplayName)
|
|
assert.Equal(t, remote2.DisplayName, result[1].DisplayName)
|
|
|
|
// Should also contain other fields including remote_id for invite/uninvite
|
|
assert.NotEmpty(t, result[0].RemoteId)
|
|
assert.NotEmpty(t, result[1].RemoteId)
|
|
assert.Equal(t, remote1.RemoteId, result[0].RemoteId)
|
|
assert.Equal(t, remote2.RemoteId, result[1].RemoteId)
|
|
assert.NotEmpty(t, result[0].Name)
|
|
assert.NotEmpty(t, result[1].Name)
|
|
// Verify the Name matches the actual remote cluster name
|
|
assert.Equal(t, remote1.Name, result[0].Name)
|
|
assert.Equal(t, remote2.Name, result[1].Name)
|
|
assert.NotZero(t, result[0].LastPingAt)
|
|
assert.NotZero(t, result[1].LastPingAt)
|
|
|
|
// Test access control - user without permissions should not be able to access
|
|
user2 := th.CreateUser(t)
|
|
_, err = th.Client.Logout(context.Background())
|
|
require.NoError(t, err)
|
|
_, _, err = th.Client.Login(context.Background(), user2.Email, user2.Password)
|
|
require.NoError(t, err)
|
|
|
|
resp, err = th.Client.DoAPIGet(context.Background(), url, "")
|
|
require.Error(t, err)
|
|
require.Equal(t, http.StatusForbidden, resp.StatusCode)
|
|
}
|
|
|
|
func TestGetSharedChannelRemotes_ReturnsEmptyForNonSharedChannel(t *testing.T) {
|
|
th := setupForSharedChannels(t).InitBasic(t)
|
|
|
|
// Create a channel that is NOT shared
|
|
channel := th.CreateChannelWithClientAndTeam(t, th.Client, model.ChannelTypeOpen, th.BasicTeam.Id)
|
|
|
|
// Request remotes for non-shared channel - should return empty array, not error
|
|
url := fmt.Sprintf("/sharedchannels/%s/remotes", channel.Id)
|
|
resp, err := th.Client.DoAPIGet(context.Background(), url, "")
|
|
require.NoError(t, err)
|
|
require.Equal(t, http.StatusOK, resp.StatusCode)
|
|
|
|
var result []*model.RemoteClusterInfo
|
|
err = json.NewDecoder(resp.Body).Decode(&result)
|
|
require.NoError(t, err)
|
|
|
|
require.NotNil(t, result)
|
|
require.Empty(t, result)
|
|
}
|
|
|
|
func TestGetSharedChannelRemotes_ReturnsEmptyForStaleSharedChannelState(t *testing.T) {
|
|
th := setupForSharedChannels(t).InitBasic(t)
|
|
|
|
channel := th.CreateChannelWithClientAndTeam(t, th.Client, model.ChannelTypeOpen, th.BasicTeam.Id)
|
|
require.NoError(t, th.App.Srv().Store().Channel().SetShared(channel.Id, true))
|
|
|
|
remote, appErr := th.App.AddRemoteCluster(&model.RemoteCluster{
|
|
Name: "stale-remote",
|
|
DisplayName: "Stale Remote",
|
|
SiteURL: "http://stale.example.com",
|
|
CreatorId: th.BasicUser.Id,
|
|
Token: model.NewId(),
|
|
LastPingAt: model.GetMillis(),
|
|
})
|
|
require.Nil(t, appErr)
|
|
|
|
// Simulate stale DB state: the channel/remote rows remain, but the SharedChannels row is missing.
|
|
_, err := th.App.Srv().Store().SharedChannel().SaveRemote(&model.SharedChannelRemote{
|
|
ChannelId: channel.Id,
|
|
RemoteId: remote.RemoteId,
|
|
CreatorId: th.BasicUser.Id,
|
|
IsInviteAccepted: true,
|
|
IsInviteConfirmed: true,
|
|
})
|
|
require.NoError(t, err)
|
|
|
|
hasRemote, err := th.App.Srv().Store().SharedChannel().HasRemote(channel.Id, remote.RemoteId)
|
|
require.NoError(t, err)
|
|
require.True(t, hasRemote)
|
|
|
|
url := fmt.Sprintf("/sharedchannels/%s/remotes", channel.Id)
|
|
resp, err := th.Client.DoAPIGet(context.Background(), url, "")
|
|
require.NoError(t, err)
|
|
require.Equal(t, http.StatusOK, resp.StatusCode)
|
|
|
|
var result []*model.RemoteClusterInfo
|
|
err = json.NewDecoder(resp.Body).Decode(&result)
|
|
require.NoError(t, err)
|
|
|
|
require.NotNil(t, result)
|
|
require.Empty(t, result)
|
|
}
|