mirror of
https://github.com/mattermost/mattermost.git
synced 2026-03-02 21:31:09 -05:00
9c272f0b20
* Implement AppendFile for FileBackend * Split test into subtests * [MM-26812] Add support for resumable file uploads (#15252) * Implement UploadSession * Implement UploadSessionStore * Add error strings * Implement resumable file uploads * Add UploadType * Fix retry layer tests * Regenerate store layers * Fix store error handling * Use base for filename * Prevent concurrent uploads on the same upload session * Fix erroneus error string * Improve error handling Co-authored-by: Mattermod <mattermod@users.noreply.github.com> * Fix translations Co-authored-by: Mattermod <mattermod@users.noreply.github.com>
242 lines
7 KiB
Go
242 lines
7 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package api4
|
|
|
|
import (
|
|
"bytes"
|
|
"io"
|
|
"net/http"
|
|
"testing"
|
|
|
|
"github.com/mattermost/mattermost-server/v5/model"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestCreateUpload(t *testing.T) {
|
|
th := Setup(t).InitBasic()
|
|
defer th.TearDown()
|
|
|
|
us := &model.UploadSession{
|
|
ChannelId: th.BasicChannel.Id,
|
|
Filename: "upload",
|
|
FileSize: 8 * 1024 * 1024,
|
|
}
|
|
|
|
t.Run("file attachments disabled", func(t *testing.T) {
|
|
th.App.UpdateConfig(func(cfg *model.Config) { *cfg.FileSettings.EnableFileAttachments = false })
|
|
defer th.App.UpdateConfig(func(cfg *model.Config) { *cfg.FileSettings.EnableFileAttachments = true })
|
|
u, resp := th.Client.CreateUpload(us)
|
|
require.Nil(t, u)
|
|
require.Error(t, resp.Error)
|
|
require.Equal(t, "api.file.attachments.disabled.app_error", resp.Error.Id)
|
|
require.Equal(t, http.StatusNotImplemented, resp.StatusCode)
|
|
})
|
|
|
|
t.Run("no permissions", func(t *testing.T) {
|
|
us.ChannelId = th.BasicPrivateChannel2.Id
|
|
u, resp := th.Client.CreateUpload(us)
|
|
require.Nil(t, u)
|
|
require.Error(t, resp.Error)
|
|
require.Equal(t, "api.context.permissions.app_error", resp.Error.Id)
|
|
require.Equal(t, http.StatusForbidden, resp.StatusCode)
|
|
})
|
|
|
|
t.Run("valid", func(t *testing.T) {
|
|
us.ChannelId = th.BasicChannel.Id
|
|
u, resp := th.Client.CreateUpload(us)
|
|
require.Nil(t, resp.Error)
|
|
require.NotEmpty(t, u)
|
|
require.Equal(t, http.StatusCreated, resp.StatusCode)
|
|
})
|
|
}
|
|
|
|
func TestGetUpload(t *testing.T) {
|
|
th := Setup(t).InitBasic()
|
|
defer th.TearDown()
|
|
|
|
us := &model.UploadSession{
|
|
Id: model.NewId(),
|
|
Type: model.UploadTypeAttachment,
|
|
CreateAt: model.GetMillis(),
|
|
UserId: th.BasicUser2.Id,
|
|
ChannelId: th.BasicChannel.Id,
|
|
Filename: "upload",
|
|
FileSize: 8 * 1024 * 1024,
|
|
}
|
|
us, err := th.App.CreateUploadSession(us)
|
|
require.Nil(t, err)
|
|
require.NotNil(t, us)
|
|
require.NotEmpty(t, us)
|
|
|
|
t.Run("upload not found", func(t *testing.T) {
|
|
u, resp := th.Client.GetUpload(model.NewId())
|
|
require.Nil(t, u)
|
|
require.Error(t, resp.Error)
|
|
require.Equal(t, "app.upload.get.app_error", resp.Error.Id)
|
|
require.Equal(t, http.StatusNotFound, resp.StatusCode)
|
|
})
|
|
|
|
t.Run("no permissions", func(t *testing.T) {
|
|
u, resp := th.Client.GetUpload(us.Id)
|
|
require.Nil(t, u)
|
|
require.Error(t, resp.Error)
|
|
require.Equal(t, "api.upload.get_upload.forbidden.app_error", resp.Error.Id)
|
|
})
|
|
|
|
t.Run("success", func(t *testing.T) {
|
|
expected, resp := th.Client.CreateUpload(us)
|
|
require.Nil(t, resp.Error)
|
|
require.NotEmpty(t, expected)
|
|
require.Equal(t, http.StatusCreated, resp.StatusCode)
|
|
|
|
u, resp := th.Client.GetUpload(expected.Id)
|
|
require.Nil(t, resp.Error)
|
|
require.NotEmpty(t, u)
|
|
require.Equal(t, expected, u)
|
|
})
|
|
}
|
|
|
|
func TestGetUploadsForUser(t *testing.T) {
|
|
th := Setup(t).InitBasic()
|
|
defer th.TearDown()
|
|
|
|
t.Run("no permissions", func(t *testing.T) {
|
|
uss, resp := th.Client.GetUploadsForUser(th.BasicUser2.Id)
|
|
require.Error(t, resp.Error)
|
|
require.Equal(t, "api.user.get_uploads_for_user.forbidden.app_error", resp.Error.Id)
|
|
require.Nil(t, uss)
|
|
})
|
|
|
|
t.Run("empty", func(t *testing.T) {
|
|
uss, resp := th.Client.GetUploadsForUser(th.BasicUser.Id)
|
|
require.Nil(t, resp.Error)
|
|
require.Empty(t, uss)
|
|
})
|
|
|
|
t.Run("success", func(t *testing.T) {
|
|
uploads := make([]*model.UploadSession, 4)
|
|
for i := 0; i < len(uploads); i++ {
|
|
us := &model.UploadSession{
|
|
Id: model.NewId(),
|
|
Type: model.UploadTypeAttachment,
|
|
CreateAt: model.GetMillis(),
|
|
UserId: th.BasicUser.Id,
|
|
ChannelId: th.BasicChannel.Id,
|
|
Filename: "upload",
|
|
FileSize: 8 * 1024 * 1024,
|
|
}
|
|
us, err := th.App.CreateUploadSession(us)
|
|
require.Nil(t, err)
|
|
require.NotNil(t, us)
|
|
require.NotEmpty(t, us)
|
|
us.Path = ""
|
|
uploads[i] = us
|
|
}
|
|
|
|
uss, resp := th.Client.GetUploadsForUser(th.BasicUser.Id)
|
|
require.Nil(t, resp.Error)
|
|
require.NotEmpty(t, uss)
|
|
require.Len(t, uss, len(uploads))
|
|
for i := range uploads {
|
|
require.Contains(t, uss, uploads[i])
|
|
}
|
|
})
|
|
}
|
|
|
|
func TestUploadData(t *testing.T) {
|
|
th := Setup(t).InitBasic()
|
|
defer th.TearDown()
|
|
if *th.App.Config().FileSettings.DriverName == "" {
|
|
t.Skip("skipping because no file driver is enabled")
|
|
}
|
|
|
|
us := &model.UploadSession{
|
|
Id: model.NewId(),
|
|
Type: model.UploadTypeAttachment,
|
|
CreateAt: model.GetMillis(),
|
|
UserId: th.BasicUser2.Id,
|
|
ChannelId: th.BasicChannel.Id,
|
|
Filename: "upload",
|
|
FileSize: 8 * 1024 * 1024,
|
|
}
|
|
us, err := th.App.CreateUploadSession(us)
|
|
require.Nil(t, err)
|
|
require.NotNil(t, us)
|
|
require.NotEmpty(t, us)
|
|
|
|
data := randomBytes(t, int(us.FileSize))
|
|
|
|
t.Run("file attachments disabled", func(t *testing.T) {
|
|
th.App.UpdateConfig(func(cfg *model.Config) { *cfg.FileSettings.EnableFileAttachments = false })
|
|
defer th.App.UpdateConfig(func(cfg *model.Config) { *cfg.FileSettings.EnableFileAttachments = true })
|
|
info, resp := th.Client.UploadData(model.NewId(), bytes.NewReader(data))
|
|
require.Nil(t, info)
|
|
require.Error(t, resp.Error)
|
|
require.Equal(t, "api.file.attachments.disabled.app_error", resp.Error.Id)
|
|
})
|
|
|
|
t.Run("upload not found", func(t *testing.T) {
|
|
info, resp := th.Client.UploadData(model.NewId(), bytes.NewReader(data))
|
|
require.Nil(t, info)
|
|
require.Error(t, resp.Error)
|
|
require.Equal(t, "app.upload.get.app_error", resp.Error.Id)
|
|
require.Equal(t, http.StatusNotFound, resp.StatusCode)
|
|
})
|
|
|
|
t.Run("no permissions", func(t *testing.T) {
|
|
info, resp := th.Client.UploadData(us.Id, bytes.NewReader(data))
|
|
require.Nil(t, info)
|
|
require.Error(t, resp.Error)
|
|
require.Equal(t, "api.context.permissions.app_error", resp.Error.Id)
|
|
})
|
|
|
|
t.Run("bad content-length", func(t *testing.T) {
|
|
info, resp := th.Client.UploadData(us.Id, bytes.NewReader(append(data, 0x00)))
|
|
require.Nil(t, info)
|
|
require.Error(t, resp.Error)
|
|
require.Equal(t, "api.upload.upload_data.invalid_content_length", resp.Error.Id)
|
|
})
|
|
|
|
t.Run("success", func(t *testing.T) {
|
|
u, resp := th.Client.CreateUpload(us)
|
|
require.Nil(t, resp.Error)
|
|
require.NotEmpty(t, u)
|
|
require.Equal(t, http.StatusCreated, resp.StatusCode)
|
|
|
|
info, resp := th.Client.UploadData(u.Id, bytes.NewReader(data))
|
|
require.Nil(t, resp.Error)
|
|
require.NotEmpty(t, info)
|
|
require.Equal(t, u.Filename, info.Name)
|
|
|
|
file, resp := th.Client.GetFile(info.Id)
|
|
require.Nil(t, resp.Error)
|
|
require.Equal(t, file, data)
|
|
})
|
|
|
|
t.Run("resume success", func(t *testing.T) {
|
|
u, resp := th.Client.CreateUpload(us)
|
|
require.Nil(t, resp.Error)
|
|
require.NotEmpty(t, u)
|
|
require.Equal(t, http.StatusCreated, resp.StatusCode)
|
|
|
|
rd := &io.LimitedReader{
|
|
R: bytes.NewReader(data),
|
|
N: 5 * 1024 * 1024,
|
|
}
|
|
info, resp := th.Client.UploadData(u.Id, rd)
|
|
require.Nil(t, resp.Error)
|
|
require.Nil(t, info)
|
|
require.Equal(t, http.StatusNoContent, resp.StatusCode)
|
|
|
|
info, resp = th.Client.UploadData(u.Id, bytes.NewReader(data[5*1024*1024:]))
|
|
require.Nil(t, resp.Error)
|
|
require.NotEmpty(t, info)
|
|
require.Equal(t, u.Filename, info.Name)
|
|
|
|
file, resp := th.Client.GetFile(info.Id)
|
|
require.Nil(t, resp.Error)
|
|
require.Equal(t, file, data)
|
|
})
|
|
}
|