[incubator-kie-drools#5709] [new-parser] Some rules do not fire in Mu…

…ltiKieBaseTest (apache#5794)
tkobayas · Oct 11, 2024 · f8ee445 · f8ee445
1 parent 77ff75c
commit f8ee445
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/...drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/MiscDRLParserTest.java b/...drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/MiscDRLParserTest.java
@@ -780,6 +780,9 @@ void parse_simpleRuleWithBindings() throws Exception {
 
         assertThat(rule.getName()).isEqualTo("simple_rule");
 
+        assertThat(rule.getStartCharacter()).isEqualTo(803);
+        assertThat(rule.getEndCharacter()).isEqualTo(996);
+
         assertThat(rule.getConsequenceLine()).isEqualTo(22);
         assertThat(rule.getConsequencePattern()).isEqualTo(2);
 

diff --git a/drools-drl/drools-drl-parser/src/main/java/org/drools/drl/parser/antlr4/DRLVisitorImpl.java b/drools-drl/drools-drl-parser/src/main/java/org/drools/drl/parser/antlr4/DRLVisitorImpl.java
@@ -270,6 +270,7 @@ public RuleDescr visitRuledef(DRLParser.RuledefContext ctx) {
             ruleDescr.setConsequence(trimThen(getTokenTextPreservingWhitespace(ctx.rhs(), tokenStream))); // RHS is just a text
         }
 
+        populateStartEnd(ruleDescr, ctx);
         return ruleDescr;
     }
-Original file line number
+Diff line change
@@ Expand Up @@
                 ruleDescr.setConsequence(trimThen(getTokenTextPreservingWhitespace(ctx.rhs(), tokenStream))); // RHS is just a text
             }
+            populateStartEnd(ruleDescr, ctx);
             return ruleDescr;
         }
@@ Expand Down @@