fixup! Move analyzer out of optimizer

datafusion/sql/src/lib.rs

@@ -44,6 +44,7 @@ mod relation;
 mod select;
 mod set_expr;
 mod statement;
+#[cfg(test)]
 mod test;
 #[cfg(feature = "unparser")]
 pub mod unparser;

datafusion/sql/src/test.rs

@@ -22,27 +22,25 @@ use datafusion_expr::{table_scan, LogicalPlan};
 use std::sync::Arc;
 
 /// some tests share a common table
-pub(crate) fn test_table_scan() -> datafusion_common::Result<LogicalPlan> {
+pub fn test_table_scan() -> datafusion_common::Result<LogicalPlan> {
     test_table_scan_with_name("test")
 }
 
 /// some tests share a common table with different names
-pub(crate) fn test_table_scan_with_name(
-    name: &str,
-) -> datafusion_common::Result<LogicalPlan> {
+pub fn test_table_scan_with_name(name: &str) -> datafusion_common::Result<LogicalPlan> {
     let schema = Schema::new(test_table_scan_fields());
     table_scan(Some(name), &schema, None)?.build()
 }
 
-pub(crate) fn test_table_scan_fields() -> Vec<Field> {
+pub fn test_table_scan_fields() -> Vec<Field> {
     vec![
         Field::new("a", DataType::UInt32, false),
         Field::new("b", DataType::UInt32, false),
         Field::new("c", DataType::UInt32, false),
     ]
 }
 
-pub(crate) fn assert_analyzed_plan_eq(
+pub fn assert_analyzed_plan_eq(
     rule: Arc<dyn AnalyzerRule + Send + Sync>,
     plan: LogicalPlan,
     expected: &str,
@@ -53,7 +51,7 @@ pub(crate) fn assert_analyzed_plan_eq(
     Ok(())
 }
 
-pub(crate) fn assert_analyzed_plan_with_config_eq(
+pub fn assert_analyzed_plan_with_config_eq(
     options: ConfigOptions,
     rule: Arc<dyn AnalyzerRule + Send + Sync>,
     plan: LogicalPlan,
@@ -67,7 +65,7 @@ pub(crate) fn assert_analyzed_plan_with_config_eq(
     Ok(())
 }
 
-pub(crate) fn assert_analyzed_plan_eq_display_indent(
+pub fn assert_analyzed_plan_eq_display_indent(
     rule: Arc<dyn AnalyzerRule + Send + Sync>,
     plan: LogicalPlan,
     expected: &str,