diff options
author | Jesús Espino <jespinog@gmail.com> | 2018-09-24 15:35:39 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-09-24 15:35:39 +0200 |
commit | 847f30a13b9e784e54455e84069deeeba281874d (patch) | |
tree | fc65ef9deaa6a02e0371617d78116fe5877299f5 /app/app_test.go | |
parent | 89cd752aa1bd8bc290b32cce531b47767fb3e4b8 (diff) | |
download | chat-847f30a13b9e784e54455e84069deeeba281874d.tar.gz chat-847f30a13b9e784e54455e84069deeeba281874d.tar.bz2 chat-847f30a13b9e784e54455e84069deeeba281874d.zip |
MM-11707: Change the default setting for EDIT_OTHERS_POSTS (#9447)
* MM-11707: Removes edit_others_posts permission from the team_admin role in MakeDefaultRoles().
* MM-11707: Tests fix.
* MM-11707: Update test store.
* MM-11707: Allow to change the permission for edit the others posts on TE
* Fixing tests
Diffstat (limited to 'app/app_test.go')
-rw-r--r-- | app/app_test.go | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/app/app_test.go b/app/app_test.go index 1849f1b03..c071643c9 100644 --- a/app/app_test.go +++ b/app/app_test.go @@ -152,7 +152,6 @@ func TestDoAdvancedPermissionsMigration(t *testing.T) { model.PERMISSION_CREATE_POST_PUBLIC.Id, }, "team_admin": []string{ - model.PERMISSION_EDIT_OTHERS_POSTS.Id, model.PERMISSION_REMOVE_USER_FROM_TEAM.Id, model.PERMISSION_MANAGE_TEAM.Id, model.PERMISSION_IMPORT_TEAM.Id, @@ -197,6 +196,7 @@ func TestDoAdvancedPermissionsMigration(t *testing.T) { model.PERMISSION_MANAGE_SYSTEM_WIDE_OAUTH.Id, model.PERMISSION_MANAGE_OTHERS_WEBHOOKS.Id, model.PERMISSION_EDIT_OTHER_USERS.Id, + model.PERMISSION_EDIT_OTHERS_POSTS.Id, model.PERMISSION_MANAGE_OAUTH.Id, model.PERMISSION_INVITE_USER.Id, model.PERMISSION_DELETE_POST.Id, @@ -222,7 +222,6 @@ func TestDoAdvancedPermissionsMigration(t *testing.T) { model.PERMISSION_GET_PUBLIC_LINK.Id, model.PERMISSION_CREATE_POST.Id, model.PERMISSION_USE_SLASH_COMMANDS.Id, - model.PERMISSION_EDIT_OTHERS_POSTS.Id, model.PERMISSION_REMOVE_USER_FROM_TEAM.Id, model.PERMISSION_MANAGE_TEAM.Id, model.PERMISSION_IMPORT_TEAM.Id, @@ -315,7 +314,6 @@ func TestDoAdvancedPermissionsMigration(t *testing.T) { model.PERMISSION_CREATE_POST_PUBLIC.Id, }, "team_admin": []string{ - model.PERMISSION_EDIT_OTHERS_POSTS.Id, model.PERMISSION_REMOVE_USER_FROM_TEAM.Id, model.PERMISSION_MANAGE_TEAM.Id, model.PERMISSION_IMPORT_TEAM.Id, @@ -362,6 +360,7 @@ func TestDoAdvancedPermissionsMigration(t *testing.T) { model.PERMISSION_MANAGE_SYSTEM_WIDE_OAUTH.Id, model.PERMISSION_MANAGE_OTHERS_WEBHOOKS.Id, model.PERMISSION_EDIT_OTHER_USERS.Id, + model.PERMISSION_EDIT_OTHERS_POSTS.Id, model.PERMISSION_MANAGE_OAUTH.Id, model.PERMISSION_INVITE_USER.Id, model.PERMISSION_DELETE_POST.Id, @@ -387,7 +386,6 @@ func TestDoAdvancedPermissionsMigration(t *testing.T) { model.PERMISSION_GET_PUBLIC_LINK.Id, model.PERMISSION_CREATE_POST.Id, model.PERMISSION_USE_SLASH_COMMANDS.Id, - model.PERMISSION_EDIT_OTHERS_POSTS.Id, model.PERMISSION_REMOVE_USER_FROM_TEAM.Id, model.PERMISSION_MANAGE_TEAM.Id, model.PERMISSION_IMPORT_TEAM.Id, @@ -496,6 +494,7 @@ func TestDoEmojisPermissionsMigration(t *testing.T) { model.PERMISSION_MANAGE_SYSTEM_WIDE_OAUTH.Id, model.PERMISSION_MANAGE_OTHERS_WEBHOOKS.Id, model.PERMISSION_EDIT_OTHER_USERS.Id, + model.PERMISSION_EDIT_OTHERS_POSTS.Id, model.PERMISSION_MANAGE_OAUTH.Id, model.PERMISSION_INVITE_USER.Id, model.PERMISSION_DELETE_POST.Id, @@ -521,7 +520,6 @@ func TestDoEmojisPermissionsMigration(t *testing.T) { model.PERMISSION_GET_PUBLIC_LINK.Id, model.PERMISSION_CREATE_POST.Id, model.PERMISSION_USE_SLASH_COMMANDS.Id, - model.PERMISSION_EDIT_OTHERS_POSTS.Id, model.PERMISSION_REMOVE_USER_FROM_TEAM.Id, model.PERMISSION_MANAGE_TEAM.Id, model.PERMISSION_IMPORT_TEAM.Id, @@ -549,7 +547,6 @@ func TestDoEmojisPermissionsMigration(t *testing.T) { role2, err2 := th.App.GetRoleByName(model.TEAM_ADMIN_ROLE_ID) assert.Nil(t, err2) expected2 := []string{ - model.PERMISSION_EDIT_OTHERS_POSTS.Id, model.PERMISSION_REMOVE_USER_FROM_TEAM.Id, model.PERMISSION_MANAGE_TEAM.Id, model.PERMISSION_IMPORT_TEAM.Id, |