chore: Remove duplicate imports in test code (#21061)

## Which issue does this PR close?

N/A

## Rationale for this change

`clippy` doesn't warn about `use` statements that re-import an item that
is already available via a glob import (`use super::*`), but in many
cases the `use` statement is redundant and can be safely removed. In
particular, there was a lot of test code that did both `use super::*`
and also imported items that were imported by the module under test; the
test-specific import can be safely removed.

## What changes are included in this PR?

* Mechanical removal of duplicate imports between test code and code
under test

## Are these changes tested?

Yes, via `cargo check` and `cargo clippy`

## Are there any user-facing changes?

No.

Author: neilconway

datafusion/catalog-listing/src/helpers.rs

@@ -462,7 +462,7 @@ mod tests {
     use std::ops::Not;
 
     use super::*;
-    use datafusion_expr::{Expr, case, col, lit};
+    use datafusion_expr::{case, col};
 
     #[test]
     fn test_split_files() {

datafusion/catalog-listing/src/table.rs

@@ -920,7 +920,6 @@ mod tests {
     use arrow::compute::SortOptions;
     use datafusion_physical_expr::expressions::Column;
     use datafusion_physical_expr_common::sort_expr::PhysicalSortExpr;
-    use std::sync::Arc;
 
     /// Helper to create a PhysicalSortExpr
     fn sort_expr(

datafusion/common/src/scalar/mod.rs

@@ -5336,7 +5336,6 @@ impl ScalarType<i32> for Date32Type {
 
 #[cfg(test)]
 mod tests {
-    use std::sync::Arc;
 
     use super::*;
     use crate::cast::{as_list_array, as_map_array, as_struct_array};
@@ -5354,7 +5353,6 @@ mod tests {
     };
     use arrow::error::ArrowError;
     use arrow::util::pretty::pretty_format_columns;
-    use chrono::NaiveDate;
     use insta::assert_snapshot;
     use rand::Rng;
 

datafusion/common/src/types/logical.rs

@@ -143,7 +143,7 @@ mod tests {
         LogicalField, LogicalFields, logical_boolean, logical_date, logical_float32,
         logical_float64, logical_int32, logical_int64, logical_null, logical_string,
     };
-    use arrow::datatypes::{DataType, Field, Fields};
+    use arrow::datatypes::{Field, Fields};
     use insta::assert_snapshot;
 
     #[test]

datafusion/common/src/types/native.rs

@@ -565,7 +565,6 @@ impl NativeType {
 mod tests {
     use super::*;
     use crate::types::LogicalField;
-    use arrow::datatypes::Field;
     use insta::assert_snapshot;
 
     #[test]

datafusion/core/src/dataframe/parquet.rs

@@ -107,7 +107,6 @@ impl DataFrame {
 #[cfg(test)]
 mod tests {
     use std::collections::HashMap;
-    use std::sync::Arc;
 
     use super::super::Result;
     use super::*;

datafusion/core/src/datasource/listing_table_factory.rs

@@ -557,17 +557,13 @@ mod tests {
 
     #[tokio::test]
     async fn test_create_with_invalid_session() {
-        use async_trait::async_trait;
-        use datafusion_catalog::Session;
-        use datafusion_common::Result;
         use datafusion_common::config::TableOptions;
         use datafusion_execution::TaskContext;
         use datafusion_execution::config::SessionConfig;
         use datafusion_physical_expr::PhysicalExpr;
         use datafusion_physical_plan::ExecutionPlan;
         use std::any::Any;
         use std::collections::HashMap;
-        use std::sync::Arc;
 
         // A mock Session that is NOT SessionState
         #[derive(Debug)]

datafusion/datasource-arrow/src/file_format.rs

@@ -557,7 +557,7 @@ mod tests {
     use datafusion_expr::execution_props::ExecutionProps;
     use datafusion_expr::{AggregateUDF, Expr, LogicalPlan, ScalarUDF, WindowUDF};
     use datafusion_physical_expr_common::physical_expr::PhysicalExpr;
-    use object_store::{chunked::ChunkedStore, memory::InMemory, path::Path};
+    use object_store::{chunked::ChunkedStore, memory::InMemory};
 
     struct MockSession {
         config: SessionConfig,

datafusion/datasource-json/src/source.rs

@@ -523,7 +523,6 @@ mod tests {
     use arrow::datatypes::{DataType, Field, Schema};
     use bytes::Bytes;
     use datafusion_datasource::FileRange;
-    use futures::TryStreamExt;
     use object_store::memory::InMemory;
     use object_store::path::Path;
     use object_store::{ObjectStoreExt, PutPayload};

datafusion/datasource-parquet/src/file_format.rs

@@ -1808,11 +1808,9 @@ async fn output_single_parquet_file_parallelized(
 #[cfg(test)]
 mod tests {
     use parquet::arrow::parquet_to_arrow_schema;
-    use std::sync::Arc;
 
     use super::*;
 
-    use arrow::datatypes::DataType;
     use parquet::schema::parser::parse_message_type;
 
     #[test]