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.
98 lines
2.7 KiB
Go
98 lines
2.7 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package telemetry
|
|
|
|
import (
|
|
"errors"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/mock"
|
|
"github.com/stretchr/testify/require"
|
|
|
|
"github.com/hanzoteam/server/server/public/model"
|
|
"github.com/hanzoteam/server/server/public/shared/mlog"
|
|
storeMocks "github.com/hanzoteam/server/server/v8/channels/store/storetest/mocks"
|
|
"github.com/hanzoteam/server/server/v8/platform/services/telemetry/mocks"
|
|
)
|
|
|
|
func TestEnsureServerID(t *testing.T) {
|
|
t.Run("test ID in database and does not run twice", func(t *testing.T) {
|
|
storeMock := &storeMocks.Store{}
|
|
|
|
systemStore := storeMocks.SystemStore{}
|
|
returnValue := &model.System{
|
|
Name: model.SystemServerId,
|
|
Value: "test",
|
|
}
|
|
systemStore.On("InsertIfExists", mock.AnythingOfType("*model.System")).Return(returnValue, nil).Once()
|
|
|
|
storeMock.On("System").Return(&systemStore)
|
|
|
|
serverIfaceMock := &mocks.ServerIface{}
|
|
|
|
testLogger, _ := mlog.NewLogger()
|
|
|
|
telemetryService, err := New(serverIfaceMock, storeMock, testLogger)
|
|
require.NoError(t, err)
|
|
|
|
assert.Equal(t, "test", telemetryService.ServerID)
|
|
|
|
telemetryService.ensureServerID()
|
|
assert.Equal(t, "test", telemetryService.ServerID)
|
|
|
|
// No more calls to the store if we try to ensure it again
|
|
telemetryService.ensureServerID()
|
|
assert.Equal(t, "test", telemetryService.ServerID)
|
|
})
|
|
|
|
t.Run("new test ID created", func(t *testing.T) {
|
|
storeMock := &storeMocks.Store{}
|
|
|
|
systemStore := storeMocks.SystemStore{}
|
|
returnValue := &model.System{
|
|
Name: model.SystemServerId,
|
|
}
|
|
|
|
var generatedID string
|
|
systemStore.On("InsertIfExists", mock.AnythingOfType("*model.System")).Return(returnValue, nil).Once().Run(func(args mock.Arguments) {
|
|
s := args.Get(0).(*model.System)
|
|
returnValue.Value = s.Value
|
|
generatedID = s.Value
|
|
})
|
|
storeMock.On("System").Return(&systemStore)
|
|
|
|
serverIfaceMock := &mocks.ServerIface{}
|
|
|
|
testLogger, _ := mlog.NewLogger()
|
|
|
|
telemetryService, err := New(serverIfaceMock, storeMock, testLogger)
|
|
require.NoError(t, err)
|
|
|
|
assert.Equal(t, generatedID, telemetryService.ServerID)
|
|
})
|
|
|
|
t.Run("fail to save test ID", func(t *testing.T) {
|
|
if testing.Short() {
|
|
t.Skip("skipping test in short mode.")
|
|
}
|
|
|
|
storeMock := &storeMocks.Store{}
|
|
|
|
systemStore := storeMocks.SystemStore{}
|
|
|
|
insertError := errors.New("insert error")
|
|
systemStore.On("InsertIfExists", mock.AnythingOfType("*model.System")).Return(nil, insertError).Times(DBAccessAttempts)
|
|
|
|
storeMock.On("System").Return(&systemStore)
|
|
|
|
serverIfaceMock := &mocks.ServerIface{}
|
|
|
|
testLogger, _ := mlog.NewLogger()
|
|
|
|
_, err := New(serverIfaceMock, storeMock, testLogger)
|
|
require.Error(t, err)
|
|
})
|
|
}
|