tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

MINOR: Don't couple GroupTest to the Member_GroupSet object.

Browse Source
This commit is contained in:
Sam Minnee 2009-11-21 17:43:49 +13:00
parent 79e0634537
commit 3619eae9eb
1 changed files with 4 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
tests/security/GroupTest.php
View File

@ -62,13 +62,9 @@ class GroupTest extends FunctionalTest {
$form->saveInto($member); $form->saveInto($member);
$updatedGroups = $member->Groups(); $updatedGroups = $member->Groups();
$controlGroups = new Member_GroupSet(
$adminGroup,
$parentGroup
);
$this->assertEquals( $this->assertEquals(
$updatedGroups->Map('ID','ID'), array($adminGroup->ID, $parentGroup->ID),
$controlGroups->Map('ID','ID'), $updatedGroups->column(),
"Adding a toplevel group works" "Adding a toplevel group works"
); );
@ -82,12 +78,9 @@ class GroupTest extends FunctionalTest {
$form->saveInto($member); $form->saveInto($member);
$member->flushCache(); $member->flushCache();
$updatedGroups = $member->Groups(); $updatedGroups = $member->Groups();
$controlGroups = new Member_GroupSet(
$adminGroup
);
$this->assertEquals( $this->assertEquals(
$updatedGroups->Map('ID','ID'), array($adminGroup->ID),
$controlGroups->Map('ID','ID'), $updatedGroups->column(),
"Removing a previously added toplevel group works" "Removing a previously added toplevel group works"
); );