Skip to content

Commit 80ad713

Browse files
authored
Remove unused assert_analyzed_plan_ne test helper (#13121)
Plan textual representation is rich. Testing it's not a particular string is difficult to make robust, that's probably why the helper is unused.
1 parent 467a804 commit 80ad713

File tree

1 file changed

+0
-14
lines changed
  • datafusion/optimizer/src/test

1 file changed

+0
-14
lines changed

datafusion/optimizer/src/test/mod.rs

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -133,20 +133,6 @@ pub fn assert_analyzed_plan_with_config_eq(
133133
Ok(())
134134
}
135135

136-
pub fn assert_analyzed_plan_ne(
137-
rule: Arc<dyn AnalyzerRule + Send + Sync>,
138-
plan: LogicalPlan,
139-
expected: &str,
140-
) -> Result<()> {
141-
let options = ConfigOptions::default();
142-
let analyzed_plan =
143-
Analyzer::with_rules(vec![rule]).execute_and_check(plan, &options, |_, _| {})?;
144-
let formatted_plan = format!("{analyzed_plan}");
145-
assert_ne!(formatted_plan, expected);
146-
147-
Ok(())
148-
}
149-
150136
pub fn assert_analyzed_plan_eq_display_indent(
151137
rule: Arc<dyn AnalyzerRule + Send + Sync>,
152138
plan: LogicalPlan,

0 commit comments

Comments
 (0)