Remove duplicate tests from test_const_evaluator_scalar_functions #1727
Description
Describe the bug
I find that we have 2 same tests:
// check that non foldable arguments are folded
// to_timestamp(a) --> to_timestamp(a) [no rewrite possible]
let expr = Expr::ScalarFunction {
args: vec![col("a")],
fun: BuiltinScalarFunction::ToTimestamp,
};
test_evaluate(expr.clone(), expr);
// check that non foldable arguments are folded
// to_timestamp(a) --> to_timestamp(a) [no rewrite possible]
let expr = Expr::ScalarFunction {
args: vec![col("a")],
fun: BuiltinScalarFunction::ToTimestamp,
};
test_evaluate(expr.clone(), expr);To Reproduce
Expected behavior
Remove one of the test or add some comment about why we need 2 same tests.
Additional context