mirror of
https://github.com/hanzoai/team.git
synced 2026-11-01 07:55:31 +00:00
Harshil SharmaandGitHub
fcd5421bb0
* Added base fr report generation * WIP * Refactoring and cleanup * lint fixes, added new tests * test fix * Several improvements * Addressed some security enhancements * Created zip writer entery later * Improved a test to check for file content * Improved error handling * Made a geneeric function * accepting comment in report API * Removed an unnecessary check * Made a geneeric function * Made the comment body not required and updated API docs
83 lines
3.4 KiB
Go
83 lines
3.4 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package localcachelayer
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
|
|
"github.com/mattermost/mattermost/server/public/model"
|
|
"github.com/mattermost/mattermost/server/public/shared/mlog"
|
|
"github.com/mattermost/mattermost/server/v8/channels/store/storetest"
|
|
"github.com/mattermost/mattermost/server/v8/channels/store/storetest/mocks"
|
|
)
|
|
|
|
func TestFileInfoStore(t *testing.T) {
|
|
StoreTestWithSqlStore(t, storetest.TestFileInfoStore)
|
|
}
|
|
|
|
func TestFileInfoStoreCache(t *testing.T) {
|
|
fakeFileInfo := model.FileInfo{Id: "123", PostId: "123"}
|
|
logger := mlog.CreateConsoleTestLogger(t)
|
|
|
|
t.Run("first call not cached, second cached and returning same data", func(t *testing.T) {
|
|
mockStore := getMockStore(t)
|
|
mockCacheProvider := getMockCacheProvider()
|
|
cachedStore, err := NewLocalCacheLayer(mockStore, nil, nil, mockCacheProvider, logger)
|
|
require.NoError(t, err)
|
|
|
|
fileInfos, err := cachedStore.FileInfo().GetForPost("123", true, true, true)
|
|
require.NoError(t, err)
|
|
assert.Equal(t, fileInfos, []*model.FileInfo{&fakeFileInfo})
|
|
mockStore.FileInfo().(*mocks.FileInfoStore).AssertNumberOfCalls(t, "GetForPost", 1)
|
|
assert.Equal(t, fileInfos, []*model.FileInfo{&fakeFileInfo})
|
|
cachedStore.FileInfo().GetForPost("123", true, true, true)
|
|
mockStore.FileInfo().(*mocks.FileInfoStore).AssertNumberOfCalls(t, "GetForPost", 1)
|
|
})
|
|
|
|
t.Run("first call not cached, second force no cached", func(t *testing.T) {
|
|
mockStore := getMockStore(t)
|
|
mockCacheProvider := getMockCacheProvider()
|
|
cachedStore, err := NewLocalCacheLayer(mockStore, nil, nil, mockCacheProvider, logger)
|
|
require.NoError(t, err)
|
|
|
|
cachedStore.FileInfo().GetForPost("123", true, true, true)
|
|
mockStore.FileInfo().(*mocks.FileInfoStore).AssertNumberOfCalls(t, "GetForPost", 1)
|
|
cachedStore.FileInfo().GetForPost("123", true, true, false)
|
|
mockStore.FileInfo().(*mocks.FileInfoStore).AssertNumberOfCalls(t, "GetForPost", 2)
|
|
})
|
|
|
|
t.Run("first call not cached, invalidate, and then not cached again", func(t *testing.T) {
|
|
mockStore := getMockStore(t)
|
|
mockCacheProvider := getMockCacheProvider()
|
|
cachedStore, err := NewLocalCacheLayer(mockStore, nil, nil, mockCacheProvider, logger)
|
|
require.NoError(t, err)
|
|
|
|
cachedStore.FileInfo().GetForPost("123", true, true, true)
|
|
mockStore.FileInfo().(*mocks.FileInfoStore).AssertNumberOfCalls(t, "GetForPost", 1)
|
|
cachedStore.FileInfo().InvalidateFileInfosForPostCache("123", true)
|
|
cachedStore.FileInfo().GetForPost("123", true, true, true)
|
|
mockStore.FileInfo().(*mocks.FileInfoStore).AssertNumberOfCalls(t, "GetForPost", 2)
|
|
})
|
|
|
|
t.Run("GetByIds cache test", func(t *testing.T) {
|
|
mockStore := getMockStore(t)
|
|
mockCacheProvider := getMockCacheProvider()
|
|
cachedStore, err := NewLocalCacheLayer(mockStore, nil, nil, mockCacheProvider, logger)
|
|
require.NoError(t, err)
|
|
|
|
fileInfos, err := cachedStore.FileInfo().GetByIds([]string{"123"}, true, true, false)
|
|
require.NoError(t, err)
|
|
assert.Equal(t, []*model.FileInfo{&fakeFileInfo}, fileInfos)
|
|
mockStore.FileInfo().(*mocks.FileInfoStore).AssertNumberOfCalls(t, "GetByIds", 1)
|
|
|
|
fileInfosCached, err := cachedStore.FileInfo().GetByIds([]string{"123"}, true, true, false)
|
|
require.NoError(t, err)
|
|
assert.Equal(t, []*model.FileInfo{&fakeFileInfo}, fileInfosCached)
|
|
mockStore.FileInfo().(*mocks.FileInfoStore).AssertNumberOfCalls(t, "GetByIds", 1)
|
|
})
|
|
}
|