updated tests

Innocent · Innocent · commit bd7710154dc2 · 2026-01-25T22:01:33.000-07:00
diff --git a/src/iceberg/test/expression_json_test.cc b/src/iceberg/test/expression_json_test.cc
@@ -65,9 +65,14 @@ TEST_F(ExpressionJsonTest, FalseExpression) {
   EXPECT_EQ(result.value()->op(), Expression::Operation::kFalse);
 }
 
-TEST_F(ExpressionJsonTest, OpToString) {
+TEST_F(ExpressionJsonTest, OperationTypeTests) {
   EXPECT_EQ(OperationTypeFromString("true"), Expression::Operation::kTrue);
   EXPECT_EQ("true", ToStringOperationType(Expression::Operation::kTrue));
+  EXPECT_TRUE(IsSetOperation(Expression::Operation::kIn));
+  EXPECT_FALSE(IsSetOperation(Expression::Operation::kTrue));
+
+  EXPECT_TRUE(IsUnaryOperation(Expression::Operation::kIsNull));
+  EXPECT_FALSE(IsUnaryOperation(Expression::Operation::kTrue));
 }
 
 }  // namespace iceberg

Original file line number	Diff line number	Diff line change
`@@ -65,9 +65,14 @@ TEST_F(ExpressionJsonTest, FalseExpression) {`
`65`	`65`	`EXPECT_EQ(result.value()->op(), Expression::Operation::kFalse);`
`66`	`66`	`}`
`67`	`67`
`68`		`-TEST_F(ExpressionJsonTest, OpToString) {`
	`68`	`+TEST_F(ExpressionJsonTest, OperationTypeTests) {`
`69`	`69`	`EXPECT_EQ(OperationTypeFromString("true"), Expression::Operation::kTrue);`
`70`	`70`	`EXPECT_EQ("true", ToStringOperationType(Expression::Operation::kTrue));`
	`71`	`+ EXPECT_TRUE(IsSetOperation(Expression::Operation::kIn));`
	`72`	`+ EXPECT_FALSE(IsSetOperation(Expression::Operation::kTrue));`
	`73`	`+`
	`74`	`+ EXPECT_TRUE(IsUnaryOperation(Expression::Operation::kIsNull));`
	`75`	`+ EXPECT_FALSE(IsUnaryOperation(Expression::Operation::kTrue));`
`71`	`76`	`}`
`72`	`77`
`73`	`78`	`} // namespace iceberg`