prepare for review

Author: andygrove

spark/src/test/scala/org/apache/comet/CometExpressionSuite.scala

@@ -1146,10 +1146,7 @@ class CometExpressionSuite extends CometTestBase with AdaptiveSparkPlanHelper {
     }
   }
 
-  // TODO: test currently fails with
-  // Unsupported Data Type: Spark `expm1` function expects Float64, got Utf8
   test("expm1") {
-    // TODO test with null inputs
     val testValues = Seq(
       -1,
       0,

spark/src/test/scala/org/apache/spark/sql/CometTestBase.scala

@@ -111,8 +111,6 @@ abstract class CometTestBase
       actualAnswer: Row,
       expectedAnswer: Row,
       absTol: Double): Unit = {
-    // scalastyle:off println
-    println(s"actual=$actualAnswer, expected=$expectedAnswer")
     require(
       actualAnswer.length == expectedAnswer.length,
       s"actual answer length ${actualAnswer.length} != " +
@@ -122,8 +120,7 @@ abstract class CometTestBase
     actualAnswer.toSeq.zip(expectedAnswer.toSeq).foreach {
       case (actual: Double, expected: Double) =>
         if (actual.isInfinity || expected.isInfinity) {
-          assert(actual.isInfinity == expected.isInfinity,
-            s"actual answer $actual != $expected")
+          assert(actual.isInfinity == expected.isInfinity, s"actual answer $actual != $expected")
         } else if (!actual.isNaN && !expected.isNaN) {
           assert(
             math.abs(actual - expected) < absTol,