Skip to content

Commit 102f879

Browse files
authored
Migrate-substrait-tests-to-insta, part2 (#15480)
* add `cargo insta` to dev dependencies * migrate `consumer_intergration.rs` tests to `insta` * Revert "migrate `consumer_intergration.rs` tests to `insta`" This reverts commit c3be2eb. * migrate `consumer_integration.rs` to `insta` inline snapshot * migrate logical plans tests to use `insta` snapshots * migrate emit_kind_tests to use `insta` snapshots * migrate function_test to use `insta` snapshots for assertions * migrate substrait_validations tests to use insta snapshots, missing `insta` mapping to `assert!` * revert `handle_emit_as_project_without_volatile_exprs` back to `assert_eq!` and remove `format!` for `assert_snapshot!` * migrate function and validation tests to use plan directly in assert_snapshot! * migrate serialize tests to use insta snapshots for assertions * migrate logical_plans test to use insta snapshots for assertions * WIP * migrate `assert_expected_plan_substrait` * refactor tests to use assert_and_generate_plan and assert_snapshot! for improved clarity and consistency * remove println! * migrate tests to use generate_plan_from_sql for improved clarity
1 parent 14635da commit 102f879

File tree

1 file changed

+204
-131
lines changed

1 file changed

+204
-131
lines changed

0 commit comments

Comments
 (0)