datafusion icon indicating copy to clipboard operation
datafusion copied to clipboard

Consolidate tests for unparser / plan to sql to make them easier to find

Open alamb opened this issue 1 year ago • 0 comments

          Since this is testing `plan_to_sql` it might make more sense to put it with the rest of the tests for plan_to_sql:  https://github.com/apache/datafusion/blob/main/datafusion/sql/tests/sql_integration.rs#L4669

However, the fact that those tests don't have "plan_to_sql" in their name is confusing too.

Maybe we could consolidate the tests into something like datafusion/sql/tests/sql_integration/plan_to_sql.rs 🤔 (as a follow on PR)

Originally posted by @alamb in https://github.com/apache/datafusion/pull/10612#discussion_r1609707420

alamb avatar May 23 '24 11:05 alamb