mariofusco commented on code in PR #6103:
URL: 
https://github.com/apache/incubator-kie-drools/pull/6103#discussion_r1775242314


##########
drools-quarkus-extension/drools-quarkus-integration-test/src/test/java/org/drools/quarkus/test/RuntimeTest.java:
##########
@@ -42,30 +47,59 @@ public class RuntimeTest {
     @Test
     public void testDrlEvaluation() {
         // canDrinkKS is the default session
-        testSimpleDrl(runtimeBuilder.newKieSession(), "org.drools.drl");
+        testSimpleDrl(runtimeBuilder.newKieSession(), "org.drools.drl", true);
     }
 
     @Test
     public void testDTableEvaluation() {
-        testSimpleDrl(runtimeBuilder.newKieSession("canDrinkKSDTable"), 
"org.drools.dtable");
+        testSimpleDrl(runtimeBuilder.newKieSession("canDrinkKSDTable"), 
"org.drools.dtable", true);
     }
 
     @Test
     public void testYamlEvaluation() {
-        testSimpleDrl(runtimeBuilder.newKieSession("canDrinkKSYaml"), 
"org.drools.yaml");
+        testSimpleDrl(runtimeBuilder.newKieSession("canDrinkKSYaml"), 
"org.drools.yaml", true);
     }
 
-    private void testSimpleDrl(KieSession ksession, String assetPackage) {
-        List<String> pkgNames = 
ksession.getKieBase().getKiePackages().stream().map(KiePackage::getName).collect(Collectors.toList());
+    @Test
+    public void testStatelessDrlEvaluation() {
+        // statelessCanDrinkKS is the default stateless session
+        testSimpleDrl(runtimeBuilder.newStatelessKieSession(), 
"org.drools.drl", false);
+    }
+
+    @Test
+    public void testStatelessDTableEvaluation() {
+        
testSimpleDrl(runtimeBuilder.newStatelessKieSession("statelessCanDrinkKSDTable"),
 "org.drools.dtable", false);
+    }
+
+    @Test
+    public void testStatelessYamlEvaluation() {
+        
testSimpleDrl(runtimeBuilder.newStatelessKieSession("statelessCanDrinkKSYaml"), 
"org.drools.yaml", false);
+    }
+
+    private void testSimpleDrl(RuntimeSession session, String assetPackage, 
boolean stateful) {

Review Comment:
   Here I'm using the new `RuntimeSession` interface to test both `KieSession` 
and `StatelessKieSession` with the same method.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to