From ae30fa4af478b0bcb161d78cab2c3768b1cded5f Mon Sep 17 00:00:00 2001
From: Andy Grove <andygrove73@gmail.com>
Date: Tue, 20 Feb 2024 09:01:47 -0700
Subject: [PATCH] code cleanup

---
 datafusion/core/tests/depcheck.rs   | 8 ++++----
 datafusion/physical-expr/Cargo.toml | 1 +
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/datafusion/core/tests/depcheck.rs b/datafusion/core/tests/depcheck.rs
index 3875ba6c5875..94448818691e 100644
--- a/datafusion/core/tests/depcheck.rs
+++ b/datafusion/core/tests/depcheck.rs
@@ -37,8 +37,8 @@ fn test_deps() -> Result<(), Box<dyn std::error::Error>> {
     {
         let deps: Vec<String> = resolve
             .deps(package_id)
-            .filter(|(a, _)| a.name().starts_with("datafusion"))
-            .map(|(a, _)| a.name().to_string())
+            .filter(|(package_id, _)| package_id.name().starts_with("datafusion"))
+            .map(|(package_id, _)| package_id.name().to_string())
             .collect();
         package_deps.insert(package_id.name().to_string(), deps);
     }
@@ -70,8 +70,8 @@ fn check_circular_deps(
         return;
     }
     seen.insert(current_dep.to_string());
-    if let Some(x) = package_deps.get(current_dep) {
-        for dep in x {
+    if let Some(deps) = package_deps.get(current_dep) {
+        for dep in deps {
             check_circular_deps(root_package, dep, package_deps, seen);
         }
     }
diff --git a/datafusion/physical-expr/Cargo.toml b/datafusion/physical-expr/Cargo.toml
index a46ad42f67c3..8f7bf99826be 100644
--- a/datafusion/physical-expr/Cargo.toml
+++ b/datafusion/physical-expr/Cargo.toml
@@ -78,6 +78,7 @@ uuid = { version = "^1.2", features = ["v4"] }
 
 [dev-dependencies]
 criterion = "0.5"
+datafusion = { workspace = true }
 rand = { workspace = true }
 rstest = { workspace = true }
 tokio = { workspace = true, features = ["rt-multi-thread"] }