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.
57 lines
1.8 KiB
Go
57 lines
1.8 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package sqlstore
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
|
|
"github.com/hanzoteam/server/server/public/model"
|
|
"github.com/hanzoteam/server/server/public/shared/mlog"
|
|
)
|
|
|
|
func TestMigration000194(t *testing.T) {
|
|
logger := mlog.CreateTestLogger(t)
|
|
|
|
settings, err := makeSqlSettings(model.DatabaseDriverPostgres)
|
|
if err != nil {
|
|
t.Skip(err)
|
|
}
|
|
|
|
store, err := New(*settings, logger, nil)
|
|
require.NoError(t, err)
|
|
defer store.Close()
|
|
|
|
master := store.GetMaster()
|
|
|
|
upSQL := readMigrationSQL(t, "000194_add_type_id_index_to_access_control_policies.up.sql")
|
|
downSQL := readMigrationSQL(t, "000194_add_type_id_index_to_access_control_policies.down.sql")
|
|
|
|
indexExists := func() bool {
|
|
var exists bool
|
|
require.NoError(t, master.Get(&exists, "SELECT EXISTS (SELECT 1 FROM pg_indexes WHERE indexname = 'idx_access_control_policies_type_id')"))
|
|
return exists
|
|
}
|
|
|
|
// CONCURRENTLY cannot run inside a transaction; ExecNoTimeout runs unwrapped.
|
|
_, err = master.ExecNoTimeout(upSQL)
|
|
require.NoError(t, err, "up migration should succeed")
|
|
assert.True(t, indexExists(), "index should exist after up migration")
|
|
|
|
// IF NOT EXISTS makes a second up a safe no-op.
|
|
_, err = master.ExecNoTimeout(upSQL)
|
|
require.NoError(t, err, "up migration should be idempotent")
|
|
assert.True(t, indexExists())
|
|
|
|
_, err = master.ExecNoTimeout(downSQL)
|
|
require.NoError(t, err, "down migration should succeed")
|
|
assert.False(t, indexExists(), "index should be gone after down migration")
|
|
|
|
// IF EXISTS makes a second down a safe no-op.
|
|
_, err = master.ExecNoTimeout(downSQL)
|
|
require.NoError(t, err, "down migration should be idempotent")
|
|
}
|