MM-27187: Use the correct page offset for cache clear methods (#15094)

* MM-27187: Use the correct page offset for cache clear methods

We were just using page and incrementing by 1. This would fetch pages
one by one like (1-100, 2-102, 3-103) rather than (1-100,100-200,200-300).

We fix that to update the correct page offset.

* Trigger CI
Этот коммит содержится в:
Agniva De Sarker
2020-07-23 19:55:16 +05:30
коммит произвёл GitHub
родитель 46207a35f5
Коммит aad940e104
4 изменённых файлов: 47 добавлений и 2 удалений

Просмотреть файл

@@ -11,7 +11,9 @@ import (
"testing"
"github.com/mattermost/mattermost-server/v5/model"
"github.com/mattermost/mattermost-server/v5/store/storetest/mocks"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/mock"
"github.com/stretchr/testify/require"
)
@@ -1136,3 +1138,24 @@ func TestInvalidateAllEmailInvites(t *testing.T) {
_, err = th.App.Srv().Store.Token().GetByToken(t3.Token)
require.Nil(t, err)
}
func TestClearTeamMembersCache(t *testing.T) {
th := SetupWithStoreMock(t)
defer th.TearDown()
mockStore := th.App.Srv().Store.(*mocks.Store)
mockTeamStore := mocks.TeamStore{}
tms := []*model.TeamMember{}
for i := 0; i < 200; i++ {
tms = append(tms, &model.TeamMember{
TeamId: "1",
})
}
mockTeamStore.On("GetMembers", "teamID", 0, 100, mock.Anything).Return(tms, nil)
mockTeamStore.On("GetMembers", "teamID", 100, 100, mock.Anything).Return([]*model.TeamMember{{
TeamId: "1",
}}, nil)
mockStore.On("Team").Return(&mockTeamStore)
th.App.ClearTeamMembersCache("teamID")
}