Skip to content

Commit

Permalink
Fix tests that falsely assumed table ordering in DB (#12248)
Browse files Browse the repository at this point in the history
- Fixes a test regression on Snowflake accidentally introduced by #12165
  • Loading branch information
radeusgd authored Feb 8, 2025
1 parent efe8786 commit d681da3
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ add_specs suite_builder (prefix : Text) setup =
t1 = table_1.get
t1.should_be_a DB_Table
(t1:DB_Column).name.should_equal "A"
(t1:DB_Column).to_vector.should_equal [1, 2, 3]
(t1:DB_Column).to_vector.should_equal_ignoring_order [1, 2, 3]
# Still has table methods
t1.column_names.should_equal ["A"]

Expand All @@ -28,7 +28,7 @@ add_specs suite_builder (prefix : Text) setup =

group_builder.specify "should allow passing the casted column to methods expecting column (with inserted cast)" <|
t1 = table_1.get
((t1:DB_Column) + (t1:DB_Column)) . to_vector . should_equal [2, 4, 6]
((t1:DB_Column) + (t1:DB_Column)) . to_vector . should_equal_ignoring_order [2, 4, 6]

group_builder.specify "should also work for DB_Tables returned from various operations" <|
t1 = table_2.get
Expand Down

0 comments on commit d681da3

Please sign in to comment.