Skip to content

Commit

Permalink
Fix failing test - test_aql_load_file_local_file_pattern_dataframe on…
Browse files Browse the repository at this point in the history
… CI (#1895)

# Description
## What is the current behavior?
Currently, the test case is failing on the main branch -
`test_aql_load_file_local_file_pattern_dataframe`.

## What is the new behavior?
A test case is failing because the index was considered in the equality
test when using `sort_value()`. With this change we are ignoring the
index. Since we are concatenating two files, if the files are
concatenated in the right order we have a test passing if they don't it
fails.

## Does this introduce a breaking change?
Nope

### Checklist
- [ ] Created tests which fail without the change (if possible)
- [ ] Extended the README / documentation, if necessary

(cherry picked from commit 6050635)
  • Loading branch information
utkarsharma2 authored and tatiana committed Apr 21, 2023
1 parent 3cfd2bb commit ee546d2
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion python-sdk/tests_integration/sql/operators/test_load_file.py
Original file line number Diff line number Diff line change
Expand Up @@ -337,7 +337,9 @@ def test_aql_load_file_local_file_pattern_dataframe(sample_dag):
def validate(input_df):
assert isinstance(input_df, pd.DataFrame)
assert test_df.shape == input_df.shape
assert test_df.sort_values("sell").equals(input_df.sort_values("sell"))
assert test_df.sort_values("sell", ignore_index=True).equals(
input_df.sort_values("sell", ignore_index=True)
)
print(input_df)

with sample_dag:
Expand Down

0 comments on commit ee546d2

Please sign in to comment.