Skip to content

Reorganize tests and rename unit_tests.py to backend_tests.py#1071

Merged
stevemessick merged 2 commits into
mainfrom
rename-tests-reorganize
Jun 24, 2026
Merged

Reorganize tests and rename unit_tests.py to backend_tests.py#1071
stevemessick merged 2 commits into
mainfrom
rename-tests-reorganize

Conversation

@stevemessick

Copy link
Copy Markdown
Contributor

This PR reorganizes the test files into a single top-level test directory with subdirectories for unit, integration, and backend tests. It also renames unit_tests.py to backend_tests.py to reflect its dependency on a running Kaggle backend.

TAG=agy
CONV=1e8b909d-d25a-422d-9783-bac0b56fdf9d

- Moved all tests to a new top-level 'test' directory.
- Structured 'test' into 'unit', 'integration', and 'backend' subdirectories.
- Renamed 'tests/unit_tests.py' to 'test/backend/backend_tests.py'.
- Updated configuration in pyproject.toml and path references in tests.

TAG=agy
CONV=1e8b909d-d25a-422d-9783-bac0b56fdf9d
@stevemessick

Copy link
Copy Markdown
Contributor Author

I will probably change the name from test to tests.

Note that there are no new tests. Everything got reorganized, renamed, etc. but it all still works.

@stevemessick stevemessick requested a review from rosbo June 23, 2026 23:01

@rosbo rosbo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The standard is to use a "tests" (plural) folder.

This is also the default directory for the hatch test command: https://hatch.pypa.io/dev/tutorials/testing/overview/#passing-arguments

Beside this directory rename, LGTM.

TAG=agy
CONV=1e8b909d-d25a-422d-9783-bac0b56fdf9d
@stevemessick stevemessick merged commit a2c4688 into main Jun 24, 2026
15 checks passed
@stevemessick stevemessick deleted the rename-tests-reorganize branch June 24, 2026 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants