You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by da...@apache.org on 2022/07/26 12:33:37 UTC
[doris] branch master updated: [enhancement](nereids) Fix the wrong order of arguments of Assert.assertEquals (#11207)
This is an automated email from the ASF dual-hosted git repository.
dataroaring pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 8a6c9d870e [enhancement](nereids) Fix the wrong order of arguments of Assert.assertEquals (#11207)
8a6c9d870e is described below
commit 8a6c9d870ed3879b3221edd28a00692ec3bca5fb
Author: Adonis Ling <ad...@gmail.com>
AuthorDate: Tue Jul 26 20:33:32 2022 +0800
[enhancement](nereids) Fix the wrong order of arguments of Assert.assertEquals (#11207)
---
.../doris/nereids/jobs/RewriteTopDownJobTest.java | 16 ++++-----
.../pattern/GroupExpressionMatchingTest.java | 6 ++--
.../rules/exploration/join/JoinCommuteTest.java | 4 +--
.../rewrite/logical/PushDownPredicateTest.java | 20 +++++------
.../doris/nereids/util/ExpressionUtilsTest.java | 40 +++++++++++-----------
5 files changed, 43 insertions(+), 43 deletions(-)
diff --git a/fe/fe-core/src/test/java/org/apache/doris/nereids/jobs/RewriteTopDownJobTest.java b/fe/fe-core/src/test/java/org/apache/doris/nereids/jobs/RewriteTopDownJobTest.java
index e28a508084..3d13777cc9 100644
--- a/fe/fe-core/src/test/java/org/apache/doris/nereids/jobs/RewriteTopDownJobTest.java
+++ b/fe/fe-core/src/test/java/org/apache/doris/nereids/jobs/RewriteTopDownJobTest.java
@@ -81,19 +81,19 @@ public class RewriteTopDownJobTest {
Assertions.assertEquals(1, rootGroup.getLogicalExpressions().size());
GroupExpression rootGroupExpression = rootGroup.getLogicalExpression();
List<Slot> output = rootGroup.getLogicalProperties().getOutput();
- Assertions.assertEquals(output.size(), 1);
- Assertions.assertEquals(output.get(0).getName(), "name");
- Assertions.assertEquals(output.get(0).getDataType(), StringType.INSTANCE);
+ Assertions.assertEquals(1, output.size());
+ Assertions.assertEquals("name", output.get(0).getName());
+ Assertions.assertEquals(StringType.INSTANCE, output.get(0).getDataType());
Assertions.assertEquals(1, rootGroupExpression.children().size());
Assertions.assertEquals(PlanType.LOGICAL_PROJECT, rootGroupExpression.getPlan().getType());
Group leafGroup = rootGroupExpression.child(0);
output = leafGroup.getLogicalProperties().getOutput();
- Assertions.assertEquals(output.size(), 2);
- Assertions.assertEquals(output.get(0).getName(), "id");
- Assertions.assertEquals(output.get(0).getDataType(), IntegerType.INSTANCE);
- Assertions.assertEquals(output.get(1).getName(), "name");
- Assertions.assertEquals(output.get(1).getDataType(), StringType.INSTANCE);
+ Assertions.assertEquals(2, output.size());
+ Assertions.assertEquals("id", output.get(0).getName());
+ Assertions.assertEquals(IntegerType.INSTANCE, output.get(0).getDataType());
+ Assertions.assertEquals("name", output.get(1).getName());
+ Assertions.assertEquals(StringType.INSTANCE, output.get(1).getDataType());
Assertions.assertEquals(1, leafGroup.getLogicalExpressions().size());
GroupExpression leafGroupExpression = leafGroup.getLogicalExpression();
Assertions.assertEquals(PlanType.LOGICAL_BOUND_RELATION, leafGroupExpression.getPlan().getType());
diff --git a/fe/fe-core/src/test/java/org/apache/doris/nereids/pattern/GroupExpressionMatchingTest.java b/fe/fe-core/src/test/java/org/apache/doris/nereids/pattern/GroupExpressionMatchingTest.java
index 7c2a2b8b34..d4f9c333b2 100644
--- a/fe/fe-core/src/test/java/org/apache/doris/nereids/pattern/GroupExpressionMatchingTest.java
+++ b/fe/fe-core/src/test/java/org/apache/doris/nereids/pattern/GroupExpressionMatchingTest.java
@@ -265,7 +265,7 @@ public class GroupExpressionMatchingTest {
case LEFT_OUTER_JOIN:
Assertions.assertTrue(context.parent instanceof LogicalJoin);
LogicalJoin parent = (LogicalJoin) context.parent;
- Assertions.assertEquals(parent.getJoinType(), JoinType.INNER_JOIN);
+ Assertions.assertEquals(JoinType.INNER_JOIN, parent.getJoinType());
break;
default:
notExpectedPlan(join, context);
@@ -287,13 +287,13 @@ public class GroupExpressionMatchingTest {
case "b": {
Assertions.assertTrue(context.parent instanceof LogicalJoin);
LogicalJoin parent = (LogicalJoin) context.parent;
- Assertions.assertEquals(parent.getJoinType(), JoinType.LEFT_OUTER_JOIN);
+ Assertions.assertEquals(JoinType.LEFT_OUTER_JOIN, parent.getJoinType());
break;
}
case "c": {
Assertions.assertTrue(context.parent instanceof LogicalJoin);
LogicalJoin parent = (LogicalJoin) context.parent;
- Assertions.assertEquals(parent.getJoinType(), JoinType.INNER_JOIN);
+ Assertions.assertEquals(JoinType.INNER_JOIN, parent.getJoinType());
break;
}
default:
diff --git a/fe/fe-core/src/test/java/org/apache/doris/nereids/rules/exploration/join/JoinCommuteTest.java b/fe/fe-core/src/test/java/org/apache/doris/nereids/rules/exploration/join/JoinCommuteTest.java
index 8d347335f6..34daec8906 100644
--- a/fe/fe-core/src/test/java/org/apache/doris/nereids/rules/exploration/join/JoinCommuteTest.java
+++ b/fe/fe-core/src/test/java/org/apache/doris/nereids/rules/exploration/join/JoinCommuteTest.java
@@ -63,8 +63,8 @@ public class JoinCommuteTest {
Assert.assertEquals(1, transform.size());
Plan newJoin = transform.get(0);
- Assert.assertEquals(newJoin.child(1), join.child(0));
- Assert.assertEquals(newJoin.child(0), join.child(1));
+ Assert.assertEquals(join.child(0), newJoin.child(1));
+ Assert.assertEquals(join.child(1), newJoin.child(0));
}
}
diff --git a/fe/fe-core/src/test/java/org/apache/doris/nereids/rules/rewrite/logical/PushDownPredicateTest.java b/fe/fe-core/src/test/java/org/apache/doris/nereids/rules/rewrite/logical/PushDownPredicateTest.java
index 7a28782763..7a81d1f9a2 100644
--- a/fe/fe-core/src/test/java/org/apache/doris/nereids/rules/rewrite/logical/PushDownPredicateTest.java
+++ b/fe/fe-core/src/test/java/org/apache/doris/nereids/rules/rewrite/logical/PushDownPredicateTest.java
@@ -135,9 +135,9 @@ public class PushDownPredicateTest {
LogicalFilter filter1 = (LogicalFilter) op2;
LogicalFilter filter2 = (LogicalFilter) op3;
- Assertions.assertEquals(join1.getCondition().get(), onCondition1);
- Assertions.assertEquals(filter1.getPredicates(), ExpressionUtils.and(onCondition2, whereCondition1));
- Assertions.assertEquals(filter2.getPredicates(), ExpressionUtils.and(onCondition3, whereCondition2));
+ Assertions.assertEquals(onCondition1, join1.getCondition().get());
+ Assertions.assertEquals(ExpressionUtils.and(onCondition2, whereCondition1), filter1.getPredicates());
+ Assertions.assertEquals(ExpressionUtils.and(onCondition3, whereCondition2), filter2.getPredicates());
}
@Test
@@ -176,9 +176,9 @@ public class PushDownPredicateTest {
LogicalJoin join1 = (LogicalJoin) op1;
LogicalFilter filter1 = (LogicalFilter) op2;
LogicalFilter filter2 = (LogicalFilter) op3;
- Assertions.assertEquals(join1.getCondition().get(), whereCondition1);
- Assertions.assertEquals(filter1.getPredicates(), whereCondition2);
- Assertions.assertEquals(filter2.getPredicates(), whereCondition3);
+ Assertions.assertEquals(whereCondition1, join1.getCondition().get());
+ Assertions.assertEquals(whereCondition2, filter1.getPredicates());
+ Assertions.assertEquals(whereCondition3, filter2.getPredicates());
}
@Test
@@ -235,10 +235,10 @@ public class PushDownPredicateTest {
Assertions.assertTrue(op1 instanceof LogicalFilter);
Assertions.assertTrue(op2 instanceof LogicalFilter);
- Assertions.assertEquals(((LogicalJoin) join2).getCondition().get(), whereCondition2);
- Assertions.assertEquals(((LogicalJoin) join3).getCondition().get(), whereCondition1);
- Assertions.assertEquals(((LogicalFilter) op1).getPredicates().toSql(), whereCondition3result.toSql());
- Assertions.assertEquals(((LogicalFilter) op2).getPredicates(), whereCondition4);
+ Assertions.assertEquals(whereCondition2, ((LogicalJoin) join2).getCondition().get());
+ Assertions.assertEquals(whereCondition1, ((LogicalJoin) join3).getCondition().get());
+ Assertions.assertEquals(whereCondition3result.toSql(), ((LogicalFilter) op1).getPredicates().toSql());
+ Assertions.assertEquals(whereCondition4, ((LogicalFilter) op2).getPredicates());
}
private Memo rewrite(Plan plan) {
diff --git a/fe/fe-core/src/test/java/org/apache/doris/nereids/util/ExpressionUtilsTest.java b/fe/fe-core/src/test/java/org/apache/doris/nereids/util/ExpressionUtilsTest.java
index 4fe1b3e2c0..ee162a211e 100644
--- a/fe/fe-core/src/test/java/org/apache/doris/nereids/util/ExpressionUtilsTest.java
+++ b/fe/fe-core/src/test/java/org/apache/doris/nereids/util/ExpressionUtilsTest.java
@@ -39,8 +39,8 @@ public class ExpressionUtilsTest {
expr = PARSER.parseExpression("a");
expressions = ExpressionUtils.extractConjunctive(expr);
- Assertions.assertEquals(expressions.size(), 1);
- Assertions.assertEquals(expressions.get(0), expr);
+ Assertions.assertEquals(1, expressions.size());
+ Assertions.assertEquals(expr, expressions.get(0));
expr = PARSER.parseExpression("a and b and c");
Expression a = PARSER.parseExpression("a");
@@ -48,20 +48,20 @@ public class ExpressionUtilsTest {
Expression c = PARSER.parseExpression("c");
expressions = ExpressionUtils.extractConjunctive(expr);
- Assertions.assertEquals(expressions.size(), 3);
- Assertions.assertEquals(expressions.get(0), a);
- Assertions.assertEquals(expressions.get(1), b);
- Assertions.assertEquals(expressions.get(2), c);
+ Assertions.assertEquals(3, expressions.size());
+ Assertions.assertEquals(a, expressions.get(0));
+ Assertions.assertEquals(b, expressions.get(1));
+ Assertions.assertEquals(c, expressions.get(2));
expr = PARSER.parseExpression("(a or b) and c and (e or f)");
expressions = ExpressionUtils.extractConjunctive(expr);
Expression aOrb = PARSER.parseExpression("a or b");
Expression eOrf = PARSER.parseExpression("e or f");
- Assertions.assertEquals(expressions.size(), 3);
- Assertions.assertEquals(expressions.get(0), aOrb);
- Assertions.assertEquals(expressions.get(1), c);
- Assertions.assertEquals(expressions.get(2), eOrf);
+ Assertions.assertEquals(3, expressions.size());
+ Assertions.assertEquals(aOrb, expressions.get(0));
+ Assertions.assertEquals(c, expressions.get(1));
+ Assertions.assertEquals(eOrf, expressions.get(2));
}
@Test
@@ -71,8 +71,8 @@ public class ExpressionUtilsTest {
expr = PARSER.parseExpression("a");
expressions = ExpressionUtils.extractDisjunctive(expr);
- Assertions.assertEquals(expressions.size(), 1);
- Assertions.assertEquals(expressions.get(0), expr);
+ Assertions.assertEquals(1, expressions.size());
+ Assertions.assertEquals(expr, expressions.get(0));
expr = PARSER.parseExpression("a or b or c");
Expression a = PARSER.parseExpression("a");
@@ -80,18 +80,18 @@ public class ExpressionUtilsTest {
Expression c = PARSER.parseExpression("c");
expressions = ExpressionUtils.extractDisjunctive(expr);
- Assertions.assertEquals(expressions.size(), 3);
- Assertions.assertEquals(expressions.get(0), a);
- Assertions.assertEquals(expressions.get(1), b);
- Assertions.assertEquals(expressions.get(2), c);
+ Assertions.assertEquals(3, expressions.size());
+ Assertions.assertEquals(a, expressions.get(0));
+ Assertions.assertEquals(b, expressions.get(1));
+ Assertions.assertEquals(c, expressions.get(2));
expr = PARSER.parseExpression("(a and b) or c or (e and f)");
expressions = ExpressionUtils.extractDisjunctive(expr);
Expression aAndb = PARSER.parseExpression("a and b");
Expression eAndf = PARSER.parseExpression("e and f");
- Assertions.assertEquals(expressions.size(), 3);
- Assertions.assertEquals(expressions.get(0), aAndb);
- Assertions.assertEquals(expressions.get(1), c);
- Assertions.assertEquals(expressions.get(2), eAndf);
+ Assertions.assertEquals(3, expressions.size());
+ Assertions.assertEquals(aAndb, expressions.get(0));
+ Assertions.assertEquals(c, expressions.get(1));
+ Assertions.assertEquals(eAndf, expressions.get(2));
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org