Skip to content

Commit

Permalink
fix: multiple_group_by_clauses_with_multiple_agg_and_non test ordering
Browse files Browse the repository at this point in the history
  • Loading branch information
akhilender-bongirwar committed Oct 23, 2024
1 parent b5cdcfd commit 26c26c5
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions crates/proof-of-sql/src/sql/parse/query_expr_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1401,10 +1401,10 @@ fn we_can_use_multiple_group_by_clauses_with_multiple_agg_and_non_agg_exprs() {
let accessor = schema_accessor_from_table_ref_with_schema(
t,
indexmap! {
"salary".parse().unwrap() => ColumnType::BigInt,
"bonus".parse().unwrap() => ColumnType::BigInt,
"tax".parse().unwrap() => ColumnType::BigInt,
"name".parse().unwrap() => ColumnType::VarChar,
"salary".parse().unwrap() => ColumnType::BigInt,
"tax".parse().unwrap() => ColumnType::BigInt,
},
);
let query_text = "select salary d1, max(tax), salary d2, sum(bonus) sum_bonus, count(name) count_s from sxt.employees group by salary, bonus, salary";
Expand All @@ -1415,7 +1415,7 @@ fn we_can_use_multiple_group_by_clauses_with_multiple_agg_and_non_agg_exprs() {

let expected_ast = QueryExpr::new(
filter(
cols_expr_plan(t, &["salary", "bonus", "tax", "name"], &accessor),
cols_expr_plan(t, &["bonus", "name", "salary", "tax"], &accessor),
tab(t),
const_bool(true),
),
Expand Down

0 comments on commit 26c26c5

Please sign in to comment.