Bug 777951 (SOA-481)

Summary: jBPM example "action" - wrong package name referenced in JUnit tests
Product: [JBoss] JBoss Enterprise SOA Platform 4 Reporter: Len DiMaggio <ldimaggi>
Component: JBPM - within SOAAssignee: Tom Baeyens <mail>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: 4.2 GA   
Target Milestone: ---   
Target Release: 4.2 CP02   
Hardware: Unspecified   
OS: Unspecified   
URL: http://jira.jboss.org/jira/browse/SOA-481
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-06-11 17:21:52 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Len DiMaggio 2008-03-28 19:31:15 UTC 
Date of First Response: 2008-06-04 11:53:07
project_key: SOA

In ActionTest.java - need to replace references 

to:
   org.jbpm.tutorial.action.MyActionHandler
with:
   org.jbpm.examples.action.MyActionHandler
Comment 1 Len DiMaggio 2008-03-28 19:32:58 UTC 
Link: Added: This issue is a dependency of JBPM-1154
Comment 2 Len DiMaggio 2008-06-04 14:25:51 UTC 
It's fixed in cvs head - when will 3.2.3 be available?

cvs -d :pserver:anonymous.jboss.com:/cvsroot/jbpm co jbpm.3

  public void testTransitionAction() {
    // The next process is a variant of the hello world process.
    // We have added an action on the transition from state s 
    // to the end-state.  The purpose of this test is to show 
    // how easy it is to integrate java code in a jBPM process.
    ProcessDefinition processDefinition = ProcessDefinition.parseXmlString(
      "<process-definition>" +
      "  <start-state>" +
      "    <transition to='s' />" +
      "  </start-state>" +
      "  <state name='s'>" +
      "    <transition to='end'>" +
      "      <action class='org.jbpm.examples.action.MyActionHandler' />" +
      "    </transition>" +
      "  </state>" +
      "  <end-state name='end' />" +
      "</process-definition>"
    );
Comment 3 Julian Coleman 2008-06-04 15:53:07 UTC 
Fixed on branch BRANCH_3_2_2_SOA_4_2 in JBPM repository with revision 1.1.2.1
of:
  jbpm/jbpm.3/jpdl/examples/action/src/main/java/org/jbpm/examples/action/ActionTest.java

Commit message:
  Pull up revisions 1.2 and 1.3:

    changed classname of actionhandler as per JBPM-1154

    second processdefinition also changed...

  to fix JIRA SOA-481.
Comment 4 Len DiMaggio 2008-06-11 17:21:52 UTC 
Updated in SOA-P CP02:

[ldimaggi@ldimaggi examples]$ pwd
/opt/CP02/jboss-soa-p-standalone.4.2.0/jbpm-jpdl/examples
[ldimaggi@ldimaggi examples]$ find . -name ActionTest.java -ls
7771098   12 -rw-r--r--   1 ldimaggi ldimaggi     4518 Jun  4 11:41 ./action/src/main/java/org/jbpm/examples/action/ActionTest.java

Matches cvs head for jBPM:

[ldimaggi@ldimaggi examples]$ pwd
/opt/jbpm.3/jpdl/examples
[ldimaggi@ldimaggi examples]$ find . -name ActionTest.java -ls
3016297   12 -rw-rw-r--   1 ldimaggi ldimaggi     4518 Mar 29 12:22 ./action/src/main/java/org/jbpm/examples/action/ActionTest.java

And - the code is correct too:

    ProcessDefinition processDefinition = ProcessDefinition.parseXmlString(
      "<process-definition>" +
      "  <start-state>" +
      "    <transition to='s' />" +
      "  </start-state>" +
      "  <state name='s'>" +
      "    <transition to='end'>" +
      "      <action class='org.jbpm.examples.action.MyActionHandler' />" +
      "    </transition>" +
      "  </state>" +
      "  <end-state name='end' />" +
      "</process-definition>"
    );

(The test runs successfully too.)