diff options
| author | Martin Kraft <mkraft@users.noreply.github.com> | 2018-02-09 10:57:07 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-02-09 10:57:07 -0500 |
| commit | 3b83cc7dd3fc8c6281bbd74b5b85a6a06efcbb6d (patch) | |
| tree | 214b35bb91e99a259b11fbf4a44c4e06adbfe9b3 /app/app_test.go | |
| parent | 0aa7ecd5e89f054ae927b246f2aec4bd6348d42b (diff) | |
| download | chat-3b83cc7dd3fc8c6281bbd74b5b85a6a06efcbb6d.tar.gz chat-3b83cc7dd3fc8c6281bbd74b5b85a6a06efcbb6d.tar.bz2 chat-3b83cc7dd3fc8c6281bbd74b5b85a6a06efcbb6d.zip | |
XYZ-51: Unit tests for and changes to SetRolePermissionsFromConfig. (#8160)
* XYZ-10: Role store.
* XYZ-37: Update unit tests to work with database roles.
* XYZ-51: Tests 'SetRolePermissionsFromConfig' against JSON from policy page.
* XYZ-51: Adds permissions in non-licensed cases also.
* XYZ-51: Removes some permissions from team_user role.
* XYZ-51: Merge fix for change to default permissions from PR 8208.
* XYZ-51: Removes unused function.
Diffstat (limited to 'app/app_test.go')
| -rw-r--r-- | app/app_test.go | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/app/app_test.go b/app/app_test.go index bbc012364..180bd21ee 100644 --- a/app/app_test.go +++ b/app/app_test.go @@ -5,6 +5,7 @@ package app import ( "flag" + "fmt" "os" "testing" @@ -275,6 +276,8 @@ func TestDoAdvancedPermissionsMigration(t *testing.T) { model.PERMISSION_GET_PUBLIC_LINK.Id, model.PERMISSION_CREATE_POST.Id, model.PERMISSION_USE_SLASH_COMMANDS.Id, + model.PERMISSION_DELETE_PUBLIC_CHANNEL.Id, + model.PERMISSION_DELETE_PRIVATE_CHANNEL.Id, model.PERMISSION_MANAGE_PRIVATE_CHANNEL_MEMBERS.Id, model.PERMISSION_DELETE_POST.Id, model.PERMISSION_EDIT_POST.Id, @@ -389,7 +392,7 @@ func TestDoAdvancedPermissionsMigration(t *testing.T) { for name, permissions := range expected2 { role, err := th.App.GetRoleByName(name) assert.Nil(t, err) - assert.Equal(t, permissions, role.Permissions) + assert.Equal(t, permissions, role.Permissions, fmt.Sprintf("'%v' did not have expected permissions", name)) } // Remove the license. |