Copilot commented on code in PR #6324:
URL:
https://github.com/apache/incubator-kie-drools/pull/6324#discussion_r2062952523
##########
drools-model/drools-model-codegen/src/test/java/org/drools/model/codegen/execmodel/generator/DrlxParseUtilTest.java:
##########
@@ -141,6 +133,24 @@ public void removeRootNodeTest() {
assertThat(findRemoveRootNodeViaScope(expr("$data.getIndexes().getValues().get(0)"))).isEqualTo(new
RemoveRootNodeResult(of(expr("$data")),
expr("getIndexes().getValues().get(0)"), expr("getIndexes()")));
}
+ @Test
+ public void
test_transformDrlNameExprToNameExpr_maintainingMethodArgumentOrder(){
+ Expression e =
expr("UtilsClass.utilityMethodWithLotsOfArguments($arg1, $arg2.field1,
$arg3.field2, $arg4.field3, $arg5.field4))");
Review Comment:
The test expression contains an extra closing parenthesis which could lead
to unexpected parsing errors. Consider removing the extra closing parenthesis
to ensure accurate evaluation of the method call.
```suggestion
Expression e =
expr("UtilsClass.utilityMethodWithLotsOfArguments($arg1, $arg2.field1,
$arg3.field2, $arg4.field3, $arg5.field4)");
```
--
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]
