inteligr8/alfresco-community-repo
1
0
Fork 0
mirror of https://github.com/Alfresco/alfresco-community-repo.git synced 2025-08-07 17:49:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merged V3.0 to HEAD

Browse Source 
12762: Another attempt at SiteServiceImplTest with fixes from Roy

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@12764 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Kevin Roast
2009-01-15 11:14:15 +00:00
parent b1cfa409b0
commit aa08480190
1 changed files with 4 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
source/java/org/alfresco/repo/site/SiteServiceImplTest.java
View File

@@ -163,13 +163,11 @@ public class SiteServiceImplTest extends BaseAlfrescoSpringTest
catch (Exception exception) {} catch (Exception exception) {}
try try
{ {
this.siteService.removeMembership(siteInfo.getShortName(), USER_THREE); this.siteService.removeMembership(siteInfo.getShortName(), USER_ONE);
fail("Shouldn't be able to do this cos you don't have permissions"); fail("Shouldn't be able to do this cos you don't have permissions");
} }
catch (Exception exception) {} catch (Exception exception) {}
this.siteService.removeMembership(siteInfo.getShortName(), USER_THREE);
authenticationComponent.setSystemUserAsCurrentUser();
this.siteService.removeMembership(siteInfo.getShortName(), USER_THREE);
} }
private void checkSiteInfo( SiteInfo siteInfo, String expectedSitePreset, String expectedShortName, String expectedTitle, private void checkSiteInfo( SiteInfo siteInfo, String expectedSitePreset, String expectedShortName, String expectedTitle,
@@ -444,15 +442,7 @@ public class SiteServiceImplTest extends BaseAlfrescoSpringTest
{ {
// As expected // As expected
} }
try this.siteService.removeMembership("testMembership", USER_THREE);
{
this.siteService.removeMembership("testMembership", USER_THREE);
fail("A member who isn't a manager shouldnt be able to remove a membership");
}
catch (AlfrescoRuntimeException e)
{
// As expected
}
this.authenticationComponent.setCurrentUser(USER_ONE); this.authenticationComponent.setCurrentUser(USER_ONE);
// Try and change the permissions of the only site manager // Try and change the permissions of the only site manager