Some checks failed
Server CI / Postgres (shard 0) (push) Has been skipped
Server CI / Postgres (shard 3) (push) Has been skipped
BuildEnv Docker Image / build-image (push) Failing after 4s
BuildEnv Docker Image / build-image-fips (push) Failing after 4s
Migration Review & Release Notes / Generate migration review & release notes (push) Failing after 3s
Server CI / Build Environment (push) Failing after 3s
Server CI / Postgres (shard 1) (push) Has been skipped
Tools CI / check-style (mattermost-govet) (push) Failing after 4s
Server CI / Postgres (shard 2) (push) Has been skipped
Tools CI / check-style (mmgotool) (push) Failing after 3s
Server CI / Elasticsearch v8 Compatibility (push) Has been skipped
Server CI / OpenSearch v2 Compatibility (push) Has been skipped
Server CI / Postgres FIPS (shard 0) (push) Has been skipped
Tools CI / Test (mattermost-govet) (push) Failing after 3s
Server CI / Postgres FIPS (shard 1) (push) Has been skipped
Server CI / Postgres FIPS (shard 2) (push) Has been skipped
Tools CI / Test (mmgotool) (push) Failing after 3s
Server CI / Postgres FIPS (shard 3) (push) Has been skipped
Server CI / Run mmctl tests (push) Has been skipped
Server CI / Run mmctl tests (FIPS) (push) Has been skipped
Server CI / Merge Postgres Test Results (push) Has been skipped
Server CI / Merge Postgres FIPS Test Results (push) Has been skipped
Server CI / Check backport migrations (push) Has been skipped
Server CI / Check generated files (push) Has been skipped
Server CI / check-style (push) Has been skipped
Server CI / Vet API (push) Has been skipped
Server CI / Build mattermost server app (push) Has been skipped
Server CI / Server CI Complete (push) Failing after 2s
Server CI / Server CI Artifacts (push) Has been skipped
Server CI / Sentry Upload (push) Has been skipped
Server CI / Server CI Report (push) Failing after 5m1s
Server CI Nightly Race / Race Detector (push) Has been skipped
CodeQL / Analyze (go) (push) Has been cancelled
CodeQL / Analyze (javascript) (push) Has been cancelled
Webapp CI Cache Warm / Warm Webapp node_modules Cache (push) Has been cancelled
E2E CI Cache Warm / Warm E2E npm Registry Cache (push) Has been cancelled
API / build (push) Has been cancelled
Web App CI / check-types (push) Has been cancelled
Web App CI / check-lint (push) Has been cancelled
Web App CI / check-i18n (push) Has been cancelled
Web App CI / check-external-links (push) Has been cancelled
Web App CI / test (platform) (push) Has been cancelled
Web App CI / test (mattermost-redux) (push) Has been cancelled
Web App CI / test (channels shard 1/4) (push) Has been cancelled
Web App CI / test (channels shard 2/4) (push) Has been cancelled
Web App CI / test (channels shard 3/4) (push) Has been cancelled
Web App CI / test (channels shard 4/4) (push) Has been cancelled
Web App CI / upload-coverage (push) Has been cancelled
Web App CI / build (push) Has been cancelled
YAML Lint / yamllint (push) Has been cancelled
Push mirrored docker images / cd/Push mirrored docker images (push) Has been cancelled
docs-cd / Build and deploy docs (push) Has been cancelled
Docs CI / build (push) Has been cancelled
E2E Cache Warm: Cypress deps / Warm Cypress deps cache (push) Has been cancelled
Scorecards supply-chain security / Scorecard analysis (push) Has been cancelled
Server CI Weekly / Postgres FIPS (push) Has been skipped
Server CI Weekly / Run mmctl tests (FIPS) (push) Has been skipped
Server CI Weekly / Postgres with binary parameters (push) Has been skipped
Server CI Weekly / Build Environment (push) Failing after 1s
Server CI Nightly Race / Build Environment (push) Failing after 1s
Server CI Cache Warm / Warm Go Module and Build Cache (push) Failing after 2s
183 lines
7.2 KiB
Go
183 lines
7.2 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package app
|
|
|
|
import (
|
|
"net/http"
|
|
"testing"
|
|
|
|
"github.com/mattermost/mattermost/server/public/model"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestRestorePostVersion(t *testing.T) {
|
|
mainHelper.Parallel(t)
|
|
th := Setup(t).InitBasic(t)
|
|
|
|
t.Run("is able to restore a post version", func(t *testing.T) {
|
|
post := th.CreatePost(t, th.BasicChannel, func(p *model.Post) {
|
|
p.Message = "original message"
|
|
})
|
|
th.PostPatch(t, post, "new message 2")
|
|
th.PostPatch(t, post, "new message 3")
|
|
|
|
// verify post's state
|
|
fetchedPost, err := th.App.Srv().Store().Post().GetSingle(th.Context, post.Id, true)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "new message 3", fetchedPost.Message)
|
|
|
|
editHistory, appErr := th.App.GetEditHistoryForPost(th.Context, post.Id)
|
|
require.Nil(t, appErr)
|
|
require.Equal(t, 2, len(editHistory))
|
|
require.Equal(t, "new message 2", editHistory[0].Message)
|
|
|
|
// now we'll restore a post version
|
|
restoredPost, isMemberForPreview, appErr := th.App.RestorePostVersion(th.Context, th.BasicUser.Id, post.Id, editHistory[0].Id)
|
|
require.Nil(t, appErr)
|
|
require.Equal(t, "new message 2", restoredPost.Message)
|
|
require.True(t, isMemberForPreview)
|
|
|
|
// verify from database
|
|
fetchedPost, err = th.App.Srv().Store().Post().GetSingle(th.Context, post.Id, true)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "new message 2", fetchedPost.Message)
|
|
|
|
// verify that we now have 3 items in post's edit history
|
|
editHistory, appErr = th.App.GetEditHistoryForPost(th.Context, post.Id)
|
|
require.Nil(t, appErr)
|
|
require.Equal(t, 3, len(editHistory))
|
|
require.Equal(t, "new message 3", editHistory[0].Message)
|
|
require.Equal(t, "new message 2", editHistory[1].Message)
|
|
require.Equal(t, "original message", editHistory[2].Message)
|
|
})
|
|
|
|
t.Run("is able to restore a post version including its files", func(t *testing.T) {
|
|
fileBytes := []byte("file contents")
|
|
fileInfo, appErr := th.App.UploadFile(th.Context, fileBytes, th.BasicChannel.Id, "file.txt")
|
|
require.Nil(t, appErr)
|
|
|
|
post := th.CreatePost(t, th.BasicChannel, func(p *model.Post) {
|
|
p.FileIds = []string{fileInfo.Id}
|
|
p.Message = "original message"
|
|
})
|
|
|
|
// this update removes all files
|
|
th.PostPatch(t, post, "new message 2", func(p *model.PostPatch) {
|
|
p.FileIds = &model.StringArray{}
|
|
})
|
|
// this update only changes the message
|
|
th.PostPatch(t, post, "new message 3")
|
|
|
|
// verify post's state
|
|
fetchedPost, err := th.App.Srv().Store().Post().GetSingle(th.Context, post.Id, true)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "new message 3", fetchedPost.Message)
|
|
require.Empty(t, fetchedPost.FileIds)
|
|
|
|
editHistory, appErr := th.App.GetEditHistoryForPost(th.Context, post.Id)
|
|
require.Nil(t, appErr)
|
|
require.Equal(t, 2, len(editHistory))
|
|
require.Equal(t, "new message 2", editHistory[0].Message)
|
|
require.Equal(t, 0, len(editHistory[0].FileIds))
|
|
|
|
require.Equal(t, "original message", editHistory[1].Message)
|
|
require.Equal(t, 1, len(editHistory[1].FileIds))
|
|
|
|
// now we'll restore a post version
|
|
restoredPost, isMemberForPreview, appErr := th.App.RestorePostVersion(th.Context, th.BasicUser.Id, post.Id, editHistory[1].Id)
|
|
require.Nil(t, appErr)
|
|
require.Equal(t, "original message", restoredPost.Message)
|
|
require.Equal(t, 1, len(restoredPost.FileIds))
|
|
require.True(t, isMemberForPreview)
|
|
|
|
// verify from database
|
|
fetchedPost, err = th.App.Srv().Store().Post().GetSingle(th.Context, post.Id, true)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "original message", fetchedPost.Message)
|
|
require.Equal(t, 1, len(fetchedPost.FileIds))
|
|
|
|
// verify edit history\
|
|
editHistory, appErr = th.App.GetEditHistoryForPost(th.Context, post.Id)
|
|
require.Nil(t, appErr)
|
|
require.Equal(t, 3, len(editHistory))
|
|
|
|
require.Equal(t, "new message 3", editHistory[0].Message)
|
|
require.Equal(t, 0, len(editHistory[0].FileIds))
|
|
|
|
require.Equal(t, "new message 2", editHistory[1].Message)
|
|
require.Equal(t, 0, len(editHistory[1].FileIds))
|
|
|
|
require.Equal(t, "original message", editHistory[2].Message)
|
|
require.Equal(t, 1, len(editHistory[2].FileIds))
|
|
})
|
|
|
|
t.Run("should return an error if trying to restore a post that is not in any edit history", func(t *testing.T) {
|
|
post := th.CreatePost(t, th.BasicChannel, func(p *model.Post) {
|
|
p.Message = "original message"
|
|
})
|
|
th.PostPatch(t, post, "new message 2")
|
|
th.PostPatch(t, post, "new message 3")
|
|
|
|
// verify post's state
|
|
fetchedPost, err := th.App.Srv().Store().Post().GetSingle(th.Context, post.Id, true)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "new message 3", fetchedPost.Message)
|
|
|
|
// now we'll restore a post version
|
|
otherPost := th.CreatePost(t, th.BasicChannel)
|
|
restoredPost, _, appErr := th.App.RestorePostVersion(th.Context, th.BasicUser.Id, post.Id, otherPost.Id)
|
|
require.NotNil(t, appErr)
|
|
require.Equal(t, http.StatusBadRequest, appErr.StatusCode)
|
|
require.Equal(t, "app.post.restore_post_version.not_an_history_item.app_error", appErr.Id)
|
|
require.Nil(t, restoredPost)
|
|
|
|
// verify from database that the post wasn't modified
|
|
fetchedPost, err = th.App.Srv().Store().Post().GetSingle(th.Context, post.Id, true)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "new message 3", fetchedPost.Message)
|
|
})
|
|
|
|
t.Run("should return an error if the post does not exist", func(t *testing.T) {
|
|
restoredPost, _, appErr := th.App.RestorePostVersion(th.Context, th.BasicUser.Id, model.NewId(), model.NewId())
|
|
require.NotNil(t, appErr)
|
|
require.Equal(t, http.StatusNotFound, appErr.StatusCode)
|
|
require.Equal(t, "app.post.restore_post_version.get_single.app_error", appErr.Id)
|
|
require.Nil(t, restoredPost)
|
|
})
|
|
|
|
t.Run("should return an error if the restore post does not exist", func(t *testing.T) {
|
|
post := th.CreatePost(t, th.BasicChannel)
|
|
|
|
// now we'll restore a post version
|
|
invalidRestorePostIUd := model.NewId()
|
|
restoredPost, _, appErr := th.App.RestorePostVersion(th.Context, th.BasicUser.Id, post.Id, invalidRestorePostIUd)
|
|
require.NotNil(t, appErr)
|
|
require.Equal(t, http.StatusNotFound, appErr.StatusCode)
|
|
require.Equal(t, "app.post.restore_post_version.get_single.app_error", appErr.Id)
|
|
require.Nil(t, restoredPost)
|
|
})
|
|
|
|
t.Run("should return an error if trying to restore a post that is in some other posts edit history", func(t *testing.T) {
|
|
post := th.CreatePost(t, th.BasicChannel)
|
|
th.PostPatch(t, post, "new message 2")
|
|
|
|
otherPost := th.CreatePost(t, th.BasicChannel, func(post *model.Post) {
|
|
post.Message = "other post original message"
|
|
})
|
|
th.PostPatch(t, otherPost, "other post new message 2")
|
|
|
|
otherPostEditHistory, appErr := th.App.GetEditHistoryForPost(th.Context, otherPost.Id)
|
|
require.Nil(t, appErr)
|
|
require.Equal(t, 1, len(otherPostEditHistory))
|
|
require.Equal(t, "other post original message", otherPostEditHistory[0].Message)
|
|
|
|
// we'll specify post's ID and other post's version ID, his should fail
|
|
restoredPost, _, appErr := th.App.RestorePostVersion(th.Context, th.BasicUser.Id, post.Id, otherPostEditHistory[0].Id)
|
|
require.NotNil(t, appErr)
|
|
require.Equal(t, "app.post.restore_post_version.not_an_history_item.app_error", appErr.Id)
|
|
require.Equal(t, http.StatusBadRequest, appErr.StatusCode)
|
|
require.Nil(t, restoredPost)
|
|
})
|
|
}
|