You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kie.apache.org by "Abhitocode (via GitHub)" <gi...@apache.org> on 2024/08/29 18:27:48 UTC

[PR] [incubator-kie-issues-1131] test migration from V7 to code generation-34 [incubator-kie-kogito-runtimes]

Abhitocode opened a new pull request, #3653:
URL: https://github.com/apache/incubator-kie-kogito-runtimes/pull/3653

   Below are the tests with issues, commented the same for each test.
   
   1. IntermediateEventTest
   
   - For tests,
    testIntermediateCatchEventCondition, testIntermediateCatchEventConditionFilterByProcessInstance, testConditionalBoundaryEventOnTaskComplete, testConditionalBoundaryEventInterrupting
    
    for the tests with getKieSession,
    How to add the person variable to the session? The model doesnot have the person field. Even after the person variable is added in the bpmn file, the test fails with a NPE,
   
   When i try to complete workItem Im getting below error:
   
   Caused by: java.lang.NullPointerException: Cannot invoke
   "org.jbpm.process.instance.impl.AssignmentAction.execute(java.util.function.Function, java.util.function.Function, org.jbpm.process.instance.impl.AssignmentProducer)" because "action" is null  at org.jbpm.workflow.core.impl.NodeIoHelper.handleAssignment(NodeIoHelper.java:113)
   exception is this, org.jbpm.workflow.instance.WorkflowRuntimeException: [DataOutputAssociations:fa7cbf6e-fe07-408b-abcd-92ea30bed450 - Task:[uuid=_2]] -- java.lang.RuntimeException: Unable to execute Assignment
   
   2. StartEventTest
    
   - For *MultileStartEvent tests
   
   we get below error:
   org.kie.kogito.process.validation.ValidationException: A process cannot have more than one start node!
   
   BPMN process definition includes multiple start events, but I if im not wrong Kogito seems to only support a single start node per process. Could you please clarify if there's a way to handle multiple start events within a single Kogito process definition or suggest an alternative approach ?
   
   guidance on above would be helpful.
   
   Closes https://github.com/apache/incubator-kie-issues/issues/1131.
   
   Note: This pr contains tests with error, only for review purpose.
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org


Re: [PR] [incubator-kie-issues-1131] test migration from V7 to code generation-34 [incubator-kie-kogito-runtimes]

Posted by "elguardian (via GitHub)" <gi...@apache.org>.
elguardian commented on PR #3653:
URL: https://github.com/apache/incubator-kie-kogito-runtimes/pull/3653#issuecomment-2352272944

   this requires a prior fix to condition activation.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org


Re: [PR] [incubator-kie-issues-1131] test migration from V7 to code generation-34 [incubator-kie-kogito-runtimes]

Posted by "Abhitocode (via GitHub)" <gi...@apache.org>.
Abhitocode closed pull request #3653: [incubator-kie-issues-1131] test migration from V7 to code generation-34
URL: https://github.com/apache/incubator-kie-kogito-runtimes/pull/3653


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org


Re: [PR] [incubator-kie-issues-1131] test migration from V7 to code generation-34 [incubator-kie-kogito-runtimes]

Posted by "kie-ci3 (via GitHub)" <gi...@apache.org>.
kie-ci3 commented on PR #3653:
URL: https://github.com/apache/incubator-kie-kogito-runtimes/pull/3653#issuecomment-2318888027

   
   **PR job** `#1` was: **UNSTABLE**
   Possible explanation: This should be test failures
   
   
   <details>
   <summary><b>Reproducer</b></summary>
   
   
   
   build-chain build full_downstream  -f 'https://raw.githubusercontent.com/${AUTHOR:apache}/incubator-kie-kogito-pipelines/${BRANCH:main}/.ci/buildchain-config-pr-cdb.yaml' -o 'bc' -p apache/incubator-kie-kogito-runtimes -u https://github.com/apache/incubator-kie-kogito-runtimes/pull/3653 --skipParallelCheckout
   
   NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution
   
   </details>
   
   
   Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-runtimes-pr/job/PR-3653/1/display/redirect
   
   **Test results:**
   - PASSED: 3233
   - FAILED: 2
   
   Those are the test failures: 
   <details>
   <summary><a href="https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-runtimes-pr/job/PR-3653/1/testReport/org.jbpm.bpmn2/IntermediateEventTest/testIntermediateCatchEventCondition/">org.jbpm.bpmn2.IntermediateEventTest.testIntermediateCatchEventCondition</a></summary>
   <br/>expected: 1<br/> but was: 5
   </details>
   <details>
   <summary><a href="https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-runtimes-pr/job/PR-3653/1/testReport/org.jbpm.bpmn2/StartEventTest/testMultipleStartEventsRegularStart/">org.jbpm.bpmn2.StartEventTest.testMultipleStartEventsRegularStart</a></summary>
   A process cannot have more than one start node!
   </details>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org


Re: [PR] [incubator-kie-issues-1131] test migration from V7 to code generation-34 [incubator-kie-kogito-runtimes]

Posted by "elguardian (via GitHub)" <gi...@apache.org>.
elguardian commented on PR #3653:
URL: https://github.com/apache/incubator-kie-kogito-runtimes/pull/3653#issuecomment-2320210480

   the test here cannot be migrated due to kie session not working on kogito. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org