From 0b1f30e8bf7a776365f34eed855f70a64165d4e3 Mon Sep 17 00:00:00 2001 From: Toshiya Kobayashi Date: Mon, 10 Jun 2024 14:37:09 +0900 Subject: [PATCH] - code format --- .../antlr4/DescrCommonPropertyTest.java | 9 +- .../drl/parser/antlr4/DescrDumperTest.java | 131 +- .../drl/parser/antlr4/MiscDRLParserTest.java | 1127 ++++++++--------- 3 files changed, 603 insertions(+), 664 deletions(-) diff --git a/drools-drl/drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/DescrCommonPropertyTest.java b/drools-drl/drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/DescrCommonPropertyTest.java index ebfcc4e7e14..2e1a0e8564e 100644 --- a/drools-drl/drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/DescrCommonPropertyTest.java +++ b/drools-drl/drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/DescrCommonPropertyTest.java @@ -332,7 +332,7 @@ void accumulateDescr() { @Test void behaviorDescr() { - final String source = "rule X when StockTick( symbol==\"ACME\") over window:length(10) then end";; + final String source = "rule X when StockTick( symbol==\"ACME\") over window:length(10) then end"; final PackageDescr pkg = parseAndGetPackageDescr(source); final RuleDescr rule = pkg.getRules().get(0); PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); @@ -349,11 +349,12 @@ void fromDescr() { " Adult( $children : children)\n" + " Child() from $children\n" + " then\n" + - "end";; + "end"; + ; final PackageDescr pkg = parseAndGetPackageDescr(source); final RuleDescr rule = pkg.getRules().get(0); PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(1); - FromDescr from = (FromDescr) pattern.getSource(); + FromDescr from = (FromDescr) pattern.getSource(); // Backward Compatibility Notes: // The old DRL6Parser doesn't populate common properties of FromDescr (seem to be wrong). @@ -376,7 +377,7 @@ void collectDescr() { final PackageDescr pkg = parseAndGetPackageDescr(source); final RuleDescr rule = pkg.getRules().get(0); PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); - CollectDescr collect = (CollectDescr) pattern.getSource(); + CollectDescr collect = (CollectDescr) pattern.getSource(); assertProperties(collect, 35, 64, 3, 21, 3, 49); } diff --git a/drools-drl/drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/DescrDumperTest.java b/drools-drl/drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/DescrDumperTest.java index 26ac3dceef3..76f750ff215 100644 --- a/drools-drl/drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/DescrDumperTest.java +++ b/drools-drl/drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/DescrDumperTest.java @@ -18,8 +18,6 @@ */ package org.drools.drl.parser.antlr4; -import static org.assertj.core.api.Assertions.assertThat; - import org.drools.compiler.lang.DescrDumper; import org.drools.compiler.lang.DumperContext; import org.drools.drl.ast.descr.AtomicExprDescr; @@ -34,6 +32,8 @@ import org.junit.jupiter.api.Test; import org.kie.internal.builder.conf.LanguageLevelOption; +import static org.assertj.core.api.Assertions.assertThat; + public class DescrDumperTest { private DescrDumper dumper; @@ -52,8 +52,8 @@ void dump() { String input = "price > 10 && < 20 || == $val || == 30"; String expected = "( price > 10 && price < 20 || price == $val || price == 30 )"; - ConstraintConnectiveDescr descr = parse( input ); - String result = dumper.dump( descr ); + ConstraintConnectiveDescr descr = parse(input); + String result = dumper.dump(descr); assertThat(result).isEqualTo(expected); } @@ -63,8 +63,8 @@ void dumpMatches() { String input = "type.toString matches \"something\\swith\\tsingle escapes\""; String expected = "type.toString ~= \"something\\swith\\tsingle escapes\""; - ConstraintConnectiveDescr descr = parse( input ); - String result = dumper.dump( descr ); + ConstraintConnectiveDescr descr = parse(input); + String result = dumper.dump(descr); assertThat(result).isEqualTo(expected); } @@ -74,8 +74,8 @@ void dumpMatches2() { String input = "type.toString matches 'something\\swith\\tsingle escapes'"; String expected = "type.toString ~= \"something\\swith\\tsingle escapes\""; - ConstraintConnectiveDescr descr = parse( input ); - String result = dumper.dump( descr ); + ConstraintConnectiveDescr descr = parse(input); + String result = dumper.dump(descr); assertThat(result).isEqualTo(expected); } @@ -85,8 +85,8 @@ void dumpMatches3() { String input = "this[\"content\"] matches \"hello ;=\""; String expected = "this[\"content\"] ~= \"hello ;=\""; - ConstraintConnectiveDescr descr = parse( input ); - String result = dumper.dump( descr ); + ConstraintConnectiveDescr descr = parse(input); + String result = dumper.dump(descr); assertThat(result).isEqualTo(expected); } @@ -96,8 +96,8 @@ void dumpContains() { String input = "list contains \"b\""; String expected = "list contains \"b\""; - ConstraintConnectiveDescr descr = parse( input ); - String result = dumper.dump( descr ); + ConstraintConnectiveDescr descr = parse(input); + String result = dumper.dump(descr); assertThat(result).isEqualTo(expected); } @@ -107,8 +107,8 @@ void dumpContains2() { String input = "list not contains \"b\""; String expected = "!( list contains \"b\" )"; - ConstraintConnectiveDescr descr = parse( input ); - String result = dumper.dump( descr ); + ConstraintConnectiveDescr descr = parse(input); + String result = dumper.dump(descr); assertThat(result).isEqualTo(expected); } @@ -118,8 +118,8 @@ void dumpExcludes() { String input = "list excludes \"b\""; String expected = "!( list contains \"b\" )"; - ConstraintConnectiveDescr descr = parse( input ); - String result = dumper.dump( descr ); + ConstraintConnectiveDescr descr = parse(input); + String result = dumper.dump(descr); assertThat(result).isEqualTo(expected); } @@ -129,8 +129,8 @@ void dumpExcludes2() { String input = "list not excludes \"b\""; String expected = "list contains \"b\""; - ConstraintConnectiveDescr descr = parse( input ); - String result = dumper.dump( descr ); + ConstraintConnectiveDescr descr = parse(input); + String result = dumper.dump(descr); assertThat(result).isEqualTo(expected); } @@ -141,8 +141,8 @@ void dumpWithDateAttr() { String input = "son.birthDate == \"01-jan-2000\""; String expected = "son.birthDate == org.drools.util.DateUtils.parseDate( \"01-jan-2000\" )"; - ConstraintConnectiveDescr descr = parse( input ); - String result = dumper.dump( descr ); + ConstraintConnectiveDescr descr = parse(input); + String result = dumper.dump(descr); assertThat(result).isEqualTo(expected); } @@ -152,8 +152,8 @@ void dumpComplex() { String input = "a ( > 60 && < 70 ) || ( > 50 && < 55 ) && a3 == \"black\" || a == 40 && a3 == \"pink\" || a == 12 && a3 == \"yellow\" || a3 == \"blue\""; String expected = "( ( a > 60 && a < 70 || a > 50 && a < 55 ) && a3 == \"black\" || a == 40 && a3 == \"pink\" || a == 12 && a3 == \"yellow\" || a3 == \"blue\" )"; - ConstraintConnectiveDescr descr = parse( input ); - String result = dumper.dump( descr ); + ConstraintConnectiveDescr descr = parse(input); + String result = dumper.dump(descr); assertThat(result).isEqualTo(expected); } @@ -163,14 +163,14 @@ void dumpBindings() { String input = "$x : property > value"; String expected = "property > value"; - ConstraintConnectiveDescr descr = parse( input ); + ConstraintConnectiveDescr descr = parse(input); DumperContext ctx = new DumperContext(); - String result = dumper.dump( descr, - ctx ); + String result = dumper.dump(descr, + ctx); assertThat(result).isEqualTo(expected); assertThat(ctx.getBindings()).hasSize(1); - BindingDescr bind = ctx.getBindings().get( 0 ); + BindingDescr bind = ctx.getBindings().get(0); assertThat(bind.getVariable()).isEqualTo("$x"); assertThat(bind.getExpression()).isEqualTo("property"); } @@ -180,20 +180,20 @@ void dumpBindings2() { String input = "( $a : a > $b : b[10].prop || 10 != 20 ) && $x : someMethod(10) == 20"; String expected = "( a > b[10].prop || 10 != 20 ) && someMethod(10) == 20"; - ConstraintConnectiveDescr descr = parse( input ); + ConstraintConnectiveDescr descr = parse(input); DumperContext ctx = new DumperContext(); - String result = dumper.dump( descr, - ctx ); + String result = dumper.dump(descr, + ctx); assertThat(result).isEqualTo(expected); assertThat(ctx.getBindings()).hasSize(3); - BindingDescr bind = ctx.getBindings().get( 0 ); + BindingDescr bind = ctx.getBindings().get(0); assertThat(bind.getVariable()).isEqualTo("$a"); assertThat(bind.getExpression()).isEqualTo("a"); - bind = ctx.getBindings().get( 1 ); + bind = ctx.getBindings().get(1); assertThat(bind.getVariable()).isEqualTo("$b"); assertThat(bind.getExpression()).isEqualTo("b[10].prop"); - bind = ctx.getBindings().get( 2 ); + bind = ctx.getBindings().get(2); assertThat(bind.getVariable()).isEqualTo("$x"); assertThat(bind.getExpression()).isEqualTo("someMethod(10)"); } @@ -203,8 +203,8 @@ void dumpBindings3() { String input = "( $a : a > $b : b[10].prop || 10 != 20 ) && $x : someMethod(10)"; String expected = "( a > b[10].prop || 10 != 20 )"; - ConstraintConnectiveDescr descr = parse( input ); - String result = dumper.dump( descr ); + ConstraintConnectiveDescr descr = parse(input); + String result = dumper.dump(descr); assertThat(result).isEqualTo(expected); } @@ -214,8 +214,8 @@ void dumpBindings4() { String input = "( $a : a > $b : b[10].prop || $x : someMethod(10) ) && 10 != 20"; String expected = "( a > b[10].prop ) && 10 != 20"; - ConstraintConnectiveDescr descr = parse( input ); - String result = dumper.dump( descr ); + ConstraintConnectiveDescr descr = parse(input); + String result = dumper.dump(descr); assertThat(result).isEqualTo(expected); } @@ -225,14 +225,14 @@ void dumpBindingsWithRestriction() { String input = "$x : age > 10 && < 20 || > 30"; String expected = "( age > 10 && age < 20 || age > 30 )"; - ConstraintConnectiveDescr descr = parse( input ); + ConstraintConnectiveDescr descr = parse(input); DumperContext ctx = new DumperContext(); - String result = dumper.dump( descr, - ctx ); + String result = dumper.dump(descr, + ctx); assertThat(result).isEqualTo(expected); assertThat(ctx.getBindings()).hasSize(1); - BindingDescr bind = ctx.getBindings().get( 0 ); + BindingDescr bind = ctx.getBindings().get(0); assertThat(bind.getVariable()).isEqualTo("$x"); assertThat(bind.getExpression()).isEqualTo("age"); } @@ -242,14 +242,14 @@ void dumpBindingsComplexOp() { String input = "$x : age in (10, 20, $someVal)"; String expected = "( age == 10 || age == 20 || age == $someVal )"; - ConstraintConnectiveDescr descr = parse( input ); + ConstraintConnectiveDescr descr = parse(input); DumperContext ctx = new DumperContext(); - String result = dumper.dump( descr, - ctx ); + String result = dumper.dump(descr, + ctx); assertThat(result).isEqualTo(expected); assertThat(ctx.getBindings()).hasSize(1); - BindingDescr bind = ctx.getBindings().get( 0 ); + BindingDescr bind = ctx.getBindings().get(0); assertThat(bind.getVariable()).isEqualTo("$x"); assertThat(bind.getExpression()).isEqualTo("age"); } @@ -259,14 +259,14 @@ void dumpBindingsComplexOp2() { String input = "$x : age not in (10, 20, $someVal)"; String expected = "age != 10 && age != 20 && age != $someVal"; - ConstraintConnectiveDescr descr = parse( input ); + ConstraintConnectiveDescr descr = parse(input); DumperContext ctx = new DumperContext(); - String result = dumper.dump( descr, - ctx ); + String result = dumper.dump(descr, + ctx); assertThat(result).isEqualTo(expected); assertThat(ctx.getBindings()).hasSize(1); - BindingDescr bind = ctx.getBindings().get( 0 ); + BindingDescr bind = ctx.getBindings().get(0); assertThat(bind.getVariable()).isEqualTo("$x"); assertThat(bind.getExpression()).isEqualTo("age"); } @@ -277,9 +277,9 @@ void processInlineCast() { String expectedInstanceof = "field1 instanceof Class"; String expectedcasted = "((Class)field1).field2"; AtomicExprDescr atomicExpr = new AtomicExprDescr(expr); - ConstraintConnectiveDescr ccd = new ConstraintConnectiveDescr( ); - ccd.addDescr( atomicExpr ); - String[] instanceofAndCastedExpr = dumper.processImplicitConstraints(expr, atomicExpr, ccd, ccd.getDescrs().indexOf( atomicExpr ), null); + ConstraintConnectiveDescr ccd = new ConstraintConnectiveDescr(); + ccd.addDescr(atomicExpr); + String[] instanceofAndCastedExpr = dumper.processImplicitConstraints(expr, atomicExpr, ccd, ccd.getDescrs().indexOf(atomicExpr), null); assertThat(ccd.getDescrs()).hasSize(2); assertThat(ccd.getDescrs().get(0).toString()).isEqualTo(expectedInstanceof); assertThat(atomicExpr.getRewrittenExpression()).isEqualTo(expectedcasted); @@ -289,8 +289,8 @@ void processInlineCast() { String expectedInstanceof2 = "((Class1)field1).field2 instanceof Class2"; expectedcasted = "((Class2)((Class1)field1).field2).field3"; atomicExpr = new AtomicExprDescr(expr); - ccd = new ConstraintConnectiveDescr( ); - instanceofAndCastedExpr = dumper.processImplicitConstraints(expr, atomicExpr, ccd, ccd.getDescrs().indexOf( atomicExpr ), null); + ccd = new ConstraintConnectiveDescr(); + instanceofAndCastedExpr = dumper.processImplicitConstraints(expr, atomicExpr, ccd, ccd.getDescrs().indexOf(atomicExpr), null); assertThat(ccd.getDescrs().get(0).toString()).isEqualTo(expectedInstanceof1); assertThat(ccd.getDescrs().get(1).toString()).isEqualTo(expectedInstanceof2); assertThat(instanceofAndCastedExpr[1]).isEqualTo(expectedcasted); @@ -303,8 +303,8 @@ void processNullSafeDereferencing() { String expectedNullCheck = "field1 != null"; String expectedExpr = "field1.field2"; AtomicExprDescr atomicExpr = new AtomicExprDescr(expr); - ConstraintConnectiveDescr ccd = new ConstraintConnectiveDescr( ); - String[] nullCheckAndExpr = dumper.processImplicitConstraints( expr, atomicExpr, ccd, ccd.getDescrs().indexOf( atomicExpr ), null ); + ConstraintConnectiveDescr ccd = new ConstraintConnectiveDescr(); + String[] nullCheckAndExpr = dumper.processImplicitConstraints(expr, atomicExpr, ccd, ccd.getDescrs().indexOf(atomicExpr), null); assertThat(ccd.getDescrs().get(0).toString()).isEqualTo(expectedNullCheck); assertThat(nullCheckAndExpr[1]).isEqualTo(expectedExpr); assertThat(atomicExpr.getRewrittenExpression()).isEqualTo(expectedExpr); @@ -314,8 +314,8 @@ void processNullSafeDereferencing() { String expectedNullCheck2 = "field1.field2 != null"; expectedExpr = "field1.field2.field3"; atomicExpr = new AtomicExprDescr(expr); - ccd = new ConstraintConnectiveDescr( ); - nullCheckAndExpr = dumper.processImplicitConstraints( expr, atomicExpr, ccd, ccd.getDescrs().indexOf( atomicExpr ), null ); + ccd = new ConstraintConnectiveDescr(); + nullCheckAndExpr = dumper.processImplicitConstraints(expr, atomicExpr, ccd, ccd.getDescrs().indexOf(atomicExpr), null); assertThat(ccd.getDescrs().get(0).toString()).isEqualTo(expectedNullCheck1); assertThat(ccd.getDescrs().get(1).toString()).isEqualTo(expectedNullCheck2); assertThat(nullCheckAndExpr[1]).isEqualTo(expectedExpr); @@ -328,8 +328,8 @@ void processImplicitConstraints() { String expectedConstraints = "field1 instanceof Class"; String expectedExpr = "((Class)field1).field2"; AtomicExprDescr atomicExpr = new AtomicExprDescr(expr); - ConstraintConnectiveDescr ccd = new ConstraintConnectiveDescr( ); - String[] constraintsAndExpr = dumper.processImplicitConstraints( expr, atomicExpr, ccd, ccd.getDescrs().indexOf( atomicExpr ), null ); + ConstraintConnectiveDescr ccd = new ConstraintConnectiveDescr(); + String[] constraintsAndExpr = dumper.processImplicitConstraints(expr, atomicExpr, ccd, ccd.getDescrs().indexOf(atomicExpr), null); assertThat(ccd.getDescrs().get(0).toString()).isEqualTo(expectedConstraints); assertThat(constraintsAndExpr[1]).isEqualTo(expectedExpr); assertThat(atomicExpr.getRewrittenExpression()).isEqualTo(expectedExpr); @@ -339,8 +339,8 @@ void processImplicitConstraints() { String expectedConstraints2 = "field1.field2 instanceof Class"; expectedExpr = "((Class)field1.field2).field3"; atomicExpr = new AtomicExprDescr(expr); - ccd = new ConstraintConnectiveDescr( ); - constraintsAndExpr = dumper.processImplicitConstraints( expr, atomicExpr, ccd, ccd.getDescrs().indexOf( atomicExpr ), null ); + ccd = new ConstraintConnectiveDescr(); + constraintsAndExpr = dumper.processImplicitConstraints(expr, atomicExpr, ccd, ccd.getDescrs().indexOf(atomicExpr), null); assertThat(ccd.getDescrs().get(0).toString()).isEqualTo(expectedConstraints1); assertThat(ccd.getDescrs().get(1).toString()).isEqualTo(expectedConstraints2); assertThat(constraintsAndExpr[1]).isEqualTo(expectedExpr); @@ -351,20 +351,19 @@ void processImplicitConstraints() { expectedConstraints2 = "((Class)field1).field2 != null"; expectedExpr = "((Class)field1).field2.field3"; atomicExpr = new AtomicExprDescr(expr); - ccd = new ConstraintConnectiveDescr( ); - constraintsAndExpr = dumper.processImplicitConstraints( expr, atomicExpr, ccd, ccd.getDescrs().indexOf( atomicExpr ), null ); + ccd = new ConstraintConnectiveDescr(); + constraintsAndExpr = dumper.processImplicitConstraints(expr, atomicExpr, ccd, ccd.getDescrs().indexOf(atomicExpr), null); assertThat(ccd.getDescrs().get(0).toString()).isEqualTo(expectedConstraints1); assertThat(ccd.getDescrs().get(1).toString()).isEqualTo(expectedConstraints2); assertThat(constraintsAndExpr[1]).isEqualTo(expectedExpr); assertThat(atomicExpr.getRewrittenExpression()).isEqualTo(expectedExpr); } - public ConstraintConnectiveDescr parse( final String constraint ) { + public ConstraintConnectiveDescr parse(final String constraint) { DrlExprParser parser = DrlExprParserFactory.getDrlExprParser(LanguageLevelOption.DRL6); - ConstraintConnectiveDescr result = parser.parse( constraint ); + ConstraintConnectiveDescr result = parser.parse(constraint); assertThat(parser.hasErrors()).as(parser.getErrors().toString()).isFalse(); return result; } - } diff --git a/drools-drl/drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/MiscDRLParserTest.java b/drools-drl/drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/MiscDRLParserTest.java index b321da260f4..3fc2cdbca39 100644 --- a/drools-drl/drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/MiscDRLParserTest.java +++ b/drools-drl/drools-drl-parser-tests/src/test/java/org/drools/drl/parser/antlr4/MiscDRLParserTest.java @@ -116,7 +116,7 @@ private String readResource(final String filename) { private PackageDescr parseAndGetPackageDescr(String drl) { try { - PackageDescr pkg = parser.parse(null, drl); + PackageDescr pkg = parser.parse(null, drl); assertThat(parser.hasErrors()).as(parser.getErrors().toString()).isFalse(); return pkg; } catch (DroolsParserException e) { @@ -210,7 +210,7 @@ void packageWithAllErrorNode() { @Test void importDef() { final String source = "package foo; import com.foo.Bar; import com.foo.Baz;"; - PackageDescr pkg =parseAndGetPackageDescr(source); + PackageDescr pkg = parseAndGetPackageDescr(source); assertThat(pkg.getName()).isEqualTo("foo"); assertThat(pkg.getImports()).hasSize(2); ImportDescr impdescr = pkg.getImports().get(0); @@ -629,8 +629,6 @@ void ruleFlowGroup() { String source = readResource("ruleflowgroup.drl"); PackageDescr pkg = parseAndGetPackageDescr(source); - - RuleDescr rule = pkg.getRules().get(0); assertThat(rule).isNotNull(); @@ -648,8 +646,6 @@ void consequenceWithDeclaration() { // Note : Removed "i\i;" from the original declaration-in-consequence.drl under drools-test-coverage // because it's not a valid java expression and doesn't make sense to test. (Right to raise a parser error) - - RuleDescr rule = pkg.getRules().get(0); assertThat(rule).isNotNull(); @@ -677,8 +673,6 @@ void or() { PackageDescr pkg = parseAndGetPackageDescr(text); RuleDescr rule = pkg.getRules().get(0); - - assertThat(rule).isNotNull(); AndDescr lhs = rule.getLhs(); @@ -692,8 +686,6 @@ void lhsWithStringQuotes() { PackageDescr pkg = parseAndGetPackageDescr(text); RuleDescr rule = pkg.getRules().get(0); - - assertThat(rule).isNotNull(); AndDescr lhs = rule.getLhs(); @@ -708,7 +700,6 @@ void lhsWithStringQuotesEscapeChars() { PackageDescr pkg = parseAndGetPackageDescr(text); RuleDescr rule = pkg.getRules().get(0); - assertThat(rule).isNotNull(); AndDescr lhs = rule.getLhs(); @@ -759,8 +750,6 @@ void emptyPattern() { String source = readResource("test_EmptyPattern.drl"); PackageDescr pkg = parseAndGetPackageDescr(source); - - assertThat(pkg.getRules()).hasSize(1); final RuleDescr ruleDescr = pkg.getRules().get(0); assertThat(ruleDescr.getName()).isEqualTo("simple rule"); @@ -794,7 +783,7 @@ void simpleFunctionCallWithFrom() { @Test void simpleAccessorWithFrom() { RuleDescr rule = parseAndGetFirstRuleDescrFromFile("test_SimpleAccessorWithFrom.drl"); - final PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); final FromDescr from = (FromDescr) pattern.getSource(); final MVELExprDescr accessor = (MVELExprDescr) from.getDataSource(); @@ -804,7 +793,7 @@ void simpleAccessorWithFrom() { @Test void simpleAccessorAndArgWithFrom() { RuleDescr rule = parseAndGetFirstRuleDescrFromFile("test_SimpleAccessorArgWithFrom.drl"); - final PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); final FromDescr from = (FromDescr) pattern.getSource(); final MVELExprDescr accessor = (MVELExprDescr) from.getDataSource(); @@ -815,7 +804,7 @@ void simpleAccessorAndArgWithFrom() { void complexChainedAccessor() { RuleDescr rule = parseAndGetFirstRuleDescrFromFile("test_ComplexChainedCallWithFrom.drl"); - final PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); final FromDescr from = (FromDescr) pattern.getSource(); final MVELExprDescr accessor = (MVELExprDescr) from.getDataSource(); @@ -895,41 +884,36 @@ void multipleRestrictionsConstraint() { // The first pattern, with 2 restrictions on a single field (plus a // connective) - PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("Person"); - assertThat( pattern.getConstraint().getDescrs()).hasSize(1); + assertThat(pattern.getConstraint().getDescrs()).hasSize(1); AndDescr and = (AndDescr) pattern.getConstraint(); - ExprConstraintDescr fld = (ExprConstraintDescr) and.getDescrs().get( 0 ); + ExprConstraintDescr fld = (ExprConstraintDescr) and.getDescrs().get(0); assertThat(fld.getExpression()).isEqualTo("age > 30 && < 40"); // the second col, with 2 fields, the first with 2 restrictions, the // second field with one - pattern = (PatternDescr) rule.getLhs().getDescrs().get( 1 ); + pattern = (PatternDescr) rule.getLhs().getDescrs().get(1); assertThat(pattern.getObjectType()).isEqualTo("Vehicle"); assertThat(pattern.getConstraint().getDescrs()).hasSize(2); and = (AndDescr) pattern.getConstraint(); - fld = (ExprConstraintDescr) and.getDescrs().get( 0 ); - assertThat(fld.getExpression()).isEqualToIgnoringWhitespace( "type == \"sedan\" || == \"wagon\""); + fld = (ExprConstraintDescr) and.getDescrs().get(0); + assertThat(fld.getExpression()).isEqualToIgnoringWhitespace("type == \"sedan\" || == \"wagon\""); // now the second field - fld = (ExprConstraintDescr) and.getDescrs().get( 1 ); - assertThat(fld.getExpression()).isEqualToIgnoringWhitespace( "age < 3"); + fld = (ExprConstraintDescr) and.getDescrs().get(1); + assertThat(fld.getExpression()).isEqualToIgnoringWhitespace("age < 3"); } -//------------------------------------------------------------------------- -// DROOLS-7271 : ported from RuleParserTest -// Failing tests are annotated with @Disabled. We can fix issues one by one -//------------------------------------------------------------------------- - @Test void lineNumberInAST() { // also see testSimpleExpander to see how this works with an expander // (should be the same). final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "simple_rule.drl" ); + "simple_rule.drl"); assertThat(rule).isNotNull(); @@ -945,17 +929,17 @@ void lineNumberInAST() { assertThat(lhs.getDescrs()).hasSize(3); // Check first pattern - final PatternDescr first = (PatternDescr) lhs.getDescrs().get( 0 ); + final PatternDescr first = (PatternDescr) lhs.getDescrs().get(0); assertThat(first.getIdentifier()).isEqualTo("foo3"); assertThat(first.getObjectType()).isEqualTo("Bar"); assertThat(first.getConstraint().getDescrs()).hasSize(1); // Check second pattern - final PatternDescr second = (PatternDescr) lhs.getDescrs().get( 1 ); + final PatternDescr second = (PatternDescr) lhs.getDescrs().get(1); assertThat(second.getIdentifier()).isEqualTo("foo4"); assertThat(second.getObjectType()).isEqualTo("Bar"); - final PatternDescr third = (PatternDescr) lhs.getDescrs().get( 2 ); + final PatternDescr third = (PatternDescr) lhs.getDescrs().get(2); assertThat(third.getObjectType()).isEqualTo("Baz"); assertThat(first.getLine()).isEqualTo(19); @@ -966,11 +950,9 @@ void lineNumberInAST() { @Test void lineNumberIncludingCommentsInRHS() { PackageDescr pkg = parseAndGetPackageDescrFromFile( - "test_CommentLineNumbersInConsequence.drl" ); - - + "test_CommentLineNumbersInConsequence.drl"); - final String rhs = (String) ((RuleDescr) pkg.getRules().get( 0 )).getConsequence(); + final String rhs = (String) ((RuleDescr) pkg.getRules().get(0)).getConsequence(); String expected = "\\s*//woot$\\s*first;$\\s*$\\s*//$\\s*$\\s*/\\* lala$\\s*$\\s*\\*/$\\s*second;$\\s*"; assertThat(Pattern.compile(expected, Pattern.DOTALL | Pattern.MULTILINE).matcher(rhs).matches()).isTrue(); @@ -979,9 +961,7 @@ void lineNumberIncludingCommentsInRHS() { @Test void lhsSemicolonDelim() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "lhs_semicolon_delim.drl" ); - - + "lhs_semicolon_delim.drl"); assertThat(rule).isNotNull(); @@ -996,7 +976,7 @@ void lhsSemicolonDelim() { // System.err.println( lhs.getDescrs() ); // Check first pattern - final PatternDescr first = (PatternDescr) lhs.getDescrs().get( 0 ); + final PatternDescr first = (PatternDescr) lhs.getDescrs().get(0); assertThat(first.getIdentifier()).isEqualTo("foo3"); assertThat(first.getObjectType()).isEqualTo("Bar"); @@ -1004,23 +984,23 @@ void lhsSemicolonDelim() { // LiteralDescr constraint = (LiteralDescr) first.getDescrs().get( 0 ); AndDescr and = (AndDescr) first.getConstraint(); - ExprConstraintDescr fld = (ExprConstraintDescr) and.getDescrs().get( 0 ); + ExprConstraintDescr fld = (ExprConstraintDescr) and.getDescrs().get(0); assertThat(fld).isNotNull(); assertThat(fld.getExpression()).isEqualTo("a==3"); // Check second pattern - final PatternDescr second = (PatternDescr) lhs.getDescrs().get( 1 ); + final PatternDescr second = (PatternDescr) lhs.getDescrs().get(1); assertThat(second.getIdentifier()).isEqualTo("foo4"); assertThat(second.getObjectType()).isEqualTo("Bar"); assertThat(second.getDescrs()).hasSize(1); - final ExprConstraintDescr fieldBindingDescr = (ExprConstraintDescr) second.getDescrs().get( 0 ); + final ExprConstraintDescr fieldBindingDescr = (ExprConstraintDescr) second.getDescrs().get(0); assertThat(fieldBindingDescr.getExpression()).isEqualTo("a4:a==4"); // Check third pattern - final PatternDescr third = (PatternDescr) lhs.getDescrs().get( 2 ); + final PatternDescr third = (PatternDescr) lhs.getDescrs().get(2); assertThat(third.getIdentifier()).isNull(); assertThat(third.getObjectType()).isEqualTo("Baz"); @@ -1030,24 +1010,22 @@ void lhsSemicolonDelim() { @Test void notNode() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "rule_not.drl" ); - - + "rule_not.drl"); assertThat(rule).isNotNull(); assertThat(rule.getName()).isEqualTo("simple_rule"); final AndDescr lhs = rule.getLhs(); assertThat(lhs.getDescrs()).hasSize(1); - final NotDescr not = (NotDescr) lhs.getDescrs().get( 0 ); + final NotDescr not = (NotDescr) lhs.getDescrs().get(0); assertThat(not.getDescrs()).hasSize(1); - final PatternDescr pattern = (PatternDescr) not.getDescrs().get( 0 ); + final PatternDescr pattern = (PatternDescr) not.getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("Cheese"); assertThat(pattern.getConstraint().getDescrs()).hasSize(1); final AndDescr and = (AndDescr) pattern.getConstraint(); - final ExprConstraintDescr fld = (ExprConstraintDescr) and.getDescrs().get( 0 ); + final ExprConstraintDescr fld = (ExprConstraintDescr) and.getDescrs().get(0); assertThat(fld.getExpression()).isEqualTo("type == \"stilton\""); } @@ -1056,33 +1034,31 @@ void notNode() { void notExistWithBrackets() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "not_exist_with_brackets.drl" ); - - + "not_exist_with_brackets.drl"); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule).isNotNull(); assertThat(rule.getName()).isEqualTo("simple_rule"); final AndDescr lhs = rule.getLhs(); assertThat(lhs.getDescrs()).hasSize(2); - final NotDescr not = (NotDescr) lhs.getDescrs().get( 0 ); + final NotDescr not = (NotDescr) lhs.getDescrs().get(0); assertThat(not.getDescrs()).hasSize(1); - final PatternDescr pattern = (PatternDescr) not.getDescrs().get( 0 ); + final PatternDescr pattern = (PatternDescr) not.getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("Cheese"); - final ExistsDescr ex = (ExistsDescr) lhs.getDescrs().get(1 ); + final ExistsDescr ex = (ExistsDescr) lhs.getDescrs().get(1); assertThat(ex.getDescrs()).hasSize(1); - final PatternDescr exPattern = (PatternDescr) ex.getDescrs().get( 0 ); + final PatternDescr exPattern = (PatternDescr) ex.getDescrs().get(0); assertThat(exPattern.getObjectType()).isEqualTo("Foo"); } @Test void simpleQuery() { final QueryDescr query = parseAndGetFirstQueryDescrFromFile( - "simple_query.drl" ); + "simple_query.drl"); assertThat(query).isNotNull(); @@ -1095,81 +1071,81 @@ void simpleQuery() { assertThat(lhs.getDescrs()).hasSize(3); // Check first pattern - final PatternDescr first = (PatternDescr) lhs.getDescrs().get( 0 ); + final PatternDescr first = (PatternDescr) lhs.getDescrs().get(0); assertThat(first.getIdentifier()).isEqualTo("foo3"); assertThat(first.getObjectType()).isEqualTo("Bar"); assertThat(first.getConstraint().getDescrs()).hasSize(1); AndDescr and = (AndDescr) first.getConstraint(); - ExprConstraintDescr fld = (ExprConstraintDescr) and.getDescrs().get( 0 ); + ExprConstraintDescr fld = (ExprConstraintDescr) and.getDescrs().get(0); assertThat(fld).isNotNull(); assertThat(fld.getExpression()).isEqualTo("a==3"); // Check second pattern - final PatternDescr second = (PatternDescr) lhs.getDescrs().get( 1 ); + final PatternDescr second = (PatternDescr) lhs.getDescrs().get(1); assertThat(second.getIdentifier()).isEqualTo("foo4"); assertThat(second.getObjectType()).isEqualTo("Bar"); assertThat(second.getDescrs()).hasSize(1); // check it has field bindings. - final ExprConstraintDescr bindingDescr = (ExprConstraintDescr) second.getDescrs().get( 0 ); + final ExprConstraintDescr bindingDescr = (ExprConstraintDescr) second.getDescrs().get(0); assertThat(bindingDescr.getExpression()).isEqualTo("a4:a==4"); } @Test void queryRuleMixed() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "query_and_rule.drl" ); + "query_and_rule.drl"); assertThat(pkg.getRules()).hasSize(4); // as queries are rules - RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getName()).isEqualTo("bar"); - QueryDescr query = (QueryDescr) pkg.getRules().get( 1 ); + QueryDescr query = (QueryDescr) pkg.getRules().get(1); assertThat(query.getName()).isEqualTo("simple_query"); - rule = (RuleDescr) pkg.getRules().get( 2 ); + rule = (RuleDescr) pkg.getRules().get(2); assertThat(rule.getName()).isEqualTo("bar2"); - query = (QueryDescr) pkg.getRules().get( 3 ); + query = (QueryDescr) pkg.getRules().get(3); assertThat(query.getName()).isEqualTo("simple_query2"); } @Test void multipleRules() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "multiple_rules.drl" ); + "multiple_rules.drl"); final List rules = pkg.getRules(); assertThat(rules).hasSize(2); - final RuleDescr rule0 = rules.get( 0 ); + final RuleDescr rule0 = rules.get(0); assertThat(rule0.getName()).isEqualTo("Like Stilton"); - final RuleDescr rule1 = rules.get( 1 ); + final RuleDescr rule1 = rules.get(1); assertThat(rule1.getName()).isEqualTo("Like Cheddar"); // checkout the first rule AndDescr lhs = rule1.getLhs(); assertThat(lhs).isNotNull(); assertThat(lhs.getDescrs()).hasSize(1); - assertThat((String) rule0.getConsequence()).isEqualToIgnoringWhitespace( "System.out.println(\"I like \" + t);"); + assertThat((String) rule0.getConsequence()).isEqualToIgnoringWhitespace("System.out.println(\"I like \" + t);"); // Check first pattern - PatternDescr first = (PatternDescr) lhs.getDescrs().get( 0 ); + PatternDescr first = (PatternDescr) lhs.getDescrs().get(0); assertThat(first.getObjectType()).isEqualTo("Cheese"); // checkout the second rule lhs = rule1.getLhs(); assertThat(lhs).isNotNull(); assertThat(lhs.getDescrs()).hasSize(1); - assertThat((String) rule1.getConsequence()).isEqualToIgnoringWhitespace( "System.out.println(\"I like \" + t);"); + assertThat((String) rule1.getConsequence()).isEqualToIgnoringWhitespace("System.out.println(\"I like \" + t);"); // Check first pattern - first = (PatternDescr) lhs.getDescrs().get( 0 ); + first = (PatternDescr) lhs.getDescrs().get(0); assertThat(first.getObjectType()).isEqualTo("Cheese"); } @@ -1227,236 +1203,242 @@ void expanderMultipleConstraintsFlush() { @Test void basicBinding() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "basic_binding.drl" ); + "basic_binding.drl"); - final RuleDescr ruleDescr = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr ruleDescr = (RuleDescr) pkg.getRules().get(0); final AndDescr lhs = ruleDescr.getLhs(); assertThat(lhs.getDescrs()).hasSize(1); - final PatternDescr cheese = (PatternDescr) lhs.getDescrs().get( 0 ); + final PatternDescr cheese = (PatternDescr) lhs.getDescrs().get(0); assertThat(cheese.getObjectType()).isEqualTo("Cheese"); assertThat(cheese.getConstraint().getDescrs()).hasSize(1); - final ExprConstraintDescr fieldBinding = (ExprConstraintDescr) cheese.getDescrs().get( 0 ); + final ExprConstraintDescr fieldBinding = (ExprConstraintDescr) cheese.getDescrs().get(0); assertThat(fieldBinding.getExpression()).isEqualToIgnoringWhitespace("$type:type"); } @Test void boundVariables() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "bindings.drl" ); + "bindings.drl"); - final RuleDescr ruleDescr = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr ruleDescr = (RuleDescr) pkg.getRules().get(0); final AndDescr lhs = ruleDescr.getLhs(); assertThat(lhs.getDescrs()).hasSize(2); - final PatternDescr cheese = (PatternDescr) lhs.getDescrs().get( 0 ); + final PatternDescr cheese = (PatternDescr) lhs.getDescrs().get(0); assertThat(cheese.getObjectType()).isEqualTo("Cheese"); assertThat(cheese.getDescrs()).hasSize(1); - ExprConstraintDescr fieldBinding = (ExprConstraintDescr) cheese.getDescrs().get( 0 ); + ExprConstraintDescr fieldBinding = (ExprConstraintDescr) cheese.getDescrs().get(0); assertThat(fieldBinding.getExpression()).isEqualTo("$type : type == \"stilton\""); - final PatternDescr person = (PatternDescr) lhs.getDescrs().get( 1 ); + final PatternDescr person = (PatternDescr) lhs.getDescrs().get(1); assertThat(person.getDescrs()).hasSize(2); - fieldBinding = (ExprConstraintDescr) person.getDescrs().get( 0 ); + fieldBinding = (ExprConstraintDescr) person.getDescrs().get(0); assertThat(fieldBinding.getExpression()).isEqualTo("$name : name == \"bob\""); - ExprConstraintDescr fld = (ExprConstraintDescr) person.getDescrs().get( 1 ); + ExprConstraintDescr fld = (ExprConstraintDescr) person.getDescrs().get(1); assertThat(fld.getExpression()).isEqualTo("likes == $type"); } @Test void orNesting() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "or_nesting.drl" ); + "or_nesting.drl"); assertThat(pkg).isNotNull(); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getName()).isEqualTo("simple_rule"); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final OrDescr or = (OrDescr) rule.getLhs().getDescrs().get( 0 ); + final OrDescr or = (OrDescr) rule.getLhs().getDescrs().get(0); assertThat(or.getDescrs()).hasSize(2); - final PatternDescr first = (PatternDescr) or.getDescrs().get( 0 ); + final PatternDescr first = (PatternDescr) or.getDescrs().get(0); assertThat(first.getObjectType()).isEqualTo("Person"); - final AndDescr and = (AndDescr) or.getDescrs().get( 1 ); + final AndDescr and = (AndDescr) or.getDescrs().get(1); assertThat(and.getDescrs()).hasSize(2); - final PatternDescr left = (PatternDescr) and.getDescrs().get( 0 ); + final PatternDescr left = (PatternDescr) and.getDescrs().get(0); assertThat(left.getObjectType()).isEqualTo("Person"); - final PatternDescr right = (PatternDescr) and.getDescrs().get( 1 ); + final PatternDescr right = (PatternDescr) and.getDescrs().get(1); assertThat(right.getObjectType()).isEqualTo("Cheese"); } - /** Test that explicit "&&", "||" works as expected */ + /** + * Test that explicit "&&", "||" works as expected + */ @Test void andOrRules() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "and_or_rule.drl" ); + "and_or_rule.drl"); assertThat(pkg).isNotNull(); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getName()).isEqualTo("simple_rule"); // we will have 3 children under the main And node final AndDescr and = rule.getLhs(); assertThat(and.getDescrs()).hasSize(3); - PatternDescr left = (PatternDescr) and.getDescrs().get( 0 ); - PatternDescr right = (PatternDescr) and.getDescrs().get( 1 ); + PatternDescr left = (PatternDescr) and.getDescrs().get(0); + PatternDescr right = (PatternDescr) and.getDescrs().get(1); assertThat(left.getObjectType()).isEqualTo("Person"); assertThat(right.getObjectType()).isEqualTo("Cheese"); assertThat(left.getConstraint().getDescrs()).hasSize(1); - ExprConstraintDescr fld = (ExprConstraintDescr) left.getConstraint().getDescrs().get( 0 ); + ExprConstraintDescr fld = (ExprConstraintDescr) left.getConstraint().getDescrs().get(0); assertThat(fld.getExpression()).isEqualTo("name == \"mark\""); assertThat(right.getConstraint().getDescrs()).hasSize(1); - fld = (ExprConstraintDescr) right.getConstraint().getDescrs().get( 0 ); + fld = (ExprConstraintDescr) right.getConstraint().getDescrs().get(0); assertThat(fld.getExpression()).isEqualTo("type == \"stilton\""); // now the "||" part - final OrDescr or = (OrDescr) and.getDescrs().get( 2 ); + final OrDescr or = (OrDescr) and.getDescrs().get(2); assertThat(or.getDescrs()).hasSize(2); - left = (PatternDescr) or.getDescrs().get( 0 ); - right = (PatternDescr) or.getDescrs().get( 1 ); + left = (PatternDescr) or.getDescrs().get(0); + right = (PatternDescr) or.getDescrs().get(1); assertThat(left.getObjectType()).isEqualTo("Person"); assertThat(right.getObjectType()).isEqualTo("Cheese"); assertThat(left.getConstraint().getDescrs()).hasSize(1); - fld = (ExprConstraintDescr) left.getConstraint().getDescrs().get( 0 ); + fld = (ExprConstraintDescr) left.getConstraint().getDescrs().get(0); assertThat(fld.getExpression()).isEqualTo("name == \"mark\""); assertThat(right.getConstraint().getDescrs()).hasSize(1); - fld = (ExprConstraintDescr) right.getConstraint().getDescrs().get( 0 ); + fld = (ExprConstraintDescr) right.getConstraint().getDescrs().get(0); assertThat(fld.getExpression()).isEqualTo("type == \"stilton\""); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "System.out.println( \"Mark and Michael\" );"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("System.out.println( \"Mark and Michael\" );"); } - /** test basic foo : Fact() || Fact() stuff */ + /** + * test basic foo : Fact() || Fact() stuff + */ @Test void orWithBinding() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "or_binding.drl" ); + "or_binding.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(2); - final OrDescr or = (OrDescr) rule.getLhs().getDescrs().get( 0 ); + final OrDescr or = (OrDescr) rule.getLhs().getDescrs().get(0); assertThat(or.getDescrs()).hasSize(2); - final PatternDescr leftPattern = (PatternDescr) or.getDescrs().get( 0 ); + final PatternDescr leftPattern = (PatternDescr) or.getDescrs().get(0); assertThat(leftPattern.getObjectType()).isEqualTo("Person"); assertThat(leftPattern.getIdentifier()).isEqualTo("foo"); - final PatternDescr rightPattern = (PatternDescr) or.getDescrs().get( 1 ); + final PatternDescr rightPattern = (PatternDescr) or.getDescrs().get(1); assertThat(rightPattern.getObjectType()).isEqualTo("Person"); assertThat(rightPattern.getIdentifier()).isEqualTo("foo"); - final PatternDescr cheeseDescr = (PatternDescr) rule.getLhs().getDescrs().get( 1 ); + final PatternDescr cheeseDescr = (PatternDescr) rule.getLhs().getDescrs().get(1); assertThat(cheeseDescr.getObjectType()).isEqualTo("Cheese"); assertThat(cheeseDescr.getIdentifier()).isEqualTo(null); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "System.out.println( \"Mark and Michael\" + bar );"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("System.out.println( \"Mark and Michael\" + bar );"); } - /** test basic foo : Fact() || Fact() stuff binding to an "or" */ + /** + * test basic foo : Fact() || Fact() stuff binding to an "or" + */ @Test void orBindingComplex() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "or_binding_complex.drl" ); + "or_binding_complex.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final OrDescr or = (OrDescr) rule.getLhs().getDescrs().get( 0 ); + final OrDescr or = (OrDescr) rule.getLhs().getDescrs().get(0); assertThat(or.getDescrs()).hasSize(2); // first fact - final PatternDescr firstFact = (PatternDescr) or.getDescrs().get( 0 ); + final PatternDescr firstFact = (PatternDescr) or.getDescrs().get(0); assertThat(firstFact.getObjectType()).isEqualTo("Person"); assertThat(firstFact.getIdentifier()).isEqualTo("foo"); // second "option" - final PatternDescr secondFact = (PatternDescr) or.getDescrs().get( 1 ); + final PatternDescr secondFact = (PatternDescr) or.getDescrs().get(1); assertThat(secondFact.getObjectType()).isEqualTo("Person"); assertThat(secondFact.getConstraint().getDescrs()).hasSize(1); assertThat(secondFact.getIdentifier()).isEqualTo("foo"); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "System.out.println( \"Mark and Michael\" + bar );"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("System.out.println( \"Mark and Michael\" + bar );"); } @Test void orBindingWithBrackets() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "or_binding_with_brackets.drl" ); + "or_binding_with_brackets.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final OrDescr or = (OrDescr) rule.getLhs().getDescrs().get( 0 ); + final OrDescr or = (OrDescr) rule.getLhs().getDescrs().get(0); assertThat(or.getDescrs()).hasSize(2); // first fact - final PatternDescr firstFact = (PatternDescr) or.getDescrs().get( 0 ); + final PatternDescr firstFact = (PatternDescr) or.getDescrs().get(0); assertThat(firstFact.getObjectType()).isEqualTo("Person"); assertThat(firstFact.getIdentifier()).isEqualTo("foo"); // second "option" - final PatternDescr secondFact = (PatternDescr) or.getDescrs().get( 0 ); + final PatternDescr secondFact = (PatternDescr) or.getDescrs().get(0); assertThat(secondFact.getObjectType()).isEqualTo("Person"); assertThat(secondFact.getIdentifier()).isEqualTo("foo"); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "System.out.println( \"Mark and Michael\" + bar );"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("System.out.println( \"Mark and Michael\" + bar );"); } @Test void parenthesesOrAndOr() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "brackets_precedence.drl" ); + "brackets_precedence.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); final AndDescr rootAnd = (AndDescr) rule.getLhs(); assertThat(rootAnd.getDescrs()).hasSize(2); - final OrDescr leftOr = (OrDescr) rootAnd.getDescrs().get( 0 ); + final OrDescr leftOr = (OrDescr) rootAnd.getDescrs().get(0); assertThat(leftOr.getDescrs()).hasSize(2); - final NotDescr not = (NotDescr) leftOr.getDescrs().get( 0 ); - final PatternDescr foo1 = (PatternDescr) not.getDescrs().get( 0 ); + final NotDescr not = (NotDescr) leftOr.getDescrs().get(0); + final PatternDescr foo1 = (PatternDescr) not.getDescrs().get(0); assertThat(foo1.getObjectType()).isEqualTo("Foo"); - final PatternDescr foo2 = (PatternDescr) leftOr.getDescrs().get( 1 ); + final PatternDescr foo2 = (PatternDescr) leftOr.getDescrs().get(1); assertThat(foo2.getObjectType()).isEqualTo("Foo"); - final OrDescr rightOr = (OrDescr) rootAnd.getDescrs().get( 1 ); + final OrDescr rightOr = (OrDescr) rootAnd.getDescrs().get(1); assertThat(rightOr.getDescrs()).hasSize(2); - final PatternDescr shoes = (PatternDescr) rightOr.getDescrs().get( 0 ); + final PatternDescr shoes = (PatternDescr) rightOr.getDescrs().get(0); assertThat(shoes.getObjectType()).isEqualTo("Shoes"); - final PatternDescr butt = (PatternDescr) rightOr.getDescrs().get( 1 ); + final PatternDescr butt = (PatternDescr) rightOr.getDescrs().get(1); assertThat(butt.getObjectType()).isEqualTo("Butt"); } @@ -1637,51 +1619,50 @@ void existsMultipleLevelNestWithThreeOrSiblings() { @Test void evalMultiple() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "eval_multiple.drl" ); + "eval_multiple.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(4); - final EvalDescr eval = (EvalDescr) rule.getLhs().getDescrs().get(0 ); - assertThat((String) eval.getContent()).isEqualToIgnoringWhitespace( "abc(\"foo\") + 5"); + final EvalDescr eval = (EvalDescr) rule.getLhs().getDescrs().get(0); + assertThat((String) eval.getContent()).isEqualToIgnoringWhitespace("abc(\"foo\") + 5"); - final PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 1 ); + final PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(1); assertThat(pattern.getObjectType()).isEqualTo("Foo"); - } @Test void withEval() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "with_eval.drl" ); + "with_eval.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(3); - PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("Foo"); - pattern = (PatternDescr) rule.getLhs().getDescrs().get( 1 ); + pattern = (PatternDescr) rule.getLhs().getDescrs().get(1); assertThat(pattern.getObjectType()).isEqualTo("Bar"); - final EvalDescr eval = (EvalDescr) rule.getLhs().getDescrs().get( 2 ); - assertThat((String) eval.getContent()).isEqualToIgnoringWhitespace( "abc(\"foo\")"); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "Kapow"); + final EvalDescr eval = (EvalDescr) rule.getLhs().getDescrs().get(2); + assertThat((String) eval.getContent()).isEqualToIgnoringWhitespace("abc(\"foo\")"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("Kapow"); } @Test void withRetval() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "with_retval.drl" ); + "with_retval.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr col = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr col = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(col.getConstraint().getDescrs()).hasSize(1); assertThat(col.getObjectType()).isEqualTo("Foo"); - final ExprConstraintDescr fld = (ExprConstraintDescr) col.getConstraint().getDescrs().get( 0 ); + final ExprConstraintDescr fld = (ExprConstraintDescr) col.getConstraint().getDescrs().get(0); assertThat(fld.getExpression()).isEqualToIgnoringWhitespace("name== (a + b)"); } @@ -1689,40 +1670,40 @@ void withRetval() { @Test void withPredicate() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "with_predicate.drl" ); + "with_predicate.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr col = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr col = (PatternDescr) rule.getLhs().getDescrs().get(0); AndDescr and = (AndDescr) col.getConstraint(); assertThat(and.getDescrs()).hasSize(2); - final ExprConstraintDescr field = (ExprConstraintDescr) col.getDescrs().get( 0 ); - final ExprConstraintDescr pred = (ExprConstraintDescr) and.getDescrs().get( 1 ); + final ExprConstraintDescr field = (ExprConstraintDescr) col.getDescrs().get(0); + final ExprConstraintDescr pred = (ExprConstraintDescr) and.getDescrs().get(1); assertThat(field.getExpression()).isEqualToIgnoringWhitespace("$age2:age"); - assertThat(pred.getExpression()).isEqualToIgnoringWhitespace( "$age2 == $age1+2"); + assertThat(pred.getExpression()).isEqualToIgnoringWhitespace("$age2 == $age1+2"); } @Test void notWithConstraint() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "not_with_constraint.drl" ); + "not_with_constraint.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(2); - PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); - final ExprConstraintDescr fieldBinding = (ExprConstraintDescr) pattern.getDescrs().get( 0 ); + PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); + final ExprConstraintDescr fieldBinding = (ExprConstraintDescr) pattern.getDescrs().get(0); assertThat(fieldBinding.getExpression()).isEqualToIgnoringWhitespace("$likes:like"); - final NotDescr not = (NotDescr) rule.getLhs().getDescrs().get( 1 ); - pattern = (PatternDescr) not.getDescrs().get( 0 ); + final NotDescr not = (NotDescr) rule.getLhs().getDescrs().get(1); + pattern = (PatternDescr) not.getDescrs().get(0); - final ExprConstraintDescr fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get( 0 ); + final ExprConstraintDescr fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get(0); assertThat(fld.getExpression()).isEqualToIgnoringWhitespace("type == $likes"); } @@ -1730,14 +1711,14 @@ void notWithConstraint() { @Test void functions() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "functions.drl" ); + "functions.drl"); assertThat(pkg.getRules()).hasSize(2); final List functions = pkg.getFunctions(); assertThat(functions).hasSize(2); - FunctionDescr func = functions.get( 0 ); + FunctionDescr func = functions.get(0); assertThat(func.getName()).isEqualTo("functionA"); assertThat(func.getReturnType()).isEqualTo("String"); assertThat(func.getParameterNames()).hasSize(2); @@ -1751,17 +1732,17 @@ void functions() { assertThat(func.getParameterTypes().get(1)).isEqualTo("Integer"); assertThat(func.getParameterNames().get(1)).isEqualTo("i"); - assertThat(func.getBody()).isEqualToIgnoringWhitespace( "foo();"); + assertThat(func.getBody()).isEqualToIgnoringWhitespace("foo();"); - func = functions.get( 1 ); + func = functions.get(1); assertThat(func.getName()).isEqualTo("functionB"); - assertThat(func.getText()).isEqualToIgnoringWhitespace( "bar();"); + assertThat(func.getText()).isEqualToIgnoringWhitespace("bar();"); } @Test void comment() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "comment.drl" ); + "comment.drl"); assertThat(pkg).isNotNull(); @@ -1771,34 +1752,34 @@ void comment() { @Test void attributes() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "rule_attributes.drl" ); + "rule_attributes.drl"); assertThat(rule.getName()).isEqualTo("simple_rule"); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "bar();"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("bar();"); final Map attrs = rule.getAttributes(); assertThat(attrs).hasSize(6); - AttributeDescr at = (AttributeDescr) attrs.get( "salience" ); + AttributeDescr at = (AttributeDescr) attrs.get("salience"); assertThat(at.getName()).isEqualTo("salience"); assertThat(at.getValue()).isEqualTo("42"); - at = (AttributeDescr) attrs.get( "agenda-group" ); + at = (AttributeDescr) attrs.get("agenda-group"); assertThat(at.getName()).isEqualTo("agenda-group"); assertThat(at.getValue()).isEqualTo("my_group"); - at = (AttributeDescr) attrs.get( "no-loop" ); + at = (AttributeDescr) attrs.get("no-loop"); assertThat(at.getName()).isEqualTo("no-loop"); assertThat(at.getValue()).isEqualTo("true"); - at = (AttributeDescr) attrs.get( "duration" ); + at = (AttributeDescr) attrs.get("duration"); assertThat(at.getName()).isEqualTo("duration"); assertThat(at.getValue()).isEqualTo("42"); - at = (AttributeDescr) attrs.get( "activation-group" ); + at = (AttributeDescr) attrs.get("activation-group"); assertThat(at.getName()).isEqualTo("activation-group"); assertThat(at.getValue()).isEqualTo("my_activation_group"); - at = (AttributeDescr) attrs.get( "lock-on-active" ); + at = (AttributeDescr) attrs.get("lock-on-active"); assertThat(at.getName()).isEqualTo("lock-on-active"); assertThat(at.getValue()).isEqualTo("true"); } @@ -1806,44 +1787,42 @@ void attributes() { @Test void attributes2() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "rule_attributes2.drl" ); - + "rule_attributes2.drl"); List rules = pkg.getRules(); assertThat(rules).hasSize(3); - RuleDescr rule = rules.get( 0 ); + RuleDescr rule = rules.get(0); assertThat(rule.getName()).isEqualTo("rule1"); Map attrs = rule.getAttributes(); assertThat(attrs).hasSize(2); - AttributeDescr at = (AttributeDescr) attrs.get( "salience" ); + AttributeDescr at = (AttributeDescr) attrs.get("salience"); assertThat(at.getName()).isEqualTo("salience"); assertThat(at.getValue()).isEqualTo("(42)"); - at = (AttributeDescr) attrs.get( "agenda-group" ); + at = (AttributeDescr) attrs.get("agenda-group"); assertThat(at.getName()).isEqualTo("agenda-group"); assertThat(at.getValue()).isEqualTo("my_group"); - rule = rules.get( 1 ); + rule = rules.get(1); assertThat(rule.getName()).isEqualTo("rule2"); attrs = rule.getAttributes(); assertThat(attrs).hasSize(2); - at = (AttributeDescr) attrs.get( "salience" ); + at = (AttributeDescr) attrs.get("salience"); assertThat(at.getName()).isEqualTo("salience"); assertThat(at.getValue()).isEqualTo("(Integer.MIN_VALUE)"); - at = (AttributeDescr) attrs.get( "no-loop" ); + at = (AttributeDescr) attrs.get("no-loop"); assertThat(at.getName()).isEqualTo("no-loop"); - rule = rules.get( 2 ); + rule = rules.get(2); assertThat(rule.getName()).isEqualTo("rule3"); attrs = rule.getAttributes(); assertThat(attrs).hasSize(2); - at = (AttributeDescr) attrs.get( "enabled" ); + at = (AttributeDescr) attrs.get("enabled"); assertThat(at.getName()).isEqualTo("enabled"); assertThat(at.getValue()).isEqualTo("(Boolean.TRUE)"); - at = (AttributeDescr) attrs.get( "activation-group" ); + at = (AttributeDescr) attrs.get("activation-group"); assertThat(at.getName()).isEqualTo("activation-group"); assertThat(at.getValue()).isEqualTo("my_activation_group"); - } @Test @@ -1851,39 +1830,37 @@ void attributeRefract() { final String source = "rule Test refract when Person() then end"; PackageDescr pkg = parseAndGetPackageDescr( - source ); - + source); - RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getName()).isEqualTo("Test"); Map attributes = rule.getAttributes(); assertThat(attributes).hasSize(1); - AttributeDescr refract = attributes.get( "refract" ); + AttributeDescr refract = attributes.get("refract"); assertThat(refract).isNotNull(); assertThat(refract.getValue()).isEqualTo("true"); - } @Test void enabledExpression() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "rule_enabled_expression.drl" ); + "rule_enabled_expression.drl"); assertThat(rule.getName()).isEqualTo("simple_rule"); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "bar();"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("bar();"); final Map attrs = rule.getAttributes(); assertThat(attrs).hasSize(3); - AttributeDescr at = (AttributeDescr) attrs.get( "enabled" ); + AttributeDescr at = (AttributeDescr) attrs.get("enabled"); assertThat(at.getName()).isEqualTo("enabled"); assertThat(at.getValue()).isEqualTo("( 1 + 1 == 2 )"); - at = (AttributeDescr) attrs.get( "salience" ); + at = (AttributeDescr) attrs.get("salience"); assertThat(at.getName()).isEqualTo("salience"); assertThat(at.getValue()).isEqualTo("( 1+2 )"); - at = (AttributeDescr) attrs.get( "lock-on-active" ); + at = (AttributeDescr) attrs.get("lock-on-active"); assertThat(at.getName()).isEqualTo("lock-on-active"); assertThat(at.getValue()).isEqualTo("true"); } @@ -1891,18 +1868,18 @@ void enabledExpression() { @Test void durationExpression() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "rule_duration_expression.drl" ); + "rule_duration_expression.drl"); assertThat(rule.getName()).isEqualTo("simple_rule"); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "bar();"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("bar();"); final Map attrs = rule.getAttributes(); assertThat(attrs).hasSize(2); - AttributeDescr at = (AttributeDescr) attrs.get( "duration" ); + AttributeDescr at = (AttributeDescr) attrs.get("duration"); assertThat(at.getName()).isEqualTo("duration"); assertThat(at.getValue()).isEqualTo("1h30m"); - at = (AttributeDescr) attrs.get( "lock-on-active" ); + at = (AttributeDescr) attrs.get("lock-on-active"); assertThat(at.getName()).isEqualTo("lock-on-active"); assertThat(at.getValue()).isEqualTo("true"); } @@ -1910,18 +1887,18 @@ void durationExpression() { @Test void calendars() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "rule_calendars_attribute.drl" ); + "rule_calendars_attribute.drl"); assertThat(rule.getName()).isEqualTo("simple_rule"); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "bar();"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("bar();"); final Map attrs = rule.getAttributes(); assertThat(attrs).hasSize(2); - AttributeDescr at = (AttributeDescr) attrs.get( "calendars" ); + AttributeDescr at = (AttributeDescr) attrs.get("calendars"); assertThat(at.getName()).isEqualTo("calendars"); assertThat(at.getValue()).isEqualTo("[ \"cal1\" ]"); - at = (AttributeDescr) attrs.get( "lock-on-active" ); + at = (AttributeDescr) attrs.get("lock-on-active"); assertThat(at.getName()).isEqualTo("lock-on-active"); assertThat(at.getValue()).isEqualTo("true"); } @@ -1929,37 +1906,37 @@ void calendars() { @Test void calendars2() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "rule_calendars_attribute2.drl" ); + "rule_calendars_attribute2.drl"); assertThat(rule.getName()).isEqualTo("simple_rule"); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "bar();"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("bar();"); final Map attrs = rule.getAttributes(); assertThat(attrs).hasSize(2); - AttributeDescr at = (AttributeDescr) attrs.get( "calendars" ); + AttributeDescr at = (AttributeDescr) attrs.get("calendars"); assertThat(at.getName()).isEqualTo("calendars"); assertThat(at.getValue()).isEqualTo("[ \"cal 1\", \"cal 2\", \"cal 3\" ]"); - at = (AttributeDescr) attrs.get( "lock-on-active" ); + at = (AttributeDescr) attrs.get("lock-on-active"); assertThat(at.getName()).isEqualTo("lock-on-active"); assertThat(at.getValue()).isEqualTo("true"); } @Test void timer() { - final RuleDescr rule = parseAndGetFirstRuleDescrFromFile("rule_timer_attribute.drl" ); + final RuleDescr rule = parseAndGetFirstRuleDescrFromFile("rule_timer_attribute.drl"); assertThat(rule.getName()).isEqualTo("simple_rule"); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "bar();"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("bar();"); final Map attrs = rule.getAttributes(); assertThat(attrs).hasSize(2); - AttributeDescr at = (AttributeDescr) attrs.get( "timer" ); + AttributeDescr at = (AttributeDescr) attrs.get("timer"); assertThat(at.getName()).isEqualTo("timer"); assertThat(at.getValue()).isEqualTo("int: 0 1; start=1_000_000, repeat-limit=0"); - at = (AttributeDescr) attrs.get( "lock-on-active" ); + at = (AttributeDescr) attrs.get("lock-on-active"); assertThat(at.getName()).isEqualTo("lock-on-active"); assertThat(at.getValue()).isEqualTo("true"); } @@ -1967,34 +1944,34 @@ void timer() { @Test void attributes_alternateSyntax() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "rule_attributes_alt.drl" ); + "rule_attributes_alt.drl"); assertThat(rule.getName()).isEqualTo("simple_rule"); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "bar();"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("bar();"); final Map attrs = rule.getAttributes(); assertThat(attrs).hasSize(6); - AttributeDescr at = attrs.get( "salience" ); + AttributeDescr at = attrs.get("salience"); assertThat(at.getName()).isEqualTo("salience"); assertThat(at.getValue()).isEqualTo("42"); - at = attrs.get( "agenda-group" ); + at = attrs.get("agenda-group"); assertThat(at.getName()).isEqualTo("agenda-group"); assertThat(at.getValue()).isEqualTo("my_group"); - at = attrs.get( "no-loop" ); + at = attrs.get("no-loop"); assertThat(at.getName()).isEqualTo("no-loop"); assertThat(at.getValue()).isEqualTo("true"); - at = attrs.get( "lock-on-active" ); + at = attrs.get("lock-on-active"); assertThat(at.getName()).isEqualTo("lock-on-active"); assertThat(at.getValue()).isEqualTo("true"); - at = attrs.get( "duration" ); + at = attrs.get("duration"); assertThat(at.getName()).isEqualTo("duration"); assertThat(at.getValue()).isEqualTo("42"); - at = attrs.get( "activation-group" ); + at = attrs.get("activation-group"); assertThat(at.getName()).isEqualTo("activation-group"); assertThat(at.getValue()).isEqualTo("my_activation_group"); } @@ -2002,13 +1979,13 @@ void attributes_alternateSyntax() { @Test void enumeration() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "enumeration.drl" ); + "enumeration.drl"); assertThat(rule.getName()).isEqualTo("simple_rule"); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr col = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr col = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(col.getObjectType()).isEqualTo("Foo"); assertThat(col.getConstraint().getDescrs()).hasSize(1); - final ExprConstraintDescr fld = (ExprConstraintDescr) col.getConstraint().getDescrs().get( 0 ); + final ExprConstraintDescr fld = (ExprConstraintDescr) col.getConstraint().getDescrs().get(0); assertThat(fld.getExpression()).isEqualToIgnoringWhitespace("bar == Foo.BAR"); } @@ -2016,17 +1993,16 @@ void enumeration() { @Test void extraLhsNewline() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "extra_lhs_newline.drl" ); - + "extra_lhs_newline.drl"); } @Test void soundsLike() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "soundslike_operator.drl" ); + "soundslike_operator.drl"); - RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); - PatternDescr pat = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + RuleDescr rule = (RuleDescr) pkg.getRules().get(0); + PatternDescr pat = (PatternDescr) rule.getLhs().getDescrs().get(0); pat.getConstraint(); } @@ -2034,12 +2010,12 @@ void soundsLike() { @Test void packageAttributes() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "package_attributes.drl" ); + "package_attributes.drl"); - AttributeDescr at = (AttributeDescr) pkg.getAttributes().get( 0 ); + AttributeDescr at = (AttributeDescr) pkg.getAttributes().get(0); assertThat(at.getName()).isEqualTo("agenda-group"); assertThat(at.getValue()).isEqualTo("x"); - at = (AttributeDescr) pkg.getAttributes().get( 1 ); + at = (AttributeDescr) pkg.getAttributes().get(1); assertThat(at.getName()).isEqualTo("dialect"); assertThat(at.getValue()).isEqualTo("java"); @@ -2047,30 +2023,29 @@ void packageAttributes() { assertThat(pkg.getImports()).hasSize(2); - RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getName()).isEqualTo("bar"); - at = (AttributeDescr) rule.getAttributes().get( "agenda-group" ); + at = (AttributeDescr) rule.getAttributes().get("agenda-group"); assertThat(at.getName()).isEqualTo("agenda-group"); assertThat(at.getValue()).isEqualTo("x"); - at = (AttributeDescr) rule.getAttributes().get( "dialect" ); + at = (AttributeDescr) rule.getAttributes().get("dialect"); assertThat(at.getName()).isEqualTo("dialect"); assertThat(at.getValue()).isEqualTo("java"); - rule = (RuleDescr) pkg.getRules().get( 1 ); + rule = (RuleDescr) pkg.getRules().get(1); assertThat(rule.getName()).isEqualTo("baz"); - at = (AttributeDescr) rule.getAttributes().get( "dialect" ); + at = (AttributeDescr) rule.getAttributes().get("dialect"); assertThat(at.getName()).isEqualTo("dialect"); assertThat(at.getValue()).isEqualTo("mvel"); - at = (AttributeDescr) rule.getAttributes().get( "agenda-group" ); + at = (AttributeDescr) rule.getAttributes().get("agenda-group"); assertThat(at.getName()).isEqualTo("agenda-group"); assertThat(at.getValue()).isEqualTo("x"); - } @Test void statementOrdering1() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "statement_ordering_1.drl" ); + "statement_ordering_1.drl"); assertThat(pkg.getRules()).hasSize(2); @@ -2092,7 +2067,7 @@ void statementOrdering1() { @Test void ruleNamesStartingWithNumbers() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "rule_names_number_prefix.drl" ); + "rule_names_number_prefix.drl"); assertThat(pkg.getRules()).hasSize(2); @@ -2103,29 +2078,28 @@ void ruleNamesStartingWithNumbers() { @Test void evalWithNewline() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "eval_with_newline.drl"); - + "eval_with_newline.drl"); } @Test void endPosition() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "test_EndPosition.drl" ); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); - final PatternDescr col = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + "test_EndPosition.drl"); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); + final PatternDescr col = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(col.getLine()).isEqualTo(23); assertThat(col.getEndLine()).isEqualTo(25); } @Test void groupBy() { - final PackageDescr pkg = parseAndGetPackageDescrFromFile( "groupBy.drl" ); + final PackageDescr pkg = parseAndGetPackageDescrFromFile("groupBy.drl"); assertThat(pkg.getRules()).hasSize(1); final RuleDescr rule = pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get(0); final GroupByDescr groupBy = (GroupByDescr) outPattern.getSource(); assertThat(groupBy.getGroupingKey()).isEqualToIgnoringWhitespace("$initial"); assertThat(groupBy.getGroupingFunction()).isEqualToIgnoringWhitespace("$p.getName().substring(0, 1)"); @@ -2157,11 +2131,11 @@ void groupBy() { @Test void qualifiedClassname() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "qualified_classname.drl" ); + "qualified_classname.drl"); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); - final PatternDescr p = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr p = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(p.getObjectType()).isEqualTo("com.cheeseco.Cheese"); } @@ -2176,7 +2150,7 @@ void accumulate() { "end"; RuleDescr rule = parseAndGetFirstRuleDescr(drl); - PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(out.getObjectType()).isEqualTo("Object"); AccumulateDescr accum = (AccumulateDescr) out.getSource(); assertThat(accum.isExternalFunction()).isTrue(); @@ -2201,18 +2175,18 @@ void accumulate() { @Test void fromAccumulate() { - final PackageDescr pkg = parseAndGetPackageDescrFromFile("from_accumulate.drl" ); + final PackageDescr pkg = parseAndGetPackageDescrFromFile("from_accumulate.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = pkg.getRules().get( 0 ); + final RuleDescr rule = pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get(0); final AccumulateDescr accum = (AccumulateDescr) outPattern.getSource(); - assertThat(accum.getInitCode()).isEqualTo( "int x = 0;"); - assertThat(accum.getActionCode()).isEqualTo( "x++;"); + assertThat(accum.getInitCode()).isEqualTo("int x = 0;"); + assertThat(accum.getActionCode()).isEqualTo("x++;"); assertThat(accum.getReverseCode()).isNull(); - assertThat(accum.getResultCode()).isEqualTo( "new Integer(x)"); + assertThat(accum.getResultCode()).isEqualTo("new Integer(x)"); assertThat(accum.isExternalFunction()).isFalse(); @@ -2231,18 +2205,18 @@ void fromAccumulate() { @Test void accumulateWithBindings() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "accumulate_with_bindings.drl" ); + "accumulate_with_bindings.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get(0); final AccumulateDescr accum = (AccumulateDescr) outPattern.getSource(); - assertThat(outPattern.getIdentifier()).isEqualTo( "$counter"); - assertThat(accum.getInitCode()).isEqualTo( "int x = 0;"); - assertThat(accum.getActionCode()).isEqualTo( "x++;"); - assertThat(accum.getResultCode()).isEqualTo( "new Integer(x)"); + assertThat(outPattern.getIdentifier()).isEqualTo("$counter"); + assertThat(accum.getInitCode()).isEqualTo("int x = 0;"); + assertThat(accum.getActionCode()).isEqualTo("x++;"); + assertThat(accum.getResultCode()).isEqualTo("new Integer(x)"); final PatternDescr pattern = (PatternDescr) accum.getInputPattern(); assertThat(pattern.getObjectType()).isEqualTo("Person"); @@ -2262,18 +2236,18 @@ void accumulateWithoutOptionalDelimiters() { + " result( new Integer(x) ) );\n" + "then\n" + "end\n"; - final PackageDescr pkg = parseAndGetPackageDescr( source ); + final PackageDescr pkg = parseAndGetPackageDescr(source); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get(0); final AccumulateDescr accum = (AccumulateDescr) outPattern.getSource(); - assertThat(outPattern.getIdentifier()).isEqualTo( "$counter"); - assertThat(accum.getInitCode()).isEqualTo( "int x = 0"); - assertThat(accum.getActionCode()).isEqualTo( "x++"); - assertThat(accum.getResultCode()).isEqualTo( "new Integer(x)"); + assertThat(outPattern.getIdentifier()).isEqualTo("$counter"); + assertThat(accum.getInitCode()).isEqualTo("int x = 0"); + assertThat(accum.getActionCode()).isEqualTo("x++"); + assertThat(accum.getResultCode()).isEqualTo("new Integer(x)"); final PatternDescr pattern = (PatternDescr) accum.getInputPattern(); assertThat(pattern.getObjectType()).isEqualTo("Person"); @@ -2285,18 +2259,18 @@ void accumulateWithoutOptionalDelimiters() { @Test void accumulateCount() { String source = "rule R when\n" + - " accumulate (\n" + - " Person(), $result : count() " + - " )" + - "then\n" + - "end"; - final PackageDescr pkg = parseAndGetPackageDescr( source ); + " accumulate (\n" + + " Person(), $result : count() " + + " )" + + "then\n" + + "end"; + final PackageDescr pkg = parseAndGetPackageDescr(source); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get(0); final AccumulateDescr accum = (AccumulateDescr) outPattern.getSource(); assertThat(accum).isNotNull(); @@ -2313,13 +2287,13 @@ void accumulateCount() { @Test void collect() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "collect.drl" ); + "collect.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get(0); final CollectDescr collect = (CollectDescr) outPattern.getSource(); final PatternDescr pattern = (PatternDescr) collect.getInputPattern(); @@ -2330,20 +2304,20 @@ void collect() { void predicate2() { // predicates are also prefixed by the eval keyword final RuleDescr rule = parseAndGetFirstRuleDescr( - "rule X when Foo(eval( $var.equals(\"xyz\") )) then end" ); + "rule X when Foo(eval( $var.equals(\"xyz\") )) then end"); - final PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); - final List< ? > constraints = pattern.getConstraint().getDescrs(); + final PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); + final List constraints = pattern.getConstraint().getDescrs(); assertThat(constraints).hasSize(1); - final ExprConstraintDescr predicate = (ExprConstraintDescr) constraints.get( 0 ); + final ExprConstraintDescr predicate = (ExprConstraintDescr) constraints.get(0); assertThat(predicate.getExpression()).isEqualToIgnoringWhitespace("eval( $var.equals(\"xyz\") )"); } @Test void escapedStrings() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "escaped-string.drl" ); + "escaped-string.drl"); assertThat(rule).isNotNull(); @@ -2351,30 +2325,30 @@ void escapedStrings() { final String expected = "String s = \"\\\"\\n\\t\\\\\";"; - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( expected); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace(expected); } @Test void nestedCEs() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "nested_conditional_elements.drl" ); + "nested_conditional_elements.drl"); assertThat(rule).isNotNull(); final AndDescr root = rule.getLhs(); - final NotDescr not1 = (NotDescr) root.getDescrs().get( 0 ); - final AndDescr and1 = (AndDescr) not1.getDescrs().get( 0 ); + final NotDescr not1 = (NotDescr) root.getDescrs().get(0); + final AndDescr and1 = (AndDescr) not1.getDescrs().get(0); - final PatternDescr state = (PatternDescr) and1.getDescrs().get( 0 ); - final NotDescr not2 = (NotDescr) and1.getDescrs().get( 1 ); - final AndDescr and2 = (AndDescr) not2.getDescrs().get( 0 ); - final PatternDescr person = (PatternDescr) and2.getDescrs().get( 0 ); - final PatternDescr cheese = (PatternDescr) and2.getDescrs().get( 1 ); + final PatternDescr state = (PatternDescr) and1.getDescrs().get(0); + final NotDescr not2 = (NotDescr) and1.getDescrs().get(1); + final AndDescr and2 = (AndDescr) not2.getDescrs().get(0); + final PatternDescr person = (PatternDescr) and2.getDescrs().get(0); + final PatternDescr cheese = (PatternDescr) and2.getDescrs().get(1); - final PatternDescr person2 = (PatternDescr) root.getDescrs().get( 1 ); - final OrDescr or = (OrDescr) root.getDescrs().get( 2 ); - final PatternDescr cheese2 = (PatternDescr) or.getDescrs().get( 0 ); - final PatternDescr cheese3 = (PatternDescr) or.getDescrs().get( 1 ); + final PatternDescr person2 = (PatternDescr) root.getDescrs().get(1); + final OrDescr or = (OrDescr) root.getDescrs().get(2); + final PatternDescr cheese2 = (PatternDescr) or.getDescrs().get(0); + final PatternDescr cheese3 = (PatternDescr) or.getDescrs().get(1); assertThat("State").isEqualTo(state.getObjectType()); assertThat("Person").isEqualTo(person.getObjectType()); @@ -2387,33 +2361,33 @@ void nestedCEs() { @Test void forall() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "forall.drl" ); + "forall.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final ForallDescr forall = (ForallDescr) rule.getLhs().getDescrs().get(0 ); + final ForallDescr forall = (ForallDescr) rule.getLhs().getDescrs().get(0); assertThat(forall.getDescrs()).hasSize(2); final PatternDescr pattern = forall.getBasePattern(); assertThat(pattern.getObjectType()).isEqualTo("Person"); final List remaining = forall.getRemainingPatterns(); assertThat(remaining).hasSize(1); - final PatternDescr cheese = (PatternDescr) remaining.get( 0 ); + final PatternDescr cheese = (PatternDescr) remaining.get(0); assertThat(cheese.getObjectType()).isEqualTo("Cheese"); } @Test void forallWithFrom() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "forallwithfrom.drl" ); + "forallwithfrom.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final ForallDescr forall = (ForallDescr) rule.getLhs().getDescrs().get( 0 ); + final ForallDescr forall = (ForallDescr) rule.getLhs().getDescrs().get(0); assertThat(forall.getDescrs()).hasSize(2); final PatternDescr pattern = forall.getBasePattern(); @@ -2421,7 +2395,7 @@ void forallWithFrom() { assertThat(((FromDescr) pattern.getSource()).getDataSource().toString()).isEqualTo("$village"); final List remaining = forall.getRemainingPatterns(); assertThat(remaining).hasSize(1); - final PatternDescr cheese = (PatternDescr) remaining.get( 0 ); + final PatternDescr cheese = (PatternDescr) remaining.get(0); assertThat(cheese.getObjectType()).isEqualTo("Cheese"); assertThat(((FromDescr) cheese.getSource()).getDataSource().toString()).isEqualTo("$cheesery"); } @@ -2430,11 +2404,11 @@ void forallWithFrom() { void memberof() { final String text = "rule X when Country( $cities : city )\nPerson( city memberOf $cities )\n then end"; AndDescr descrs = parseAndGetFirstRuleDescr( - text).getLhs(); + text).getLhs(); assertThat(descrs.getDescrs()).hasSize(2); - PatternDescr pat = (PatternDescr) descrs.getDescrs().get( 1 ); - ExprConstraintDescr fieldConstr = (ExprConstraintDescr) pat.getConstraint().getDescrs().get( 0 ); + PatternDescr pat = (PatternDescr) descrs.getDescrs().get(1); + ExprConstraintDescr fieldConstr = (ExprConstraintDescr) pat.getConstraint().getDescrs().get(0); assertThat(fieldConstr.getExpression()).isEqualTo("city memberOf $cities"); } @@ -2443,11 +2417,11 @@ void memberof() { void notMemberof() { final String text = "rule X when Country( $cities : city )\nPerson( city not memberOf $cities ) then end\n"; AndDescr descrs = parseAndGetFirstRuleDescr( - text).getLhs(); + text).getLhs(); assertThat(descrs.getDescrs()).hasSize(2); - PatternDescr pat = (PatternDescr) descrs.getDescrs().get( 1 ); - ExprConstraintDescr fieldConstr = (ExprConstraintDescr) pat.getConstraint().getDescrs().get( 0 ); + PatternDescr pat = (PatternDescr) descrs.getDescrs().get(1); + ExprConstraintDescr fieldConstr = (ExprConstraintDescr) pat.getConstraint().getDescrs().get(0); assertThat(fieldConstr.getExpression()).isEqualTo("city not memberOf $cities"); } @@ -2455,94 +2429,91 @@ void notMemberof() { @Test void inOperator() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "in_operator_test.drl" ); + "in_operator_test.drl"); assertThat(rule).isNotNull(); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "consequence();"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("consequence();"); assertThat(rule.getName()).isEqualTo("simple_rule"); assertThat(rule.getLhs().getDescrs()).hasSize(2); // The first pattern, with 2 restrictions on a single field (plus a // connective) - PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("Person"); assertThat(pattern.getConstraint().getDescrs()).hasSize(1); - ExprConstraintDescr fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get( 0 ); + ExprConstraintDescr fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get(0); assertThat(fld.getExpression()).isEqualTo("age > 30 && < 40"); // the second col, with 2 fields, the first with 2 restrictions, the // second field with one - pattern = (PatternDescr) rule.getLhs().getDescrs().get( 1 ); + pattern = (PatternDescr) rule.getLhs().getDescrs().get(1); assertThat(pattern.getObjectType()).isEqualTo("Vehicle"); assertThat(pattern.getConstraint().getDescrs()).hasSize(2); - fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get( 0 ); + fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get(0); assertThat(fld.getExpression()).isEqualToIgnoringWhitespace("type in ( \"sedan\", \"wagon\" )"); // now the second field - fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get( 1 ); + fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get(1); assertThat(fld.getExpression()).isEqualTo("age < 3"); - } @Test void notInOperator() { final RuleDescr rule = parseAndGetFirstRuleDescrFromFile( - "notin_operator_test.drl" ); + "notin_operator_test.drl"); assertThat(rule).isNotNull(); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "consequence();"); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("consequence();"); assertThat(rule.getName()).isEqualTo("simple_rule"); assertThat(rule.getLhs().getDescrs()).hasSize(2); // The first pattern, with 2 restrictions on a single field (plus a // connective) - PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("Person"); assertThat(pattern.getConstraint().getDescrs()).hasSize(1); - ExprConstraintDescr fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get( 0 ); + ExprConstraintDescr fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get(0); assertThat(fld.getExpression()).isEqualTo("age > 30 && < 40"); // the second col, with 2 fields, the first with 2 restrictions, the // second field with one - pattern = (PatternDescr) rule.getLhs().getDescrs().get( 1 ); + pattern = (PatternDescr) rule.getLhs().getDescrs().get(1); assertThat(pattern.getObjectType()).isEqualTo("Vehicle"); assertThat(pattern.getConstraint().getDescrs()).hasSize(2); - fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get( 0 ); + fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get(0); assertThat(fld.getExpression()).isEqualToIgnoringWhitespace("type not in ( \"sedan\", \"wagon\" )"); // now the second field - fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get( 1 ); + fld = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get(1); assertThat(fld.getExpression()).isEqualTo("age < 3"); - } @Test void checkOrDescr() { final String text = "rule X when Person( eval( age == 25 ) || ( eval( name.equals( \"bob\" ) ) && eval( age == 30 ) ) ) then end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.getDescrs()).hasSize(1); assertThat(AndDescr.class).isEqualTo(pattern.getConstraint().getClass()); assertThat(pattern.getConstraint().getDescrs().get(0).getClass()).isEqualTo(ExprConstraintDescr.class); - } @Test void constraintAndConnective() { final String text = "rule X when Person( age < 42 && location==\"atlanta\") then end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.getDescrs()).hasSize(1); - ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get( 0 ); + ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get(0); assertThat(fcd.getExpression()).isEqualToIgnoringWhitespace("age < 42 && location==\"atlanta\""); } @@ -2550,10 +2521,10 @@ void constraintAndConnective() { void constraintOrConnective() { final String text = "rule X when Person( age < 42 || location==\"atlanta\") then end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.getDescrs()).hasSize(1); - ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get( 0 ); + ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get(0); assertThat(fcd.getExpression()).isEqualToIgnoringWhitespace("age < 42 || location==\"atlanta\""); } @@ -2562,11 +2533,11 @@ void restrictions() { final String text = "rule X when Foo( bar > 1 || == 1 ) then end\n"; AndDescr descrs = (AndDescr) parseAndGetFirstRuleDescr( - text ).getLhs(); + text).getLhs(); assertThat(descrs.getDescrs()).hasSize(1); - PatternDescr pat = (PatternDescr) descrs.getDescrs().get( 0 ); - ExprConstraintDescr fieldConstr = (ExprConstraintDescr) pat.getConstraint().getDescrs().get( 0 ); + PatternDescr pat = (PatternDescr) descrs.getDescrs().get(0); + ExprConstraintDescr fieldConstr = (ExprConstraintDescr) pat.getConstraint().getDescrs().get(0); assertThat(fieldConstr.getExpression()).isEqualTo("bar > 1 || == 1"); } @@ -2574,55 +2545,53 @@ void restrictions() { @Test void semicolon() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "semicolon.drl" ); - - + "semicolon.drl"); assertThat(pkg.getName()).isEqualTo("org.drools.mvel.compiler"); assertThat(pkg.getGlobals()).hasSize(1); assertThat(pkg.getRules()).hasSize(3); - final RuleDescr rule1 = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule1 = (RuleDescr) pkg.getRules().get(0); assertThat(rule1.getLhs().getDescrs()).hasSize(2); - final RuleDescr query1 = (RuleDescr) pkg.getRules().get( 1 ); + final RuleDescr query1 = (RuleDescr) pkg.getRules().get(1); assertThat(query1.getLhs().getDescrs()).hasSize(3); - final RuleDescr rule2 = (RuleDescr) pkg.getRules().get( 2 ); + final RuleDescr rule2 = (RuleDescr) pkg.getRules().get(2); assertThat(rule2.getLhs().getDescrs()).hasSize(2); } @Test void eval() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "eval_parsing.drl" ); + "eval_parsing.drl"); assertThat(pkg.getName()).isEqualTo("org.drools.mvel.compiler"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule1 = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule1 = (RuleDescr) pkg.getRules().get(0); assertThat(rule1.getLhs().getDescrs()).hasSize(1); } @Test void accumulateReverse() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "accumulateReverse.drl" ); + "accumulateReverse.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get(0); final AccumulateDescr accum = (AccumulateDescr) out.getSource(); - assertThat(accum.getInitCode()).isEqualToIgnoringWhitespace( "int x = 0 ;" - ); - assertThat(accum.getActionCode()).isEqualToIgnoringWhitespace( "x++;" - ); - assertThat(accum.getReverseCode()).isEqualToIgnoringWhitespace( "x--;" - ); - assertThat(accum.getResultCode()).isEqualToIgnoringWhitespace( "new Integer(x)" - ); + assertThat(accum.getInitCode()).isEqualToIgnoringWhitespace("int x = 0 ;" + ); + assertThat(accum.getActionCode()).isEqualToIgnoringWhitespace("x++;" + ); + assertThat(accum.getReverseCode()).isEqualToIgnoringWhitespace("x--;" + ); + assertThat(accum.getResultCode()).isEqualToIgnoringWhitespace("new Integer(x)" + ); assertThat(accum.isExternalFunction()).isFalse(); final PatternDescr pattern = (PatternDescr) accum.getInputPattern(); @@ -2632,18 +2601,18 @@ void accumulateReverse() { @Test void accumulateExternalFunction() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "accumulateExternalFunction.drl" ); + "accumulateExternalFunction.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get(0); final AccumulateDescr accum = (AccumulateDescr) out.getSource(); - assertThat(accum.getFunctions().get( 0 ).getParams()[0]).isEqualToIgnoringWhitespace( "$age" - ); - assertThat(accum.getFunctions().get( 0 ).getFunction()).isEqualToIgnoringWhitespace( "average" - ); + assertThat(accum.getFunctions().get(0).getParams()[0]).isEqualToIgnoringWhitespace("$age" + ); + assertThat(accum.getFunctions().get(0).getFunction()).isEqualToIgnoringWhitespace("average" + ); assertThat(accum.isExternalFunction()).isTrue(); final PatternDescr pattern = (PatternDescr) accum.getInputPattern(); @@ -2653,13 +2622,13 @@ void accumulateExternalFunction() { @Test void collectWithNestedFrom() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "collect_with_nested_from.drl" ); + "collect_with_nested_from.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get(0); final CollectDescr collect = (CollectDescr) out.getSource(); PatternDescr person = (PatternDescr) collect.getInputPattern(); @@ -2674,13 +2643,13 @@ void collectWithNestedFrom() { @Test void accumulateWithNestedFrom() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "accumulate_with_nested_from.drl" ); + "accumulate_with_nested_from.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get(0); final AccumulateDescr accumulate = (AccumulateDescr) out.getSource(); PatternDescr person = (PatternDescr) accumulate.getInputPattern(); @@ -2695,14 +2664,14 @@ void accumulateWithNestedFrom() { @Test void accumulateMultipleFunctions() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "accumulateMultipleFunctions.drl" ); + "accumulateMultipleFunctions.drl"); assertThat(pkg.getRules()).hasSize(1); - RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(out.getObjectType()).isEqualTo("Object"); AccumulateDescr accum = (AccumulateDescr) out.getSource(); assertThat(accum.isExternalFunction()).isTrue(); @@ -2735,14 +2704,14 @@ void accumulateMnemonic() { "then\n" + "end\n"; PackageDescr pkg = parseAndGetPackageDescr( - drl ); + drl); assertThat(pkg.getRules()).hasSize(1); - RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(out.getObjectType()).isEqualTo("Object"); AccumulateDescr accum = (AccumulateDescr) out.getSource(); assertThat(accum.isExternalFunction()).isTrue(); @@ -2767,14 +2736,14 @@ void accumulateMnemonic2() { "then\n" + "end\n"; PackageDescr pkg = parseAndGetPackageDescr( - drl ); + drl); assertThat(pkg.getRules()).hasSize(1); - RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(out.getObjectType()).isEqualTo("Number"); AccumulateDescr accum = (AccumulateDescr) out.getSource(); assertThat(accum.isExternalFunction()).isTrue(); @@ -2801,7 +2770,7 @@ void importAccumulate() { "then\n" + "end\n"; PackageDescr pkg = parseAndGetPackageDescr( - drl ); + drl); assertThat(pkg.getAccumulateImports()).hasSize(2); AccumulateImportDescr imp = (AccumulateImportDescr) pkg.getAccumulateImports().get(0); @@ -2814,10 +2783,10 @@ void importAccumulate() { assertThat(pkg.getRules()).hasSize(1); - RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(out.getObjectType()).isEqualTo("Object"); AccumulateDescr accum = (AccumulateDescr) out.getSource(); assertThat(accum.isExternalFunction()).isTrue(); @@ -2839,14 +2808,14 @@ void importAccumulate() { @Test void accumulateMultipleFunctionsConstraint() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "accumulateMultipleFunctionsConstraint.drl" ); + "accumulateMultipleFunctionsConstraint.drl"); assertThat(pkg.getRules()).hasSize(1); - RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr out = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(out.getObjectType()).isEqualTo("Object"); assertThat(out.getConstraint().getDescrs()).hasSize(2); assertThat(out.getConstraint().getDescrs().get(0).toString()).isEqualTo("$a1 > 10 && $M1 <= 100"); @@ -2875,23 +2844,23 @@ void accumulateMultipleFunctionsConstraint() { @Test void orCE() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "or_ce.drl" ); + "or_ce.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(2); - final PatternDescr person = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr person = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(person.getObjectType()).isEqualTo("Person"); assertThat(person.getIdentifier()).isEqualTo("$p"); - final OrDescr or = (OrDescr) rule.getLhs().getDescrs().get( 1 ); + final OrDescr or = (OrDescr) rule.getLhs().getDescrs().get(1); assertThat(or.getDescrs()).hasSize(2); - final PatternDescr cheese1 = (PatternDescr) or.getDescrs().get( 0 ); + final PatternDescr cheese1 = (PatternDescr) or.getDescrs().get(0); assertThat(cheese1.getObjectType()).isEqualTo("Cheese"); assertThat(cheese1.getIdentifier()).isEqualTo("$c"); - final PatternDescr cheese2 = (PatternDescr) or.getDescrs().get( 1 ); + final PatternDescr cheese2 = (PatternDescr) or.getDescrs().get(1); assertThat(cheese2.getObjectType()).isEqualTo("Cheese"); assertThat(cheese2.getIdentifier()).isNull(); } @@ -2900,70 +2869,69 @@ void orCE() { void ruleSingleLine() { final String text = "rule \"another test\" salience 10 when eval( true ) then System.out.println(1); end"; RuleDescr rule = parseAndGetFirstRuleDescr( - text ); + text); assertThat(rule.getName()).isEqualTo("another test"); - assertThat((String)rule.getConsequence()).isEqualToIgnoringWhitespace("System.out.println(1); "); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("System.out.println(1); "); } @Test void ruleTwoLines() { final String text = "rule \"another test\" salience 10 when eval( true ) then System.out.println(1);\n end"; RuleDescr rule = parseAndGetFirstRuleDescr( - text ); + text); assertThat(rule.getName()).isEqualTo("another test"); - assertThat((String)rule.getConsequence()).isEqualToIgnoringWhitespace("System.out.println(1);\n "); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("System.out.println(1);\n "); } @Test void ruleParseLhs3() { final String text = "rule X when (or\nnot Person()\n(and Cheese()\nMeat()\nWine())) then end"; AndDescr pattern = parseAndGetFirstRuleDescr( - text ).getLhs(); + text).getLhs(); assertThat(pattern.getDescrs()).hasSize(1); - OrDescr or = (OrDescr) pattern.getDescrs().get( 0 ); + OrDescr or = (OrDescr) pattern.getDescrs().get(0); assertThat(or.getDescrs()).hasSize(2); - NotDescr not = (NotDescr) or.getDescrs().get( 0 ); - AndDescr and = (AndDescr) or.getDescrs().get( 1 ); + NotDescr not = (NotDescr) or.getDescrs().get(0); + AndDescr and = (AndDescr) or.getDescrs().get(1); assertThat(not.getDescrs()).hasSize(1); - PatternDescr person = (PatternDescr) not.getDescrs().get( 0 ); + PatternDescr person = (PatternDescr) not.getDescrs().get(0); assertThat(person.getObjectType()).isEqualTo("Person"); assertThat(and.getDescrs()).hasSize(3); - PatternDescr cheese = (PatternDescr) and.getDescrs().get( 0 ); + PatternDescr cheese = (PatternDescr) and.getDescrs().get(0); assertThat(cheese.getObjectType()).isEqualTo("Cheese"); - PatternDescr meat = (PatternDescr) and.getDescrs().get( 1 ); + PatternDescr meat = (PatternDescr) and.getDescrs().get(1); assertThat(meat.getObjectType()).isEqualTo("Meat"); - PatternDescr wine = (PatternDescr) and.getDescrs().get( 2 ); + PatternDescr wine = (PatternDescr) and.getDescrs().get(2); assertThat(wine.getObjectType()).isEqualTo("Wine"); - } @Test void accumulateMultiPattern() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "accumulate_multi_pattern.drl" ); + "accumulate_multi_pattern.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(1); - final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get(0); final AccumulateDescr accum = (AccumulateDescr) outPattern.getSource(); - assertThat(outPattern.getIdentifier()).isEqualToIgnoringWhitespace( "$counter" - ); - assertThat(accum.getInitCode()).isEqualToIgnoringWhitespace( "int x = 0 ;" - ); - assertThat(accum.getActionCode()).isEqualToIgnoringWhitespace( "x++;" - ); - assertThat(accum.getResultCode()).isEqualToIgnoringWhitespace( "new Integer(x)" - ); + assertThat(outPattern.getIdentifier()).isEqualToIgnoringWhitespace("$counter" + ); + assertThat(accum.getInitCode()).isEqualToIgnoringWhitespace("int x = 0 ;" + ); + assertThat(accum.getActionCode()).isEqualToIgnoringWhitespace("x++;" + ); + assertThat(accum.getResultCode()).isEqualToIgnoringWhitespace("new Integer(x)" + ); final AndDescr and = (AndDescr) accum.getInput(); assertThat(and.getDescrs()).hasSize(2); - final PatternDescr person = (PatternDescr) and.getDescrs().get( 0 ); - final PatternDescr cheese = (PatternDescr) and.getDescrs().get( 1 ); + final PatternDescr person = (PatternDescr) and.getDescrs().get(0); + final PatternDescr cheese = (PatternDescr) and.getDescrs().get(1); assertThat(person.getObjectType()).isEqualTo("Person"); assertThat(cheese.getObjectType()).isEqualTo("Cheese"); } @@ -2972,74 +2940,70 @@ void accumulateMultiPattern() { void pluggableOperators() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "pluggable_operators.drl" ); - + "pluggable_operators.drl"); assertThat(pkg.getRules()).hasSize(1); - final RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + final RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getLhs().getDescrs()).hasSize(5); - final PatternDescr eventA = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr eventA = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(eventA.getIdentifier()).isEqualTo("$a"); assertThat(eventA.getObjectType()).isEqualTo("EventA"); - final PatternDescr eventB = (PatternDescr) rule.getLhs().getDescrs().get( 1 ); + final PatternDescr eventB = (PatternDescr) rule.getLhs().getDescrs().get(1); assertThat(eventB.getIdentifier()).isEqualTo("$b"); assertThat(eventB.getObjectType()).isEqualTo("EventB"); assertThat(eventB.getConstraint().getDescrs()).hasSize(1); assertThat(eventB.getConstraint().getDescrs()).hasSize(1); - final ExprConstraintDescr fcdB = (ExprConstraintDescr) eventB.getConstraint().getDescrs().get( 0 ); + final ExprConstraintDescr fcdB = (ExprConstraintDescr) eventB.getConstraint().getDescrs().get(0); assertThat(fcdB.getExpression()).isEqualTo("this after[1,10] $a || this not after[15,20] $a"); - final PatternDescr eventC = (PatternDescr) rule.getLhs().getDescrs().get( 2 ); + final PatternDescr eventC = (PatternDescr) rule.getLhs().getDescrs().get(2); assertThat(eventC.getIdentifier()).isEqualTo("$c"); assertThat(eventC.getObjectType()).isEqualTo("EventC"); assertThat(eventC.getConstraint().getDescrs()).hasSize(1); - final ExprConstraintDescr fcdC = (ExprConstraintDescr) eventC.getConstraint().getDescrs().get( 0 ); + final ExprConstraintDescr fcdC = (ExprConstraintDescr) eventC.getConstraint().getDescrs().get(0); assertThat(fcdC.getExpression()).isEqualTo("this finishes $b"); - final PatternDescr eventD = (PatternDescr) rule.getLhs().getDescrs().get( 3 ); + final PatternDescr eventD = (PatternDescr) rule.getLhs().getDescrs().get(3); assertThat(eventD.getIdentifier()).isEqualTo("$d"); assertThat(eventD.getObjectType()).isEqualTo("EventD"); assertThat(eventD.getConstraint().getDescrs()).hasSize(1); - final ExprConstraintDescr fcdD = (ExprConstraintDescr) eventD.getConstraint().getDescrs().get( 0 ); + final ExprConstraintDescr fcdD = (ExprConstraintDescr) eventD.getConstraint().getDescrs().get(0); assertThat(fcdD.getExpression()).isEqualTo("this not starts $a"); - final PatternDescr eventE = (PatternDescr) rule.getLhs().getDescrs().get( 4 ); + final PatternDescr eventE = (PatternDescr) rule.getLhs().getDescrs().get(4); assertThat(eventE.getIdentifier()).isEqualTo("$e"); assertThat(eventE.getObjectType()).isEqualTo("EventE"); assertThat(eventE.getConstraint().getDescrs()).hasSize(1); - ExprConstraintDescr fcdE = (ExprConstraintDescr) eventE.getConstraint().getDescrs().get( 0 ); + ExprConstraintDescr fcdE = (ExprConstraintDescr) eventE.getConstraint().getDescrs().get(0); assertThat(fcdE.getExpression()).isEqualTo("this not before[1, 10] $b || after[1, 10] $c && this after[1, 5] $d"); } @Test void ruleMetadata() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "Rule_with_Metadata.drl" ); - + "Rule_with_Metadata.drl"); // @fooAttribute(barValue) // @fooAtt2(barVal2) - RuleDescr rule = pkg.getRules().get( 0 ); + RuleDescr rule = pkg.getRules().get(0); assertThat(rule.getAnnotationNames()).contains("fooMeta1"); assertThat(rule.getAnnotation("fooMeta1").getValue()).isEqualTo("barVal1"); assertThat(rule.getAnnotationNames()).contains("fooMeta2"); assertThat(rule.getAnnotation("fooMeta2").getValue()).isEqualTo("barVal2"); - assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace( "System.out.println(\"Consequence\");" - ); + assertThat((String) rule.getConsequence()).isEqualToIgnoringWhitespace("System.out.println(\"Consequence\");" + ); } @Test void ruleExtends() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "Rule_with_Extends.drl" ); - + "Rule_with_Extends.drl"); - - RuleDescr rule = pkg.getRules().get( 0 ); + RuleDescr rule = pkg.getRules().get(0); assertThat(rule.getParentName() != null).isTrue(); assertThat(rule.getParentName()).isEqualTo("rule1"); @@ -3047,37 +3011,35 @@ void ruleExtends() { assertThat(lhs).isNotNull(); assertThat(lhs.getDescrs()).hasSize(1); - PatternDescr pattern = (PatternDescr) lhs.getDescrs().get( 0 ); + PatternDescr pattern = (PatternDescr) lhs.getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("foo"); assertThat(pattern.getIdentifier()).isEqualTo("$foo"); - } @Test void typeDeclarationWithFields() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "declare_type_with_fields.drl" ); - + "declare_type_with_fields.drl"); List td = pkg.getTypeDeclarations(); assertThat(td).hasSize(3); - TypeDeclarationDescr d = td.get( 0 ); + TypeDeclarationDescr d = td.get(0); assertThat(d.getTypeName()).isEqualTo("SomeFact"); assertThat(d.getFields()).hasSize(2); assertThat(d.getFields()).containsKey("name"); assertThat(d.getFields()).containsKey("age"); - TypeFieldDescr f = d.getFields().get("name" ); + TypeFieldDescr f = d.getFields().get("name"); assertThat(f.getPattern().getObjectType()).isEqualTo("String"); - f = d.getFields().get( "age" ); + f = d.getFields().get("age"); assertThat(f.getPattern().getObjectType()).isEqualTo("Integer"); - d = td.get( 1 ); + d = td.get(1); assertThat(d.getTypeName()).isEqualTo("AnotherFact"); - TypeDeclarationDescr type = td.get( 2 ); + TypeDeclarationDescr type = td.get(2); assertThat(type.getTypeName()).isEqualTo("Person"); assertThat(type.getAnnotation("role").getValue()).isEqualTo("fact"); @@ -3086,74 +3048,71 @@ void typeDeclarationWithFields() { assertThat(type.getAnnotation("doc").getValue("date")).isEqualTo("Calendar.getInstance().getDate()"); assertThat(type.getFields()).hasSize(2); - TypeFieldDescr field = type.getFields().get( "name" ); + TypeFieldDescr field = type.getFields().get("name"); assertThat(field.getFieldName()).isEqualTo("name"); assertThat(field.getPattern().getObjectType()).isEqualTo("String"); assertThat(field.getInitExpr()).isEqualTo("\"John Doe\""); assertThat(field.getAnnotation("length").getValue("max")).isEqualTo("50"); - assertThat( field.getAnnotation( "key" ) ).isNotNull(); + assertThat(field.getAnnotation("key")).isNotNull(); - field = type.getFields().get( "age" ); + field = type.getFields().get("age"); assertThat(field.getFieldName()).isEqualTo("age"); assertThat(field.getPattern().getObjectType()).isEqualTo("int"); assertThat(field.getInitExpr()).isEqualTo("-1"); assertThat(field.getAnnotation("ranged").getValue("min")).isEqualTo("0"); assertThat(field.getAnnotation("ranged").getValue("max")).isEqualTo("150"); assertThat(field.getAnnotation("ranged").getValue("unknown")).isEqualTo("-1"); - } @Test void qualifiedTypeDeclaration() { final PackageDescr pkg = parseAndGetPackageDescrFromFile( - "qualified_type_declaration.drl" ); + "qualified_type_declaration.drl"); - TypeDeclarationDescr someFact = pkg.getTypeDeclarations().get( 0 ); + TypeDeclarationDescr someFact = pkg.getTypeDeclarations().get(0); assertThat(someFact.getTypeName()).isEqualTo("SomeFact"); assertThat(someFact.getNamespace()).isEqualTo("com.sample1"); - EnumDeclarationDescr color = pkg.getEnumDeclarations().get( 0 ); + EnumDeclarationDescr color = pkg.getEnumDeclarations().get(0); assertThat(color.getTypeName()).isEqualTo("Color"); assertThat(color.getNamespace()).isEqualTo("com.sample2"); } @Test void parenthesesOneLevelNestWithThreeSiblings() { - final PackageDescr pkg = parseAndGetPackageDescrFromFile( "Rule_with_nested_LHS.drl" ); + final PackageDescr pkg = parseAndGetPackageDescrFromFile("Rule_with_nested_LHS.drl"); - - - RuleDescr rule = pkg.getRules().get( 0 ); + RuleDescr rule = pkg.getRules().get(0); assertThat(rule.getName()).isEqualTo("test"); AndDescr lhs = rule.getLhs(); assertThat(lhs).isNotNull(); assertThat(lhs.getDescrs()).hasSize(2); - PatternDescr a = (PatternDescr) lhs.getDescrs().get( 0 ); + PatternDescr a = (PatternDescr) lhs.getDescrs().get(0); assertThat(a.getObjectType()).isEqualTo("A"); - OrDescr or = (OrDescr) lhs.getDescrs().get( 1 ); + OrDescr or = (OrDescr) lhs.getDescrs().get(1); assertThat(or.getDescrs()).hasSize(3); - AndDescr and1 = (AndDescr) or.getDescrs().get( 0 ); + AndDescr and1 = (AndDescr) or.getDescrs().get(0); assertThat(and1.getDescrs()).hasSize(2); - PatternDescr b = (PatternDescr) and1.getDescrs().get( 0 ); - PatternDescr c = (PatternDescr) and1.getDescrs().get( 1 ); + PatternDescr b = (PatternDescr) and1.getDescrs().get(0); + PatternDescr c = (PatternDescr) and1.getDescrs().get(1); assertThat(b.getObjectType()).isEqualTo("B"); assertThat(c.getObjectType()).isEqualTo("C"); - AndDescr and2 = (AndDescr) or.getDescrs().get( 1 ); + AndDescr and2 = (AndDescr) or.getDescrs().get(1); assertThat(and2.getDescrs()).hasSize(2); - PatternDescr d = (PatternDescr) and2.getDescrs().get( 0 ); - PatternDescr e = (PatternDescr) and2.getDescrs().get( 1 ); + PatternDescr d = (PatternDescr) and2.getDescrs().get(0); + PatternDescr e = (PatternDescr) and2.getDescrs().get(1); assertThat(d.getObjectType()).isEqualTo("D"); assertThat(e.getObjectType()).isEqualTo("E"); - AndDescr and3 = (AndDescr) or.getDescrs().get( 2 ); + AndDescr and3 = (AndDescr) or.getDescrs().get(2); assertThat(and3.getDescrs()).hasSize(2); - PatternDescr f = (PatternDescr) and3.getDescrs().get( 0 ); - PatternDescr g = (PatternDescr) and3.getDescrs().get( 1 ); + PatternDescr f = (PatternDescr) and3.getDescrs().get(0); + PatternDescr g = (PatternDescr) and3.getDescrs().get(1); assertThat(f.getObjectType()).isEqualTo("F"); assertThat(g.getObjectType()).isEqualTo("G"); } @@ -3163,14 +3122,13 @@ void entryPoint() { final String text = "rule X when StockTick( symbol==\"ACME\") from entry-point StreamA then end"; PackageDescr pkg = parseAndGetPackageDescr( - text ); + text); - - RuleDescr rule = pkg.getRules().get( 0 ); - PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + RuleDescr rule = pkg.getRules().get(0); + PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(pattern.getDescrs()).hasSize(1); - ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get( 0 ); + ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get(0); assertThat(fcd.getExpression()).isEqualTo("symbol==\"ACME\""); assertThat(pattern.getSource()).isNotNull(); @@ -3183,14 +3141,13 @@ void entryPoint2() { final String text = "rule X when StockTick( symbol==\"ACME\") from entry-point \"StreamA\" then end"; PackageDescr pkg = parseAndGetPackageDescr( - text ); + text); - - RuleDescr rule = pkg.getRules().get( 0 ); - PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + RuleDescr rule = pkg.getRules().get(0); + PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(pattern.getDescrs()).hasSize(1); - ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get( 0 ); + ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get(0); assertThat(fcd.getExpression()).isEqualTo("symbol==\"ACME\""); assertThat(pattern.getSource()).isNotNull(); @@ -3202,20 +3159,19 @@ void entryPoint2() { void slidingWindow() { final String text = "rule X when StockTick( symbol==\"ACME\") over window:length(10) then end"; - PackageDescr pkg = parseAndGetPackageDescr( text ); - + PackageDescr pkg = parseAndGetPackageDescr(text); - RuleDescr rule = pkg.getRules().get( 0 ); - PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + RuleDescr rule = pkg.getRules().get(0); + PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(pattern.getDescrs()).hasSize(1); - ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get( 0 ); + ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get(0); assertThat(fcd.getExpression()).isEqualTo("symbol==\"ACME\""); List behaviors = pattern.getBehaviors(); assertThat(behaviors).isNotNull(); assertThat(behaviors).hasSize(1); - BehaviorDescr descr = behaviors.get( 0 ); + BehaviorDescr descr = behaviors.get(0); assertThat(descr.getType()).isEqualTo("window"); assertThat(descr.getSubType()).isEqualTo("length"); assertThat(descr.getParameters().get(0)).isEqualTo("10"); @@ -3226,19 +3182,18 @@ void ruleOldSyntax1() { final String source = "rule \"Test\" when ( not $r :LiteralRestriction( operator == Operator.EQUAL ) ) then end"; PackageDescr pkg = parseAndGetPackageDescr( - source ); - + source); - RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getName()).isEqualTo("Test"); assertThat(rule.getLhs().getDescrs()).hasSize(1); assertThat(((NotDescr) rule.getLhs().getDescrs().get(0)).getDescrs()).hasSize(1); - NotDescr notDescr = (NotDescr) rule.getLhs().getDescrs().get( 0 ); - PatternDescr patternDescr = (PatternDescr) notDescr.getDescrs().get( 0 ); + NotDescr notDescr = (NotDescr) rule.getLhs().getDescrs().get(0); + PatternDescr patternDescr = (PatternDescr) notDescr.getDescrs().get(0); assertThat(patternDescr.getIdentifier()).isEqualTo("$r"); assertThat(patternDescr.getDescrs()).hasSize(1); - ExprConstraintDescr fieldConstraintDescr = (ExprConstraintDescr) patternDescr.getDescrs().get( 0 ); + ExprConstraintDescr fieldConstraintDescr = (ExprConstraintDescr) patternDescr.getDescrs().get(0); assertThat(fieldConstraintDescr.getExpression()).isEqualToIgnoringWhitespace("operator == Operator.EQUAL"); } @@ -3247,17 +3202,16 @@ void ruleOldSyntax2() { final String source = "rule \"Test\" when ( $r :LiteralRestriction( operator == Operator.EQUAL ) ) then end"; PackageDescr pkg = parseAndGetPackageDescr( - source ); - + source); - RuleDescr rule = (RuleDescr) pkg.getRules().get( 0 ); + RuleDescr rule = (RuleDescr) pkg.getRules().get(0); assertThat(rule.getName()).isEqualTo("Test"); assertThat(rule.getLhs().getDescrs()).hasSize(1); - PatternDescr patternDescr = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr patternDescr = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(patternDescr.getIdentifier()).isEqualTo("$r"); assertThat(patternDescr.getDescrs()).hasSize(1); - ExprConstraintDescr fieldConstraintDescr = (ExprConstraintDescr) patternDescr.getDescrs().get( 0 ); + ExprConstraintDescr fieldConstraintDescr = (ExprConstraintDescr) patternDescr.getDescrs().get(0); assertThat(fieldConstraintDescr.getExpression()).isEqualToIgnoringWhitespace("operator == Operator.EQUAL"); } @@ -3265,9 +3219,7 @@ void ruleOldSyntax2() { void typeWithMetaData() { PackageDescr pkg = parseAndGetPackageDescrFromFile( - "type_with_meta.drl" ); - - + "type_with_meta.drl"); final List declarations = pkg.getTypeDeclarations(); @@ -3278,10 +3230,10 @@ void typeWithMetaData() { void nullConstraints() { final String text = "rule X when Person( name == null ) then end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.getDescrs()).hasSize(1); - ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get( 0 ); + ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get(0); assertThat(fcd.getExpression()).isEqualTo("name == null"); assertThat(fcd.getPosition()).isEqualTo(0); assertThat(fcd.getType()).isEqualTo(ExprConstraintDescr.Type.NAMED); @@ -3291,14 +3243,14 @@ void nullConstraints() { void positionalConstraintsOnly() { final String text = "rule X when Person( \"Mark\", 42; ) then end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.getDescrs()).hasSize(2); - ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get( 0 ); + ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get(0); assertThat(fcd.getExpression()).isEqualTo("\"Mark\""); assertThat(fcd.getPosition()).isEqualTo(0); assertThat(fcd.getType()).isEqualTo(ExprConstraintDescr.Type.POSITIONAL); - fcd = (ExprConstraintDescr) pattern.getDescrs().get( 1 ); + fcd = (ExprConstraintDescr) pattern.getDescrs().get(1); assertThat(fcd.getExpression()).isEqualTo("42"); assertThat(fcd.getPosition()).isEqualTo(1); assertThat(fcd.getType()).isEqualTo(ExprConstraintDescr.Type.POSITIONAL); @@ -3308,16 +3260,16 @@ void positionalConstraintsOnly() { void isQuery() { final String text = "rule X when ?person( \"Mark\", 42; ) then end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.isQuery()).isTrue(); assertThat(pattern.getDescrs()).hasSize(2); - ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get( 0 ); + ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get(0); assertThat(fcd.getExpression()).isEqualTo("\"Mark\""); assertThat(fcd.getPosition()).isEqualTo(0); assertThat(fcd.getType()).isEqualTo(ExprConstraintDescr.Type.POSITIONAL); - fcd = (ExprConstraintDescr) pattern.getDescrs().get( 1 ); + fcd = (ExprConstraintDescr) pattern.getDescrs().get(1); assertThat(fcd.getExpression()).isEqualTo("42"); assertThat(fcd.getPosition()).isEqualTo(1); assertThat(fcd.getType()).isEqualTo(ExprConstraintDescr.Type.POSITIONAL); @@ -3329,18 +3281,16 @@ void fromFollowedByQuery() { // prefix for queries from the ternary operator "? :" final String text = "rule X when Cheese() from $cheesery ?person( \"Mark\", 42; ) then end"; RuleDescr rule = parseAndGetFirstRuleDescr( - text ); + text); - - PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("Cheese"); assertThat(pattern.getSource().getText()).isEqualTo("from $cheesery"); assertThat(pattern.isQuery()).isFalse(); - pattern = (PatternDescr) rule.getLhs().getDescrs().get( 1 ); + pattern = (PatternDescr) rule.getLhs().getDescrs().get(1); assertThat(pattern.getObjectType()).isEqualTo("person"); assertThat(pattern.isQuery()).isTrue(); - } @Test @@ -3349,18 +3299,16 @@ void fromWithTernaryFollowedByQuery() { // prefix for queries from the ternary operator "? :" final String text = "rule X when Cheese() from (isFull ? $cheesery : $market) ?person( \"Mark\", 42; ) then end"; RuleDescr rule = parseAndGetFirstRuleDescr( - text ); - + text); - PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + PatternDescr pattern = (PatternDescr) rule.getLhs().getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("Cheese"); assertThat(pattern.getSource().getText()).isEqualToIgnoringWhitespace("from (isFull ? $cheesery : $market)"); assertThat(pattern.isQuery()).isFalse(); - pattern = (PatternDescr) rule.getLhs().getDescrs().get( 1 ); + pattern = (PatternDescr) rule.getLhs().getDescrs().get(1); assertThat(pattern.getObjectType()).isEqualTo("person"); assertThat(pattern.isQuery()).isTrue(); - } @Test @@ -3368,13 +3316,13 @@ void multiValueAnnotationsBackwardCompatibility() { // multiple values with no keys are parsed as a single value final String text = "rule X @ann1( val1, val2 ) @ann2( \"val1\", \"val2\" ) when then end"; RuleDescr rule = parseAndGetFirstRuleDescr( - text ); + text); - AnnotationDescr ann = rule.getAnnotation("ann1" ); + AnnotationDescr ann = rule.getAnnotation("ann1"); assertThat(ann).isNotNull(); assertThat(ann.getValue()).isEqualTo("val1, val2"); - ann = rule.getAnnotation( "ann2" ); + ann = rule.getAnnotation("ann2"); assertThat(ann).isNotNull(); assertThat(ann.getValue()).isEqualTo("\"val1\", \"val2\""); assertThat(ann.getSingleValueAsString()).isEqualTo("\"val1\", \"val2\""); @@ -3384,41 +3332,39 @@ void multiValueAnnotationsBackwardCompatibility() { void positionalsAndNamedConstraints() { final String text = "rule X when Person( \"Mark\", 42; location == \"atlanta\" ) then end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.getDescrs()).hasSize(3); - ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get( 0 ); + ExprConstraintDescr fcd = (ExprConstraintDescr) pattern.getDescrs().get(0); assertThat(fcd.getExpression()).isEqualTo("\"Mark\""); assertThat(fcd.getPosition()).isEqualTo(0); assertThat(fcd.getType()).isEqualTo(ExprConstraintDescr.Type.POSITIONAL); - fcd = (ExprConstraintDescr) pattern.getDescrs().get( 1 ); + fcd = (ExprConstraintDescr) pattern.getDescrs().get(1); assertThat(fcd.getExpression()).isEqualTo("42"); assertThat(fcd.getPosition()).isEqualTo(1); assertThat(fcd.getType()).isEqualTo(ExprConstraintDescr.Type.POSITIONAL); - fcd = (ExprConstraintDescr) pattern.getDescrs().get( 2 ); + fcd = (ExprConstraintDescr) pattern.getDescrs().get(2); assertThat(fcd.getExpression()).isEqualTo("location == \"atlanta\""); assertThat(fcd.getPosition()).isEqualTo(2); assertThat(fcd.getType()).isEqualTo(ExprConstraintDescr.Type.NAMED); - } @Test void unificationBinding() { final String text = "rule X when $p := Person( $name := name, $loc : location ) then end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.getIdentifier()).isEqualTo("$p"); assertThat(pattern.isUnification()).isTrue(); assertThat(pattern.getDescrs()).hasSize(2); - ExprConstraintDescr bindingDescr = (ExprConstraintDescr) pattern.getDescrs().get( 0 ); + ExprConstraintDescr bindingDescr = (ExprConstraintDescr) pattern.getDescrs().get(0); assertThat(bindingDescr.getExpression()).isEqualTo("$name := name"); - bindingDescr = (ExprConstraintDescr) pattern.getDescrs().get( 1 ); + bindingDescr = (ExprConstraintDescr) pattern.getDescrs().get(1); assertThat(bindingDescr.getExpression()).isEqualTo("$loc : location"); - } @Test @@ -3428,19 +3374,19 @@ void bigLiterals() { " bigInteger < 50I, " + " bigDecimal < 50.2B ) then end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.getDescrs()).hasSize(4); - ExprConstraintDescr ecd = (ExprConstraintDescr) pattern.getDescrs().get( 0 ); + ExprConstraintDescr ecd = (ExprConstraintDescr) pattern.getDescrs().get(0); assertThat(ecd.getExpression()).isEqualTo("bigInteger == (10I)"); - ecd = (ExprConstraintDescr) pattern.getDescrs().get( 1 ); + ecd = (ExprConstraintDescr) pattern.getDescrs().get(1); assertThat(ecd.getExpression()).isEqualTo("bigDecimal == (10B)"); - ecd = (ExprConstraintDescr) pattern.getDescrs().get( 2 ); + ecd = (ExprConstraintDescr) pattern.getDescrs().get(2); assertThat(ecd.getExpression()).isEqualTo("bigInteger < 50I"); - ecd = (ExprConstraintDescr) pattern.getDescrs().get( 3 ); + ecd = (ExprConstraintDescr) pattern.getDescrs().get(3); assertThat(ecd.getExpression()).isEqualTo("bigDecimal < 50.2B"); } @@ -3448,13 +3394,13 @@ void bigLiterals() { void bindingComposite() { final String text = "rule X when Person( $name : name == \"Bob\" || $loc : location == \"Montreal\" ) then end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("Person"); assertThat(pattern.isUnification()).isFalse(); // embedded bindings are extracted at compile time - List< ? > constraints = pattern.getDescrs(); + List constraints = pattern.getDescrs(); assertThat(constraints).hasSize(1); assertThat(((ExprConstraintDescr) constraints.get(0)).getExpression()).isEqualTo("$name : name == \"Bob\" || $loc : location == \"Montreal\""); } @@ -3463,13 +3409,13 @@ void bindingComposite() { void bindingCompositeWithMethods() { final String text = "rule X when Person( $name : name.toUpperCase() == \"Bob\" || $loc : location[0].city == \"Montreal\" ) then end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("Person"); assertThat(pattern.isUnification()).isFalse(); // embedded bindings are extracted at compile time - List< ? > constraints = pattern.getDescrs(); + List constraints = pattern.getDescrs(); assertThat(constraints).hasSize(1); assertThat(((ExprConstraintDescr) constraints.get(0)).getExpression()).isEqualTo("$name : name.toUpperCase() == \"Bob\" || $loc : location[0].city == \"Montreal\""); } @@ -3483,12 +3429,11 @@ void pluggableOperators2() { "then\n" + "end"; PatternDescr pattern = (PatternDescr) ((ExistsDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 )).getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0)).getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("TelephoneCall"); - ExprConstraintDescr constr = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get( 0 ); + ExprConstraintDescr constr = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get(0); assertThat(constr.getText()).isEqualTo("this finishes [1m] \"25-May-2011\""); - } @Test @@ -3499,12 +3444,11 @@ void inlineEval() { "then\n" + "end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("Person"); - ExprConstraintDescr constr = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get( 0 ); + ExprConstraintDescr constr = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get(0); assertThat(constr.getText()).isEqualToIgnoringWhitespace("eval( name.startsWith(\"b\") && name.finishesWith(\"b\"))"); - } @Test @@ -3515,12 +3459,11 @@ void infinityLiteral() { "then\n" + "end"; PatternDescr pattern = (PatternDescr) parseAndGetFirstRuleDescr( - text ).getLhs().getDescrs().get( 0 ); + text).getLhs().getDescrs().get(0); assertThat(pattern.getObjectType()).isEqualTo("StockTick"); - ExprConstraintDescr constr = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get( 0 ); + ExprConstraintDescr constr = (ExprConstraintDescr) pattern.getConstraint().getDescrs().get(0); assertThat(constr.getText()).isEqualTo("this after[-*,*] $another"); - } static Stream entryPointIds() { @@ -3539,7 +3482,7 @@ void parse_EntryPointDeclaration(String sourceId, String expectedId) { " @foo( true )\n" + "end"; PackageDescr pkg = parseAndGetPackageDescr( - text ); + text); assertThat(pkg.getName()).isEqualTo("org.drools"); assertThat(pkg.getEntryPointDeclarations()).hasSize(1); @@ -3562,7 +3505,7 @@ void windowDeclaration() { " from entry-point stStream\n" + "end"; PackageDescr pkg = parseAndGetPackageDescr( - text ); + text); assertThat(pkg.getName()).isEqualTo("org.drools"); assertThat(pkg.getWindowDeclarations()).hasSize(1); @@ -3580,7 +3523,7 @@ void windowDeclaration() { assertThat(pd.getSource().getText()).isEqualTo("stStream"); assertThat(pd.getBehaviors()).hasSize(1); - BehaviorDescr bd = pd.getBehaviors().get( 0 ); + BehaviorDescr bd = pd.getBehaviors().get(0); assertThat(bd.getType()).isEqualTo("window"); assertThat(bd.getSubType()).isEqualTo("length"); assertThat(bd.getParameters()).hasSize(2); @@ -3597,7 +3540,7 @@ void windowUsage() { "then\n" + "end\n"; PackageDescr pkg = parseAndGetPackageDescr( - text ); + text); assertThat(pkg.getName()).isEqualTo("org.drools"); assertThat(pkg.getRules()).hasSize(1); @@ -3622,7 +3565,7 @@ void endInRhs() { "then\n" + " System.out.println($s.endsWith(\"xyz\"));\n" + "end\n"; - PackageDescr packageDescr = parseAndGetPackageDescr(text ); + PackageDescr packageDescr = parseAndGetPackageDescr(text); RuleDescr ruleDescr = packageDescr.getRules().get(0); assertThat(ruleDescr.getConsequence().toString()).isEqualToIgnoringWhitespace("System.out.println($s.endsWith(\"xyz\"));"); @@ -3637,7 +3580,7 @@ void endTokenInRhs() { "then\n" + " int end = 10;\n" + "end\n"; - PackageDescr packageDescr = parseAndGetPackageDescr(text ); + PackageDescr packageDescr = parseAndGetPackageDescr(text); RuleDescr ruleDescr = packageDescr.getRules().get(0); assertThat(ruleDescr.getConsequence().toString()).isEqualToIgnoringWhitespace("int end = 10;"); @@ -3652,7 +3595,7 @@ void ruleTokenInRhs() { "then\n" + " int rule = 10;\n" + "end\n"; - PackageDescr packageDescr = parseAndGetPackageDescr(text ); + PackageDescr packageDescr = parseAndGetPackageDescr(text); RuleDescr ruleDescr = packageDescr.getRules().get(0); assertThat(ruleDescr.getConsequence().toString()).isEqualToIgnoringWhitespace("int rule = 10;"); @@ -3666,7 +3609,7 @@ void semicolonEnd() { " $s : String()\n" + "then\n" + " delete($s);end\n"; // no space after semicolon - PackageDescr packageDescr = parseAndGetPackageDescr(text ); + PackageDescr packageDescr = parseAndGetPackageDescr(text); RuleDescr ruleDescr = packageDescr.getRules().get(0); assertThat(ruleDescr.getConsequence().toString()).isEqualToIgnoringWhitespace("delete($s);"); @@ -3680,7 +3623,7 @@ void braceEnd() { " $p : Person()\n" + "then\n" + " modify($p) { setAge(2) }end\n"; // no space after right brace - PackageDescr packageDescr = parseAndGetPackageDescr(text ); + PackageDescr packageDescr = parseAndGetPackageDescr(text); RuleDescr ruleDescr = packageDescr.getRules().get(0); assertThat(ruleDescr.getConsequence().toString()).isEqualToIgnoringWhitespace("modify($p) { setAge(2) }"); @@ -4051,14 +3994,13 @@ void functionWithStringLiteral() { final String text = "function String star(String s) { return \"*\"; }"; PackageDescr packageDescr = parseAndGetPackageDescr(text); - - FunctionDescr function = packageDescr.getFunctions().get( 0 ); + FunctionDescr function = packageDescr.getFunctions().get(0); assertThat(function.getName()).isEqualTo("star"); assertThat(function.getReturnType()).isEqualTo("String"); assertThat(function.getParameterTypes().get(0)).isEqualTo("String"); assertThat(function.getParameterNames().get(0)).isEqualTo("s"); - assertThat(function.getBody()).isEqualToIgnoringWhitespace( "return \"*\";"); + assertThat(function.getBody()).isEqualToIgnoringWhitespace("return \"*\";"); } @Test @@ -4066,14 +4008,13 @@ void functionWithStringLiteralAddition() { final String text = "function String addStar(String s) { return s + \"*\"; }"; PackageDescr packageDescr = parseAndGetPackageDescr(text); - - FunctionDescr function = packageDescr.getFunctions().get( 0 ); + FunctionDescr function = packageDescr.getFunctions().get(0); assertThat(function.getName()).isEqualTo("addStar"); assertThat(function.getReturnType()).isEqualTo("String"); assertThat(function.getParameterTypes().get(0)).isEqualTo("String"); assertThat(function.getParameterNames().get(0)).isEqualTo("s"); - assertThat(function.getBody()).isEqualToIgnoringWhitespace( "return s + \"*\";"); + assertThat(function.getBody()).isEqualToIgnoringWhitespace("return s + \"*\";"); } @Test @@ -4084,14 +4025,13 @@ void functionWithMultipleBlockStatements() { "}"; PackageDescr packageDescr = parseAndGetPackageDescr(text); - - FunctionDescr function = packageDescr.getFunctions().get( 0 ); + FunctionDescr function = packageDescr.getFunctions().get(0); assertThat(function.getName()).isEqualTo("star"); assertThat(function.getReturnType()).isEqualTo("String"); assertThat(function.getParameterTypes().get(0)).isEqualTo("String"); assertThat(function.getParameterNames().get(0)).isEqualTo("s"); - assertThat(function.getBody()).isEqualToIgnoringWhitespace( "String result = s + \"*\"; return result;"); + assertThat(function.getBody()).isEqualToIgnoringWhitespace("String result = s + \"*\"; return result;"); } @Test @@ -4104,7 +4044,6 @@ void lhsPatternAnnotation() { "end\n"; PackageDescr packageDescr = parseAndGetPackageDescr(text); - RuleDescr ruleDescr = packageDescr.getRules().get(0); PatternDescr patternDescr = (PatternDescr) ruleDescr.getLhs().getDescrs().get(0); AnnotationDescr annotationDescr = patternDescr.getAnnotations().iterator().next(); @@ -4336,12 +4275,12 @@ void ooPathLhsPattern() { assertThat(ruleDescr.getLhs().getDescrs().get(0)).isInstanceOfSatisfying(NotDescr.class, notDescr -> { assertThat(notDescr.getDescrs()).hasSize(1); assertThat(notDescr.getDescrs().get(0)).isInstanceOfSatisfying(PatternDescr.class, patternDescr -> { - assertThat(patternDescr.getConstraint().getDescrs()).hasSize(1); - assertThat(patternDescr.getConstraint().getDescrs().get(0)).isInstanceOfSatisfying(ExprConstraintDescr.class, exprConstraintDescr -> { - assertThat(exprConstraintDescr.getExpression()).isEqualTo("/strings [ this == \"It Does Work\" ]"); - assertThat(exprConstraintDescr.getType()).isEqualTo(ExprConstraintDescr.Type.NAMED); - assertThat(exprConstraintDescr.getPosition()).isEqualTo(0); - }); + assertThat(patternDescr.getConstraint().getDescrs()).hasSize(1); + assertThat(patternDescr.getConstraint().getDescrs().get(0)).isInstanceOfSatisfying(ExprConstraintDescr.class, exprConstraintDescr -> { + assertThat(exprConstraintDescr.getExpression()).isEqualTo("/strings [ this == \"It Does Work\" ]"); + assertThat(exprConstraintDescr.getType()).isEqualTo(ExprConstraintDescr.Type.NAMED); + assertThat(exprConstraintDescr.getPosition()).isEqualTo(0); + }); }); }); } @@ -4496,15 +4435,15 @@ void pluggableEvaluator() { void namedConsequenceDo() { final String text = "rule R when\n" + - " $r : Result()\n" + - " $p1 : Person(name == \"Mark\")\n" + - " do[FoundMark]\n" + - " $p2 : Person(name != \"Mark\", age > $p1.age)\n" + - "then\n" + - " $r.addValue($p2.getName() + \" is older than \" + $p1.getName());\n" + - "then[FoundMark]\n" + - " $r.addValue(\"Found \" + $p1.getName());\n" + - "end"; + " $r : Result()\n" + + " $p1 : Person(name == \"Mark\")\n" + + " do[FoundMark]\n" + + " $p2 : Person(name != \"Mark\", age > $p1.age)\n" + + "then\n" + + " $r.addValue($p2.getName() + \" is older than \" + $p1.getName());\n" + + "then[FoundMark]\n" + + " $r.addValue(\"Found \" + $p1.getName());\n" + + "end"; PackageDescr packageDescr = parseAndGetPackageDescr(text); RuleDescr ruleDescr = packageDescr.getRules().get(0); NamedConsequenceDescr namedConsequenceDescr = (NamedConsequenceDescr) ruleDescr.getLhs().getDescrs().get(2); @@ -4680,10 +4619,10 @@ void namedConsequenceOrWithBindVariables() { OrDescr orDescr = (OrDescr) ruleDescr.getLhs().getDescrs().get(1); PatternDescr patternDescr1 = (PatternDescr) orDescr.getDescrs().get(0); assertThat(patternDescr1.getIdentifier()).isEqualTo("$p1"); - assertThat(((ExprConstraintDescr)patternDescr1.getConstraint().getDescrs().get(0)).getExpression()).isEqualTo("name == \"Mark\""); + assertThat(((ExprConstraintDescr) patternDescr1.getConstraint().getDescrs().get(0)).getExpression()).isEqualTo("name == \"Mark\""); PatternDescr patternDescr2 = (PatternDescr) orDescr.getDescrs().get(1); assertThat(patternDescr2.getIdentifier()).isEqualTo("$p1"); - assertThat(((ExprConstraintDescr)patternDescr2.getConstraint().getDescrs().get(0)).getExpression()).isEqualTo("name == \"Mario\""); + assertThat(((ExprConstraintDescr) patternDescr2.getConstraint().getDescrs().get(0)).getExpression()).isEqualTo("name == \"Mario\""); NamedConsequenceDescr namedConsequenceDescr = (NamedConsequenceDescr) ruleDescr.getLhs().getDescrs().get(2); assertThat(namedConsequenceDescr.getName()).isEqualTo("FoundMarkOrMario"); @@ -5277,12 +5216,12 @@ void accumulateWithEmptyActionAndReverse() { "then end"; RuleDescr rule = parseAndGetFirstRuleDescr(drl); - final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get( 0 ); + final PatternDescr outPattern = (PatternDescr) rule.getLhs().getDescrs().get(0); final AccumulateDescr accum = (AccumulateDescr) outPattern.getSource(); - assertThat(accum.getInitCode()).isEqualTo( "double total = 0;"); + assertThat(accum.getInitCode()).isEqualTo("double total = 0;"); assertThat(accum.getActionCode()).isEmpty(); assertThat(accum.getReverseCode()).isEmpty(); - assertThat(accum.getResultCode()).isEqualTo( "new Double( total )"); + assertThat(accum.getResultCode()).isEqualTo("new Double( total )"); assertThat(accum.isExternalFunction()).isFalse();