From 74bf65e461fda68116d286dcdf0296ce8e48a09b Mon Sep 17 00:00:00 2001 From: Hongze Zhang Date: Thu, 21 Mar 2024 13:38:19 +0800 Subject: [PATCH] fixup --- .../glutenproject/cbo/CboOperationSuite.scala | 22 ++++++++++--------- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/gluten-cbo/common/src/test/scala/io/glutenproject/cbo/CboOperationSuite.scala b/gluten-cbo/common/src/test/scala/io/glutenproject/cbo/CboOperationSuite.scala index 960fba62d7c0f..1a92e3492708e 100644 --- a/gluten-cbo/common/src/test/scala/io/glutenproject/cbo/CboOperationSuite.scala +++ b/gluten-cbo/common/src/test/scala/io/glutenproject/cbo/CboOperationSuite.scala @@ -422,18 +422,20 @@ object CboOperationSuite extends CboSuiteBase { private object PlanModelWithStats { implicit class PlanModelWithStatsImplicits[T <: AnyRef](model: PlanModelWithStats[T]) { def assertPlanOpsLte(bounds: (Int, Int, Int, Int)): Unit = { + val actual = ( + model.childrenOfCount, + model.withNewChildrenCount, + model.hashCodeCount, + model.equalsCount) assert( - List( - model.childrenOfCount, - model.withNewChildrenCount, - model.hashCodeCount, - model.equalsCount).zip(List(bounds._1, bounds._2, bounds._3, bounds._4)).forall { - case (count, bound) => - count <= bound - }, + List(actual._1, actual._2, actual._3, actual._4) + .zip(List(bounds._1, bounds._2, bounds._3, bounds._4)) + .forall { + case (count, bound) => + count <= bound + }, s"Assertion failed. The expected bounds: $bounds, actual: " + - s"${(model.childrenOfCount, model.withNewChildrenCount, model.hashCodeCount, - model.equalsCount)}" + s"$actual" ) } }