Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(python): More test restructure #6961

Merged
merged 8 commits into from
Feb 17, 2023

Conversation

stinodego
Copy link
Member

Relates to #6364

I did not change any tests, just shuffled some things around.

The aim of this PR is to better structure the py-polars/tests/unit folder. Currently, the folder contains more than 50 test modules, some containing only a single test, and others containing more than 2000 lines of code.

Setting up a sensible structure isn't that easy, as there are many overlapping concepts and data structures. This PR is not a final solution, just an improvement on the current situation.

Changes:

  • Created some new folders and moved the appropriate tests there:
    • namespaces: I had already made this one - moved some more namespace methods there
    • datatypes: Test module for testing behaviour of specific data types in various operations.
    • operations: Test module for extensive testing of specific operations like join or explode.
    • utils: For things like show_versions and the like.

@github-actions github-actions bot added python Related to Python Polars test Related to the test suite labels Feb 17, 2023
@ritchie46 ritchie46 merged commit cec13c3 into pola-rs:master Feb 17, 2023
@stinodego stinodego deleted the more-test-restructure branch February 22, 2023 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python Related to Python Polars test Related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants