diff --git a/source/test-java/org/alfresco/repo/web/scripts/WebScriptTestSuite.java b/source/test-java/org/alfresco/repo/web/scripts/WebScriptTestSuite.java index 2396820098..ac1177e144 100644 --- a/source/test-java/org/alfresco/repo/web/scripts/WebScriptTestSuite.java +++ b/source/test-java/org/alfresco/repo/web/scripts/WebScriptTestSuite.java @@ -52,7 +52,6 @@ import org.alfresco.repo.web.scripts.tagging.TaggingServiceTest; import org.alfresco.repo.web.scripts.thumbnail.ThumbnailServiceTest; import org.alfresco.repo.web.scripts.transfer.TransferWebScriptTest; import org.alfresco.repo.web.scripts.workflow.ActivitiWorkflowRestApiTest; -import org.alfresco.repo.web.scripts.workflow.JBPMWorkflowRestApiTest; import org.alfresco.repo.web.scripts.workflow.WorkflowModelBuilderTest; /** @@ -95,7 +94,6 @@ public class WebScriptTestSuite extends TestSuite suite.addTestSuite( TransferWebScriptTest.class ); suite.addTestSuite( WorkflowModelBuilderTest.class ); suite.addTestSuite( ActivitiWorkflowRestApiTest.class ); - suite.addTestSuite( JBPMWorkflowRestApiTest.class ); suite.addTestSuite( PublishingRestApiTest.class ); suite.addTestSuite( SOLRWebScriptTest.class ); suite.addTestSuite( SubscriptionServiceRestApiTest.class ); diff --git a/source/test-java/org/alfresco/repo/web/scripts/invite/InviteServiceTest.java b/source/test-java/org/alfresco/repo/web/scripts/invite/InviteServiceTest.java index ee8434ca9c..f8d39a5dd4 100644 --- a/source/test-java/org/alfresco/repo/web/scripts/invite/InviteServiceTest.java +++ b/source/test-java/org/alfresco/repo/web/scripts/invite/InviteServiceTest.java @@ -38,7 +38,6 @@ import org.alfresco.repo.site.SiteModel; import org.alfresco.repo.transaction.RetryingTransactionHelper; import org.alfresco.repo.transaction.RetryingTransactionHelper.RetryingTransactionCallback; import org.alfresco.repo.web.scripts.BaseWebScriptTest; -import org.alfresco.repo.workflow.jbpm.JBPMEngine; import org.alfresco.service.cmr.repository.NodeRef; import org.alfresco.service.cmr.repository.NodeService; import org.alfresco.service.cmr.repository.datatype.DefaultTypeConverter; @@ -92,7 +91,7 @@ public class InviteServiceTest extends BaseWebScriptTest // can be removed in the tearDown() method private List inviteeEmailAddrs; - private static final String WF_DEFINITION_INVITE = WorkflowModelNominatedInvitation.WORKFLOW_DEFINITION_NAME; + private static final String WF_DEFINITION_INVITE = WorkflowModelNominatedInvitation.WORKFLOW_DEFINITION_NAME_ACTIVITI; private static final String USER_INVITER = "InviterUser"; private static final String USER_INVITER_2 = "InviterUser2"; @@ -165,14 +164,14 @@ public class InviteServiceTest extends BaseWebScriptTest { public Object doWork() throws Exception { - // redeploy invite process definition in case it has been modified - WorkflowDefinition inviteWfDefinition = workflowService.getDefinitionByName( - "jbpm$" + WorkflowModelNominatedInvitation.WF_PROCESS_INVITE.toPrefixString(namespaceService)); - workflowService.undeployDefinition(inviteWfDefinition.id); - ClassPathResource inviteWfResource = new ClassPathResource( - "alfresco/workflow/invitation-nominated_processdefinition.xml"); - workflowService.deployDefinition( - JBPMEngine.ENGINE_ID, inviteWfResource.getInputStream(), MimetypeMap.MIMETYPE_XML); +// // redeploy invite process definition in case it has been modified +// WorkflowDefinition inviteWfDefinition = workflowService.getDefinitionByName( +// "jbpm$" + WorkflowModelNominatedInvitation.WF_PROCESS_INVITE.toPrefixString(namespaceService)); +// workflowService.undeployDefinition(inviteWfDefinition.id); +// ClassPathResource inviteWfResource = new ClassPathResource( +// "alfresco/workflow/invitation-nominated_processdefinition.xml"); +// workflowService.deployDefinition( +// JBPMEngine.ENGINE_ID, inviteWfResource.getInputStream(), MimetypeMap.MIMETYPE_XML); // Create new invitee email address list inviteeEmailAddrs = new ArrayList(); diff --git a/source/test-java/org/alfresco/repo/web/scripts/workflow/JBPMWorkflowRestApiTest.java b/source/test-java/org/alfresco/repo/web/scripts/workflow/JBPMWorkflowRestApiTest.java deleted file mode 100644 index dda5b93c75..0000000000 --- a/source/test-java/org/alfresco/repo/web/scripts/workflow/JBPMWorkflowRestApiTest.java +++ /dev/null @@ -1,131 +0,0 @@ -/* - * Copyright (C) 2005-2011 Alfresco Software Limited. - * - * This file is part of Alfresco - * - * Alfresco is free software: you can redistribute it and/or modify - * it under the terms of the GNU Lesser General Public License as published by - * the Free Software Foundation, either version 3 of the License, or - * (at your option) any later version. - * - * Alfresco is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU Lesser General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public License - * along with Alfresco. If not, see . - */ -package org.alfresco.repo.web.scripts.workflow; - -import org.alfresco.repo.workflow.jbpm.JBPMEngine; -import org.json.JSONArray; -import org.json.JSONObject; -import org.springframework.extensions.webscripts.Status; -import org.springframework.extensions.webscripts.TestWebScriptServer.GetRequest; -import org.springframework.extensions.webscripts.TestWebScriptServer.Response; - -/** - * @author Nick Smith - * @author Frederik Heremans - * @since 3.4.e - */ -public class JBPMWorkflowRestApiTest extends AbstractWorkflowRestApiTest -{ - private static final String ADHOC_WORKFLOW_DEFINITION_NAME = "jbpm$wf:adhoc"; - private static final String ADHOC_WORKFLOW_DEFINITION_TITLE = "Adhoc Workflow (JBPM)"; - private static final String ADHOC_WORKFLOW_DEFINITION_DESCRIPTION = "Assign arbitrary task to colleague using JBPM workflow engine"; - private static final String REVIEW_WORKFLOW_DEFINITION_NAME = "jbpm$wf:review"; - private static final String REVIEW_POOLED_WORKFLOW_DEFINITION_NAME = "jbpm$wf:reviewpooled"; - - @Override - protected String getAdhocWorkflowDefinitionName() - { - return ADHOC_WORKFLOW_DEFINITION_NAME; - } - - @Override - protected String getAdhocWorkflowDefinitionTitle() - { - return ADHOC_WORKFLOW_DEFINITION_TITLE; - } - - @Override - protected String getAdhocWorkflowDefinitionDescription() - { - return ADHOC_WORKFLOW_DEFINITION_DESCRIPTION; - } - - @Override - protected String getReviewWorkflowDefinitionName() - { - return REVIEW_WORKFLOW_DEFINITION_NAME; - } - - @Override - protected String getReviewPooledWorkflowDefinitionName() - { - return REVIEW_POOLED_WORKFLOW_DEFINITION_NAME; - } - - @Override - protected void approveTask(String taskId) throws Exception - { - String transition = "approve"; - checkTransitionExists(transition, taskId); - - // Take transition - workflowService.endTask(taskId, transition); - } - - @Override - protected void rejectTask(String taskId) throws Exception - { - String transition = "reject"; - checkTransitionExists(transition, taskId); - - // Take transition - workflowService.endTask(taskId, transition); - } - - private void checkTransitionExists(String transitionName, String taskId) throws Exception - { - // Transition to "Approve", check if transition is available - Response response = sendRequest(new GetRequest(URL_TASKS + "/" + taskId), Status.STATUS_OK); - String jsonStr = response.getContentAsString(); - JSONObject json = new JSONObject(jsonStr); - JSONObject result = json.getJSONObject("data"); - assertNotNull(result); - - JSONObject definition = result.getJSONObject("definition"); - assertNotNull(definition); - JSONObject node = definition.getJSONObject("node"); - assertNotNull(node); - JSONArray transitions = node.getJSONArray("transitions"); - assertNotNull(transitions); - - boolean found = false; - // Find approve transition - for (int i=0; i < transitions.length(); i++) - { - String name = transitions.getJSONObject(i).getString("id"); - if (name.equals("approve")) - { - found = true; - break; - } - } - assertTrue("Transition 'Approve' not found", found); - } - - public void testNothing() throws Exception - { - //NOOP - } - - @Override - protected String getEngine() - { - return JBPMEngine.ENGINE_ID; - } -}