Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[incubator-kie-issues-1131] test migration from V7 to code generation-23 #3635

Closed

Conversation

Abhitocode
Copy link
Contributor

@Abhitocode Abhitocode commented Aug 23, 2024

Issue's with testEventBasedSplit3 and testEventBaedSplit5 tests within IntermediateEventTest.java

The test can be identified by referring to IntermediateEventTest.java:
https:/apache/incubator-kie-kogito-runtimes/blob/main/jbpm/jbpm-tests/src/test/java/org/jbpm/bpmn2/IntermediateEventTest.java

Closes apache/incubator-kie-issues#1131.

Issue 1:

testEventBasedSplit3

Error: java.lang.NullPointerException: Cannot invoke "org.drools.core.common.InternalAgenda.isRuleActiveInRuleFlowGroup(String, String, String)" because the return value of "org.drools.core.common.InternalKnowledgeRuntime.getAgenda()" is null
at org.jbpm.bpmn2.IntermediateEventTest.testEventBasedSplit3(IntermediateEventTest.java:479)

org.opentest4j.AssertionFailedError:
expected: 1
but was: 5

In the v7, we use:

kruntime.getKieSession().insert(jack);

But there appears to be an issue because the jack variable doesn't exist in our data model. Could you please guide on how we should handle this.

Issue 2:

testEventBasedSplit5

The receive task handler needs a kruntime object. Need guidance on how to create this handler in v9 as we don't have the kruntime object.

ReceiveTaskHandler receiveTaskHandler = new ReceiveTaskHandler(null); ProcessTestHelper.registerHandler(app, "Receive Task", receiveTaskHandler);

@Abhitocode Abhitocode changed the title [incubator-kie-issues-1131] test migration from V7 to code generation-20 [incubator-kie-issues-1131] test migration from V7 to code generation-23 Aug 23, 2024
@elguardian
Copy link
Contributor

we are not supporting this sort of scenario with drools. set this as standby. I need to fix this in the engine

@Abhitocode
Copy link
Contributor Author

Okay

@Abhitocode Abhitocode closed this Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate process test migration from v7 legacy runtime to code generation
2 participants