SilverStripe\Security\Tests\PermissionTest

Complexity

Total Complexity

17

Size/Duplication

Total Lines	271
Duplicated Lines	0 %

Importance

Changes

0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	testGettingMembersByPermission()	0	14	2
A	testPermissionAreInheritedFromOneRole()	0	6	1
A	testRolesAndPermissionsFromParentGroupsAreInherited()	0	16	1
A	testDirectlyAppliedPermissions()	0	4	1
A	testPermissionsForMember()	0	16	1
A	testGetCodesGrouped()	0	4	1
A	testHiddenPermissions()	0	11	1
A	testEmptyMemberFails()	0	9	1
A	testCMSAccess()	0	21	2
A	testGetCodesUngrouped()	0	4	1
A	testLeftAndMainAccessAll()	0	8	1
A	testPermissionAreInheritedFromMultipleRoles()	0	8	1
A	testDenyThenGrantPermission()	0	45	1
A	testGrantPermission()	0	35	1
A	testDenyPermission()	0	34	1

<?php

namespace SilverStripe\Security\Tests;

use SilverStripe\Security\Permission;
use SilverStripe\Security\Group;
use SilverStripe\Security\Member;
use SilverStripe\Security\PermissionCheckboxSetField;
use SilverStripe\Core\Config\Config;
use SilverStripe\Dev\SapphireTest;

/**
 * @skipUpgrade
 */
class PermissionTest extends SapphireTest
{

    protected static $fixture_file = 'PermissionTest.yml';

    public function testGetCodesGrouped()
    {
        $codes = Permission::get_codes();
        $this->assertArrayNotHasKey('SITETREE_VIEW_ALL', $codes);
    }

    public function testGetCodesUngrouped()
    {
        $codes = Permission::get_codes(false);
        $this->assertArrayHasKey('SITETREE_VIEW_ALL', $codes);
    }

    public function testDirectlyAppliedPermissions()
    {
        $member = $this->objFromFixture(Member::class, 'author');
        $this->assertTrue(Permission::checkMember($member, "SITETREE_VIEW_ALL"));
    }

    public function testCMSAccess()
    {
        $members = Member::get()->byIDs($this->allFixtureIDs(Member::class));
        foreach ($members as $member) {
            $this->assertTrue(Permission::checkMember($member, 'CMS_ACCESS'));
            $this->assertTrue(Permission::checkMember($member, ['CMS_ACCESS', 'CMS_ACCESS_Security']));
            $this->assertTrue(Permission::checkMember($member, ['CMS_ACCESS_Security', 'CMS_ACCESS']));
        }

        $member = new Member();
        $member->update(
            [
            'FirstName' => 'No CMS',
            'Surname' => 'Access',
            'Email' => 'no-access@example.com',
            ]
        );
        $member->write();
        $this->assertFalse(Permission::checkMember($member, 'CMS_ACCESS'));
        $this->assertFalse(Permission::checkMember($member, ['CMS_ACCESS', 'CMS_ACCESS_Security']));
        $this->assertFalse(Permission::checkMember($member, ['CMS_ACCESS_Security', 'CMS_ACCESS']));
    }

    public function testLeftAndMainAccessAll()
    {
        //add user and group
        $member = $this->objFromFixture(Member::class, 'leftandmain');

        $this->assertTrue(Permission::checkMember($member, "CMS_ACCESS_MyAdmin"));
        $this->assertTrue(Permission::checkMember($member, "CMS_ACCESS_AssetAdmin"));
        $this->assertTrue(Permission::checkMember($member, "CMS_ACCESS_SecurityAdmin"));
    }

    public function testPermissionAreInheritedFromOneRole()
    {
        $member = $this->objFromFixture(Member::class, 'author');
        $this->assertTrue(Permission::checkMember($member, "CMS_ACCESS_MyAdmin"));
        $this->assertTrue(Permission::checkMember($member, "CMS_ACCESS_AssetAdmin"));
        $this->assertFalse(Permission::checkMember($member, "CMS_ACCESS_SecurityAdmin"));
    }

    public function testPermissionAreInheritedFromMultipleRoles()
    {
        $member = $this->objFromFixture(Member::class, 'access');
        $this->assertTrue(Permission::checkMember($member, "CMS_ACCESS_MyAdmin"));
        $this->assertTrue(Permission::checkMember($member, "CMS_ACCESS_AssetAdmin"));
        $this->assertTrue(Permission::checkMember($member, "CMS_ACCESS_SecurityAdmin"));
        $this->assertTrue(Permission::checkMember($member, "EDIT_PERMISSIONS"));
        $this->assertFalse(Permission::checkMember($member, "SITETREE_VIEW_ALL"));
    }

    public function testPermissionsForMember()
    {
        $member = $this->objFromFixture(Member::class, 'access');
        $permissions = Permission::permissions_for_member($member->ID);
        $this->assertEquals(4, count($permissions ?? []));
        $this->assertTrue(in_array('CMS_ACCESS_MyAdmin', $permissions ?? []));
        $this->assertTrue(in_array('CMS_ACCESS_AssetAdmin', $permissions ?? []));
        $this->assertTrue(in_array('CMS_ACCESS_SecurityAdmin', $permissions ?? []));
        $this->assertTrue(in_array('EDIT_PERMISSIONS', $permissions ?? []));

        $group = $this->objFromFixture("SilverStripe\\Security\\Group", "access");

        Permission::deny($group->ID, "CMS_ACCESS_MyAdmin");
        $permissions = Permission::permissions_for_member($member->ID);
        $this->assertEquals(3, count($permissions ?? []));
        $this->assertFalse(in_array('CMS_ACCESS_MyAdmin', $permissions ?? []));
    }

    public function testRolesAndPermissionsFromParentGroupsAreInherited()
    {
        $member = $this->objFromFixture(Member::class, 'globalauthor');

        // Check that permissions applied to the group are there
        $this->assertTrue(Permission::checkMember($member, "SITETREE_EDIT_ALL"));

        // Check that roles from parent groups are there
        $this->assertTrue(Permission::checkMember($member, "CMS_ACCESS_MyAdmin"));
        $this->assertTrue(Permission::checkMember($member, "CMS_ACCESS_AssetAdmin"));

        // Check that permissions from parent groups are there
        $this->assertTrue(Permission::checkMember($member, "SITETREE_VIEW_ALL"));

        // Check that a random permission that shouldn't be there isn't
        $this->assertFalse(Permission::checkMember($member, "CMS_ACCESS_SecurityAdmin"));
    }
    /**
     * Ensure the the get_*_by_permission functions are permission role aware
     */
    public function testGettingMembersByPermission()
    {
        $accessMember = $this->objFromFixture(Member::class, 'access');
        $accessAuthor = $this->objFromFixture(Member::class, 'author');

        $result = Permission::get_members_by_permission(['CMS_ACCESS_SecurityAdmin']);
        $resultIDs = $result ? $result->column() : [];

        $this->assertContains(
            $accessMember->ID,
            $resultIDs,
            'Member is found via a permission attached to a role'
        );
        $this->assertNotContains($accessAuthor->ID, $resultIDs);
    }


    public function testHiddenPermissions()
    {
        $permissionCheckboxSet = new PermissionCheckboxSetField('Permissions', 'Permissions', Permission::class, 'GroupID');
        $this->assertStringContainsString('CMS_ACCESS_LeftAndMain', $permissionCheckboxSet->Field());

        Config::modify()->merge(Permission::class, 'hidden_permissions', ['CMS_ACCESS_LeftAndMain']);

        $this->assertStringNotContainsString('CMS_ACCESS_LeftAndMain', $permissionCheckboxSet->Field());

        Config::inst()->remove(Permission::class, 'hidden_permissions');

The method remove() does not exist on SilverStripe\Config\Collections\ConfigCollectionInterface. It seems like you code against a sub-type of SilverStripe\Config\Collections\ConfigCollectionInterface such as SilverStripe\Config\Collections\MutableConfigCollectionInterface.

        $this->assertStringContainsString('CMS_ACCESS_LeftAndMain', $permissionCheckboxSet->Field());
    }

    public function testEmptyMemberFails()
    {
        $member = new Member();
        $this->assertFalse($member->exists());

        $this->logInWithPermission('ADMIN');

        $this->assertFalse(Permission::checkMember($member, 'ADMIN'));
        $this->assertFalse(Permission::checkMember($member, 'CMS_ACCESS_LeftAndMain'));
    }

    public function testGrantPermission()
    {
        $group = $this->objFromFixture(Group::class, 'testpermissiongroup');
        $id = $group->ID;

        Permission::grant($id, 'CMS_ACCESS_CMSMain');
        Permission::grant($id, 'CMS_ACCESS_AssetAdmin');
        Permission::grant($id, 'CMS_ACCESS_ReportAdmin');

        $groupPermission = Permission::get()->filter(['GroupID' => $id]);

        $this->assertEquals(3, $groupPermission->count());
        $this->assertEquals(0, $groupPermission->first()->Arg);

The property Arg does not exist on SilverStripe\ORM\DataObject. Since you implemented __get, consider adding a @property annotation.

        $this->assertEquals(1, $groupPermission->first()->Type);

The property Type does not exist on SilverStripe\ORM\DataObject. Since you implemented __get, consider adding a @property annotation.

        Permission::grant($id, 'CMS_ACCESS_CMSMain', 'all');
        Permission::grant($id, 'CMS_ACCESS_AssetAdmin', 'all');
        Permission::grant($id, 'CMS_ACCESS_ReportAdmin', 'all');

        $groupPermission = Permission::get()->filter(['GroupID' => $id]);

        $this->assertEquals(3, $groupPermission->count());
        $this->assertEquals(-1, $groupPermission->first()->Arg);
        $this->assertEquals(1, $groupPermission->first()->Type);

        Permission::grant($id, 'CMS_ACCESS_CMSMain', 'any');
        Permission::grant($id, 'CMS_ACCESS_AssetAdmin', 'any');
        Permission::grant($id, 'CMS_ACCESS_ReportAdmin', 'any');

        $groupPermission = Permission::get()->filter(['GroupID' => $id]);

        $this->assertEquals(3, $groupPermission->count());
        $this->assertEquals(-1, $groupPermission->first()->Arg);
        $this->assertEquals(1, $groupPermission->first()->Type);
    }

    public function testDenyPermission()
    {
        $group = $this->objFromFixture(Group::class, 'testpermissiongroup');
        $id = $group->ID;

        Permission::deny($id, 'CMS_ACCESS_CMSMain');
        Permission::deny($id, 'CMS_ACCESS_AssetAdmin');
        Permission::deny($id, 'CMS_ACCESS_ReportAdmin');

        $groupPermission = Permission::get()->filter(['GroupID' => $id]);

        $this->assertEquals(3, $groupPermission->count());
        $this->assertEquals(0, $groupPermission->first()->Arg);

The property Arg does not exist on SilverStripe\ORM\DataObject. Since you implemented __get, consider adding a @property annotation.

        $this->assertEquals(-1, $groupPermission->first()->Type);

The property Type does not exist on SilverStripe\ORM\DataObject. Since you implemented __get, consider adding a @property annotation.

        Permission::deny($id, 'CMS_ACCESS_CMSMain', 'all');
        Permission::deny($id, 'CMS_ACCESS_AssetAdmin', 'all');
        Permission::deny($id, 'CMS_ACCESS_ReportAdmin', 'all');

        $groupPermission = Permission::get()->filter(['GroupID' => $id]);

        $this->assertEquals(3, $groupPermission->count());
        $this->assertEquals(-1, $groupPermission->first()->Arg);
        $this->assertEquals(-1, $groupPermission->first()->Type);

        Permission::deny($id, 'CMS_ACCESS_CMSMain', 'any');
        Permission::deny($id, 'CMS_ACCESS_AssetAdmin', 'any');
        Permission::deny($id, 'CMS_ACCESS_ReportAdmin', 'any');

        $groupPermission = Permission::get()->filter(['GroupID' => $id]);

        $this->assertEquals(3, $groupPermission->count());
        $this->assertEquals(-1, $groupPermission->first()->Arg);
        $this->assertEquals(-1, $groupPermission->first()->Type);
    }

    public function testDenyThenGrantPermission()
    {
        $member = $this->objFromFixture(Member::class, 'testcmseditormember');
        $group = $this->objFromFixture(Group::class, 'testcmseditorgroup');
        $id = $group->ID;

        $this->logInAs($member);

        Permission::grant($id, 'TEST_CMS_EDITOR');
        $groupPermission = Permission::get()->filter(['GroupID' => $id]);

        $this->assertEquals(1, $groupPermission->count());
        $this->assertEquals(1, $groupPermission->first()->Type);

The property Type does not exist on SilverStripe\ORM\DataObject. Since you implemented __get, consider adding a @property annotation.

        $this->assertTrue(Permission::check('TEST_CMS_EDITOR'));

        Permission::deny($id, 'TEST_CMS_EDITOR');
        $groupPermission = Permission::get()->filter(['GroupID' => $id]);

        $this->assertEquals(1, $groupPermission->count());
        $this->assertEquals(-1, $groupPermission->last()->Type);
        $this->assertFalse(Permission::check('TEST_CMS_EDITOR'));

        Permission::grant($id, 'TEST_CMS_EDITOR');
        $groupPermission = Permission::get()->filter(['GroupID' => $id]);

        $this->assertEquals(1, $groupPermission->count());
        $this->assertEquals(1, $groupPermission->first()->Type);
        $this->assertTrue(Permission::check('TEST_CMS_EDITOR'));

        Permission::grant($id, 'CMS_ACCESS_AssetAdmin');
        $groupPermission = Permission::get()->filter(['GroupID' => $id]);
        $this->assertEquals(2, $groupPermission->count());

        $groupPermissionAssetAdmin = Permission::get()->filter(
            [
                'GroupID' => $id,
                'Code' => 'CMS_ACCESS_AssetAdmin',
            ]
        );
        $this->assertEquals(1, $groupPermissionAssetAdmin->count());
        $this->assertEquals(1, $groupPermissionAssetAdmin->first()->Type);

        $this->assertTrue(Permission::check('CMS_ACCESS_AssetAdmin'));

        $this->logOut();
    }
}