Fix failing unit tests

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@43174 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Dave Ward
2012-10-28 10:57:36 +00:00
parent de47da1766
commit 05c5279912
2 changed files with 78 additions and 79 deletions

View File

@@ -116,13 +116,13 @@ public class RepoDictionaryDAOTest extends TestCase
private void initDictionaryCaches(DictionaryDAOImpl dictionaryDAO) private void initDictionaryCaches(DictionaryDAOImpl dictionaryDAO)
{ {
SimpleCache<String,DictionaryRegistry> dictionaryCache = new DefaultSimpleCache<String, DictionaryRegistry>(11, getClass().getName()); SimpleCache<String,DictionaryRegistry> dictionaryCache = new DefaultSimpleCache<String, DictionaryRegistry>(11, getClass().getName() + ".dictionary");
dictionaryDAO.setDictionaryRegistryCache(dictionaryCache); dictionaryDAO.setDictionaryRegistryCache(dictionaryCache);
} }
private void initNamespaceCaches(NamespaceDAOImpl namespaceDAO) private void initNamespaceCaches(NamespaceDAOImpl namespaceDAO)
{ {
SimpleCache<String, NamespaceRegistry> namespaceCache = new NullCache<String, NamespaceRegistry>(); SimpleCache<String, NamespaceRegistry> namespaceCache = new DefaultSimpleCache<String, NamespaceRegistry>(11, getClass().getName() + ".namespace");
namespaceDAO.setNamespaceRegistryCache(namespaceCache); namespaceDAO.setNamespaceRegistryCache(namespaceCache);
} }

View File

@@ -206,25 +206,25 @@ public class SubscriptionServiceActivitiesTest extends TestCase
createSite(userId1+"mod2", SiteVisibility.MODERATED); createSite(userId1+"mod2", SiteVisibility.MODERATED);
siteService.setMembership(userId1+"mod2", userId1, SiteModel.SITE_MANAGER); siteService.setMembership(userId1+"mod2", userId1, SiteModel.SITE_MANAGER);
List<String> feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 0, feed.size());
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 0, feed.size());
// userId1 + 5, userId2 + 0
generateFeed();
feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 5, feed.size());
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 0, feed.size());
return null; return null;
} }
}, AuthenticationUtil.getAdminUserName()); }, AuthenticationUtil.getAdminUserName());
List<String> feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 0, feed.size());
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 0, feed.size());
// userId1 + 5, userId2 + 0
generateFeed();
feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 5, feed.size());
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 0, feed.size());
AuthenticationUtil.runAs(new AuthenticationUtil.RunAsWork<Object>() AuthenticationUtil.runAs(new AuthenticationUtil.RunAsWork<Object>()
{ {
@Override @Override
@@ -245,14 +245,22 @@ public class SubscriptionServiceActivitiesTest extends TestCase
} }
}, userId2); }, userId2);
// userId1 + 5, userId2 + 2 AuthenticationUtil.runAs(new AuthenticationUtil.RunAsWork<Object>()
generateFeed(); {
@Override
feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null); public Object doWork() throws Exception
assertEquals(feed.toString(), 7, feed.size()); {
// userId1 + 5, userId2 + 2
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null); generateFeed();
assertEquals(feed.toString(), 2, feed.size());
List<String> feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 7, feed.size());
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 2, feed.size());
return null;
}
}, AuthenticationUtil.getAdminUserName());
AuthenticationUtil.runAs(new AuthenticationUtil.RunAsWork<Object>() AuthenticationUtil.runAs(new AuthenticationUtil.RunAsWork<Object>()
{ {
@@ -268,38 +276,38 @@ public class SubscriptionServiceActivitiesTest extends TestCase
} }
}, userId1); }, userId1);
// userId1 + 5, userId2 + 1
generateFeed();
feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 12, feed.size());
// note: userId2 should not see activities from followers in moderated sites that they do not belong do (ALF-16460)
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 3, feed.size());
AuthenticationUtil.runAs(new AuthenticationUtil.RunAsWork<Object>() AuthenticationUtil.runAs(new AuthenticationUtil.RunAsWork<Object>()
{ {
@Override @Override
public Object doWork() throws Exception public Object doWork() throws Exception
{ {
// userId1 + 5, userId2 + 1
generateFeed();
List <String> feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 12, feed.size());
// note: userId2 should not see activities from followers in moderated sites that they do not belong do (ALF-16460)
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 3, feed.size());
siteService.setMembership(userId1+"priv2", userId2, SiteModel.SITE_CONSUMER); siteService.setMembership(userId1+"priv2", userId2, SiteModel.SITE_CONSUMER);
siteService.setMembership(userId1+"mod2", userId2, SiteModel.SITE_MANAGER); siteService.setMembership(userId1+"mod2", userId2, SiteModel.SITE_MANAGER);
// userId1 + 2, userId2 + 2
generateFeed();
feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 14, feed.size());
// note: userId2 should not see activities from followers in moderated sites that they do not belong do (ALF-16460)
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 5, feed.size());
return null; return null;
} }
}, AuthenticationUtil.getAdminUserName()); }, AuthenticationUtil.getAdminUserName());
// userId1 + 2, userId2 + 2
generateFeed();
feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 14, feed.size());
// note: userId2 should not see activities from followers in moderated sites that they do not belong do (ALF-16460)
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 5, feed.size());
AuthenticationUtil.runAs(new AuthenticationUtil.RunAsWork<Object>() AuthenticationUtil.runAs(new AuthenticationUtil.RunAsWork<Object>()
{ {
@Override @Override
@@ -315,53 +323,44 @@ public class SubscriptionServiceActivitiesTest extends TestCase
} }
}, userId1); }, userId1);
// userId1 + 5, userId2 + 3
generateFeed();
feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 19, feed.size());
// note: userId2 should not see activities from followers in moderated sites that they do not belong do (ALF-16460)
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 8, feed.size());
AuthenticationUtil.runAs(new AuthenticationUtil.RunAsWork<Object>() AuthenticationUtil.runAs(new AuthenticationUtil.RunAsWork<Object>()
{ {
@Override @Override
public Object doWork() throws Exception public Object doWork() throws Exception
{ {
// userId1 + 5, userId2 + 3
generateFeed();
List<String> feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 19, feed.size());
// note: userId2 should not see activities from followers in moderated sites that they do not belong do (ALF-16460)
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 8, feed.size());
deleteSite(userId1+"pub"); deleteSite(userId1+"pub");
deleteSite(userId1+"priv1"); deleteSite(userId1+"priv1");
deleteSite(userId1+"priv2"); deleteSite(userId1+"priv2");
deleteSite(userId1+"mod1"); deleteSite(userId1+"mod1");
deleteSite(userId1+"mod2"); deleteSite(userId1+"mod2");
return null; feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
} assertEquals(feed.toString(), 2, feed.size());
}, AuthenticationUtil.getAdminUserName());
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null); assertEquals(feed.toString(), 2, feed.size());
assertEquals(feed.toString(), 2, feed.size());
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 2, feed.size());
AuthenticationUtil.runAs(new AuthenticationUtil.RunAsWork<Object>()
{
@Override
public Object doWork() throws Exception
{
deletePerson(userId1); deletePerson(userId1);
deletePerson(userId2); deletePerson(userId2);
feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 0, feed.size());
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 0, feed.size());
return null; return null;
} }
}, AuthenticationUtil.getAdminUserName()); }, AuthenticationUtil.getAdminUserName());
feed = activityService.getUserFeedEntries(userId1, "json", null, false, false, null, null);
assertEquals(feed.toString(), 0, feed.size());
feed = activityService.getUserFeedEntries(userId2, "json", null, false, false, null, null);
assertEquals(feed.toString(), 0, feed.size());
} }
} }