[MM-17109] Bump model version to 5.16 (#12172)
* Bump model version to 5.16 * Make db schema version indipended of app version
Этот коммит содержится в:
коммит произвёл
GitHub
родитель
3a3676eb12
Коммит
3323e7a619
@@ -13,6 +13,7 @@ import (
|
|||||||
// It should be maintained in chronological order with most current
|
// It should be maintained in chronological order with most current
|
||||||
// release at the front of the list.
|
// release at the front of the list.
|
||||||
var versions = []string{
|
var versions = []string{
|
||||||
|
"5.16.0",
|
||||||
"5.15.0",
|
"5.15.0",
|
||||||
"5.14.0",
|
"5.14.0",
|
||||||
"5.13.0",
|
"5.13.0",
|
||||||
|
|||||||
@@ -20,6 +20,7 @@ import (
|
|||||||
)
|
)
|
||||||
|
|
||||||
const (
|
const (
|
||||||
|
CURRENT_SCHEMA_VERSION = VERSION_5_15_0
|
||||||
VERSION_5_16_0 = "5.16.0"
|
VERSION_5_16_0 = "5.16.0"
|
||||||
VERSION_5_15_0 = "5.15.0"
|
VERSION_5_15_0 = "5.15.0"
|
||||||
VERSION_5_14_0 = "5.14.0"
|
VERSION_5_14_0 = "5.14.0"
|
||||||
|
|||||||
@@ -6,7 +6,6 @@ package sqlstore
|
|||||||
import (
|
import (
|
||||||
"testing"
|
"testing"
|
||||||
|
|
||||||
"github.com/mattermost/mattermost-server/model"
|
|
||||||
"github.com/mattermost/mattermost-server/store"
|
"github.com/mattermost/mattermost-server/store"
|
||||||
"github.com/stretchr/testify/require"
|
"github.com/stretchr/testify/require"
|
||||||
)
|
)
|
||||||
@@ -36,25 +35,25 @@ func TestStoreUpgrade(t *testing.T) {
|
|||||||
|
|
||||||
t.Run("upgrade from earliest supported version", func(t *testing.T) {
|
t.Run("upgrade from earliest supported version", func(t *testing.T) {
|
||||||
saveSchemaVersion(sqlStore, VERSION_3_0_0)
|
saveSchemaVersion(sqlStore, VERSION_3_0_0)
|
||||||
err := UpgradeDatabase(sqlStore, model.CurrentVersion)
|
err := UpgradeDatabase(sqlStore, CURRENT_SCHEMA_VERSION)
|
||||||
require.NoError(t, err)
|
require.NoError(t, err)
|
||||||
require.Equal(t, model.CurrentVersion, sqlStore.GetCurrentSchemaVersion())
|
require.Equal(t, CURRENT_SCHEMA_VERSION, sqlStore.GetCurrentSchemaVersion())
|
||||||
})
|
})
|
||||||
|
|
||||||
t.Run("upgrade from no existing version", func(t *testing.T) {
|
t.Run("upgrade from no existing version", func(t *testing.T) {
|
||||||
saveSchemaVersion(sqlStore, "")
|
saveSchemaVersion(sqlStore, "")
|
||||||
err := UpgradeDatabase(sqlStore, model.CurrentVersion)
|
err := UpgradeDatabase(sqlStore, CURRENT_SCHEMA_VERSION)
|
||||||
require.NoError(t, err)
|
require.NoError(t, err)
|
||||||
require.Equal(t, model.CurrentVersion, sqlStore.GetCurrentSchemaVersion())
|
require.Equal(t, CURRENT_SCHEMA_VERSION, sqlStore.GetCurrentSchemaVersion())
|
||||||
})
|
})
|
||||||
|
|
||||||
t.Run("upgrade schema running earlier minor version", func(t *testing.T) {
|
t.Run("upgrade schema running earlier minor version", func(t *testing.T) {
|
||||||
saveSchemaVersion(sqlStore, "5.1.0")
|
saveSchemaVersion(sqlStore, "5.1.0")
|
||||||
err := UpgradeDatabase(sqlStore, "5.8.0")
|
err := UpgradeDatabase(sqlStore, "5.8.0")
|
||||||
require.NoError(t, err)
|
require.NoError(t, err)
|
||||||
// Assert model.CurrentVersion, not 5.8.0, since the migrations will move
|
// Assert CURRENT_SCHEMA_VERSION, not 5.8.0, since the migrations will move
|
||||||
// past 5.8.0 regardless of the input parameter.
|
// past 5.8.0 regardless of the input parameter.
|
||||||
require.Equal(t, model.CurrentVersion, sqlStore.GetCurrentSchemaVersion())
|
require.Equal(t, CURRENT_SCHEMA_VERSION, sqlStore.GetCurrentSchemaVersion())
|
||||||
})
|
})
|
||||||
|
|
||||||
t.Run("upgrade schema running later minor version", func(t *testing.T) {
|
t.Run("upgrade schema running later minor version", func(t *testing.T) {
|
||||||
@@ -66,9 +65,9 @@ func TestStoreUpgrade(t *testing.T) {
|
|||||||
|
|
||||||
t.Run("upgrade schema running earlier major version", func(t *testing.T) {
|
t.Run("upgrade schema running earlier major version", func(t *testing.T) {
|
||||||
saveSchemaVersion(sqlStore, "4.1.0")
|
saveSchemaVersion(sqlStore, "4.1.0")
|
||||||
err := UpgradeDatabase(sqlStore, model.CurrentVersion)
|
err := UpgradeDatabase(sqlStore, CURRENT_SCHEMA_VERSION)
|
||||||
require.NoError(t, err)
|
require.NoError(t, err)
|
||||||
require.Equal(t, model.CurrentVersion, sqlStore.GetCurrentSchemaVersion())
|
require.Equal(t, CURRENT_SCHEMA_VERSION, sqlStore.GetCurrentSchemaVersion())
|
||||||
})
|
})
|
||||||
|
|
||||||
t.Run("upgrade schema running later major version", func(t *testing.T) {
|
t.Run("upgrade schema running later major version", func(t *testing.T) {
|
||||||
@@ -94,12 +93,12 @@ func TestSaveSchemaVersion(t *testing.T) {
|
|||||||
})
|
})
|
||||||
|
|
||||||
t.Run("set current version", func(t *testing.T) {
|
t.Run("set current version", func(t *testing.T) {
|
||||||
saveSchemaVersion(sqlStore, model.CurrentVersion)
|
saveSchemaVersion(sqlStore, CURRENT_SCHEMA_VERSION)
|
||||||
props, err := ss.System().Get()
|
props, err := ss.System().Get()
|
||||||
require.Nil(t, err)
|
require.Nil(t, err)
|
||||||
|
|
||||||
require.Equal(t, model.CurrentVersion, props["Version"])
|
require.Equal(t, CURRENT_SCHEMA_VERSION, props["Version"])
|
||||||
require.Equal(t, model.CurrentVersion, sqlStore.GetCurrentSchemaVersion())
|
require.Equal(t, CURRENT_SCHEMA_VERSION, sqlStore.GetCurrentSchemaVersion())
|
||||||
})
|
})
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|||||||
Ссылка в новой задаче
Block a user