From 3b83cc7dd3fc8c6281bbd74b5b85a6a06efcbb6d Mon Sep 17 00:00:00 2001 From: Martin Kraft Date: Fri, 9 Feb 2018 10:57:07 -0500 Subject: 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. --- app/app_test.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'app/app_test.go') 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. -- cgit v1.2.3-1-g7c22