Skip to content

Comments

TST: Refactor and expand classifiers test suite#26

Merged
dguijo merged 11 commits intomainfrom
test-classifiers
Jul 14, 2025
Merged

TST: Refactor and expand classifiers test suite#26
dguijo merged 11 commits intomainfrom
test-classifiers

Conversation

@angelsevillamol
Copy link
Collaborator

What does this implement/fix? Explain your changes

This PR refactors the classifier test suite to improve code quality and expand test coverage.

  • Add reusable X and y fixtures and load_dataset utility.
  • Remove unused tests and fixtures.
  • Parametrize tests with pytest.mark.parametrize.
  • Simplify logic and rename test functions/variables for clarity.
  • Add input validation tests for OrdinalDecomposition.
  • Add docstrings and clean up comments.

Copy link
Member

@dguijo dguijo left a comment

Choose a reason for hiding this comment

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

Good work ☺️

@dguijo dguijo merged commit 6a19f70 into main Jul 14, 2025
15 of 17 checks passed
@angelsevillamol angelsevillamol deleted the test-classifiers branch July 14, 2025 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants