Fix RowExpressionTest to be forward compatible with variant.toJson ch…

…anges.
prestodb · Oct 11, 2023 · 627da72 · 627da72
1 parent b8c2241
commit 627da72
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/presto-native-execution/presto_cpp/main/types/tests/RowExpressionTest.cpp b/presto-native-execution/presto_cpp/main/types/tests/RowExpressionTest.cpp
@@ -479,7 +479,7 @@ TEST_F(RowExpressionTest, call) {
     {
       auto cexpr = std::static_pointer_cast<const ConstantTypedExpr>(iexpr[1]);
       ASSERT_EQ(cexpr->type()->toString(), "VARCHAR");
-      ASSERT_EQ(cexpr->value().toJson(), "\"foo\"");
+      ASSERT_EQ(cexpr->value().toJson(cexpr->type()), "\"foo\"");
     }
   }
 }
@@ -585,7 +585,7 @@ TEST_F(RowExpressionTest, special) {
       auto cexpr = std::static_pointer_cast<const ConstantTypedExpr>(
           arg0expr->inputs()[1]);
       ASSERT_EQ(cexpr->type()->toString(), "BIGINT");
-      ASSERT_EQ(cexpr->value().toJson(), "10");
+      ASSERT_EQ(cexpr->value().toJson(cexpr->type()), "10");
     }
   }
 
@@ -605,7 +605,7 @@ TEST_F(RowExpressionTest, special) {
       auto cexpr = std::static_pointer_cast<const ConstantTypedExpr>(
           arg1expr->inputs()[1]);
       ASSERT_EQ(cexpr->type()->toString(), "VARCHAR");
-      ASSERT_EQ(cexpr->value().toJson(), "\"foo\"");
+      ASSERT_EQ(cexpr->value().toJson(cexpr->type()), "\"foo\"");
     }
   }
 }