Skip to content

Commit

Permalink
Remove unused assert_analyzed_plan_ne test helper
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
findepi committed Oct 26, 2024
1 parent 7b2284c commit 4b6f1c6
Showing 1 changed file with 0 additions and 14 deletions.
14 changes: 0 additions & 14 deletions datafusion/optimizer/src/test/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -133,20 +133,6 @@ pub fn assert_analyzed_plan_with_config_eq(
Ok(())
}

pub fn assert_analyzed_plan_ne(
rule: Arc<dyn AnalyzerRule + Send + Sync>,
plan: LogicalPlan,
expected: &str,
) -> Result<()> {
let options = ConfigOptions::default();
let analyzed_plan =
Analyzer::with_rules(vec![rule]).execute_and_check(plan, &options, |_, _| {})?;
let formatted_plan = format!("{analyzed_plan}");
assert_ne!(formatted_plan, expected);

Ok(())
}

pub fn assert_analyzed_plan_eq_display_indent(
rule: Arc<dyn AnalyzerRule + Send + Sync>,
plan: LogicalPlan,
Expand Down

0 comments on commit 4b6f1c6

Please sign in to comment.