From eff32bc2df7ee61244cba8a3953bfaf6251e4318 Mon Sep 17 00:00:00 2001 From: Neil McErlean Date: Fri, 8 Mar 2013 10:08:18 +0000 Subject: [PATCH] Another build fix. This one fixes LinksServiceImplTest.linksListingPermissionsChecking() I don't know how this ever passed, but it did. I simply added some missing transaction wrapping. git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@47791 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261 --- .../org/alfresco/repo/links/LinksServiceImplTest.java | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/source/java/org/alfresco/repo/links/LinksServiceImplTest.java b/source/java/org/alfresco/repo/links/LinksServiceImplTest.java index b32dde4945..787c0cd70d 100644 --- a/source/java/org/alfresco/repo/links/LinksServiceImplTest.java +++ b/source/java/org/alfresco/repo/links/LinksServiceImplTest.java @@ -491,8 +491,15 @@ public class LinksServiceImplTest assertEquals(0, results.getPage().size()); // Double check that we're only allowed to see the 1st site - assertEquals(true, SITE_SERVICE.isMember(LINKS_SITE.getShortName(), TEST_USER)); - assertEquals(false, SITE_SERVICE.isMember(ALTERNATE_LINKS_SITE.getShortName(), TEST_USER)); + TRANSACTION_HELPER.doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback() + { + @Override public Void execute() throws Throwable + { + assertEquals(true, SITE_SERVICE.isMember(LINKS_SITE.getShortName(), TEST_USER)); + assertEquals(false, SITE_SERVICE.isMember(ALTERNATE_LINKS_SITE.getShortName(), TEST_USER)); + return null; + } + }); // Now become the test user