Bug 1075134

Summary: Simulation Engine: Simulation which contains parallel gateway fails
Product: [Retired] JBoss BPMS Platform 6 Reporter: Sona Mala <smala>
Component: jBPM DesignerAssignee: Maciej Swiderski <mswiders>
Status: CLOSED CURRENTRELEASE QA Contact: Sona Mala <smala>
Severity: high Docs Contact:
Priority: high    
Version: 6.0.1CC: kverlaen, mbiarnes
Target Milestone: CR1Keywords: Regression
Target Release: 6.0.1   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-08-06 20:02:05 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:
Embargoed:

Description Sona Mala 2014-03-11 14:43:38 UTC
Description of problem:

Simulation ends with an alert "Unable to perform simulation: Process is not valid for simulation - use validation to find errors". This alert was added with related issue.

The condition, which was added by the related issue, does not reflect type of gateway. Not sure how BPMN2SimulationDataProvider works, but I think that its behaviour is gateway's type related.


Version-Release number of selected component (if applicable):
6.0.1 ER2 BPMS

How reproducible:
Use parallel gateway instead of XOR gateway

Steps to Reproduce:
1. create process (Start->Task->XOR->Task->End)
                                 |->Task->End
2. set probability for outgoing sequence flow to the sum is equal to 100
3. run simulation
4. change XOR to Parallel gateway (do not change probability)
5. run simulation

Actual results:
step 3: simulation returns results
step 5: an alert is shown


Expected results:
step 5: simulation returns results which are expected for Parallel gateway (all outgoing sequence flow are executed)

I think that the condition should be gateway's type related.

Additional info:

Comment 1 Maciej Swiderski 2014-03-12 10:16:30 UTC
Sona, very good catch. I missed gateway check while verifying probability.

droolsjbpm-integration
master:
https://github.com/droolsjbpm/droolsjbpm-integration/commit/9dbeb471c3b2485662ebde90815c957731e68324

6.0.x:
https://github.com/droolsjbpm/droolsjbpm-integration/commit/310f576346ceabb3f04ad7ba971f763137e3737f

Comment 2 Michael 2014-03-17 10:50:00 UTC
Ticket cherry-picked into 6.0.1.CR1. 
Commit: 2c098bbb655bf818f206be81e79b27a78bb4b7f5

Comment 3 Sona Mala 2014-03-24 08:45:55 UTC
Verified for 6.0.1.CR1 BPMS.

Number of instances is right and an executed path is right,