mirror of
https://github.com/Alfresco/alfresco-community-repo.git
synced 2025-08-07 17:49:17 +00:00
Fix a TODO - Remove a static for a test that can be done without one
git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@31311 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
@@ -76,8 +76,7 @@ public class ScheduledPersistedActionServiceImpl implements ScheduledPersistedAc
|
|||||||
protected static final String JOB_SCHEDULE_NODEREF = "ScheduleNodeRef";
|
protected static final String JOB_SCHEDULE_NODEREF = "ScheduleNodeRef";
|
||||||
protected static final String JOB_ACTION_NODEREF = "ActionNodeRef";
|
protected static final String JOB_ACTION_NODEREF = "ActionNodeRef";
|
||||||
|
|
||||||
// TODO: Should be private NodeRef scheduledActionRootNodeRef;
|
protected NodeRef SCHEDULED_ACTION_ROOT_NODE_REF;
|
||||||
protected static NodeRef SCHEDULED_ACTION_ROOT_NODE_REF;
|
|
||||||
|
|
||||||
protected static final Set<QName> ACTION_TYPES = new HashSet<QName>(Arrays
|
protected static final Set<QName> ACTION_TYPES = new HashSet<QName>(Arrays
|
||||||
.asList(new QName[] { ActionModel.TYPE_ACTION_SCHEDULE }));
|
.asList(new QName[] { ActionModel.TYPE_ACTION_SCHEDULE }));
|
||||||
|
@@ -117,17 +117,13 @@ public class ScheduledPersistedActionServiceTest extends TestCase
|
|||||||
testAction = new TestAction(actionService
|
testAction = new TestAction(actionService
|
||||||
.createAction(SleepActionExecuter.NAME));
|
.createAction(SleepActionExecuter.NAME));
|
||||||
runtimeActionService.createActionNodeRef(
|
runtimeActionService.createActionNodeRef(
|
||||||
//
|
testAction, serviceImpl.SCHEDULED_ACTION_ROOT_NODE_REF,
|
||||||
testAction,
|
|
||||||
ScheduledPersistedActionServiceImpl.SCHEDULED_ACTION_ROOT_NODE_REF,
|
|
||||||
ContentModel.ASSOC_CONTAINS, QName.createQName("TestAction"));
|
ContentModel.ASSOC_CONTAINS, QName.createQName("TestAction"));
|
||||||
|
|
||||||
testAction2 = new TestAction(actionService
|
testAction2 = new TestAction(actionService
|
||||||
.createAction(SleepActionExecuter.NAME));
|
.createAction(SleepActionExecuter.NAME));
|
||||||
runtimeActionService.createActionNodeRef(
|
runtimeActionService.createActionNodeRef(
|
||||||
//
|
testAction2, serviceImpl.SCHEDULED_ACTION_ROOT_NODE_REF,
|
||||||
testAction2,
|
|
||||||
ScheduledPersistedActionServiceImpl.SCHEDULED_ACTION_ROOT_NODE_REF,
|
|
||||||
ContentModel.ASSOC_CONTAINS, QName.createQName("TestAction2"));
|
ContentModel.ASSOC_CONTAINS, QName.createQName("TestAction2"));
|
||||||
|
|
||||||
testAction3 = new TestAction(actionService
|
testAction3 = new TestAction(actionService
|
||||||
@@ -182,8 +178,7 @@ public class ScheduledPersistedActionServiceTest extends TestCase
|
|||||||
// Persist the 3rd action
|
// Persist the 3rd action
|
||||||
runtimeActionService.createActionNodeRef(
|
runtimeActionService.createActionNodeRef(
|
||||||
//
|
//
|
||||||
testAction3,
|
testAction3, serviceImpl.SCHEDULED_ACTION_ROOT_NODE_REF,
|
||||||
ScheduledPersistedActionServiceImpl.SCHEDULED_ACTION_ROOT_NODE_REF,
|
|
||||||
ContentModel.ASSOC_CONTAINS, QName.createQName("TestAction3"));
|
ContentModel.ASSOC_CONTAINS, QName.createQName("TestAction3"));
|
||||||
|
|
||||||
assertEquals(null, schedule.getPersistedAtNodeRef());
|
assertEquals(null, schedule.getPersistedAtNodeRef());
|
||||||
@@ -673,9 +668,7 @@ public class ScheduledPersistedActionServiceTest extends TestCase
|
|||||||
.getPersistedAtNodeRef());
|
.getPersistedAtNodeRef());
|
||||||
|
|
||||||
runtimeActionService.createActionNodeRef(
|
runtimeActionService.createActionNodeRef(
|
||||||
//
|
testAction3, serviceImpl.SCHEDULED_ACTION_ROOT_NODE_REF,
|
||||||
testAction3,
|
|
||||||
ScheduledPersistedActionServiceImpl.SCHEDULED_ACTION_ROOT_NODE_REF,
|
|
||||||
ContentModel.ASSOC_CONTAINS, QName.createQName("TestAction3"));
|
ContentModel.ASSOC_CONTAINS, QName.createQName("TestAction3"));
|
||||||
|
|
||||||
assertNotNull(schedule.getActionNodeRef());
|
assertNotNull(schedule.getActionNodeRef());
|
||||||
|
Reference in New Issue
Block a user