Skip to content

Converting test_study to pytest - #1439

Merged
PGijsbers merged 4 commits into
openml:mainfrom
prady0t:adding-more-pytest
Oct 16, 2026
Merged

PGijsbers merged 4 commits into
openml:mainfrom
prady0t:adding-more-pytest

Conversation

@prady0t

@prady0t prady0t commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

I've made some conversions to the test_study_functions.py file, and it now only uses pytest. List of changes made:

  • Removed the unittest.TestCase class structure
  • Converted setUp / tearDown into pytest fixtures
  • Replaced helper methods with fixtures
  • Removed all uses of self
  • Replaced unittest.assert* with plain assert
  • Removed test wrapper methods & replaced with parametrization
  • Moved shared utilities into conftest.py

Metadata

Details

Signed-off-by: Pradyot Ranjan <99216956+pradyotRanjan@users.noreply.github.com>
Signed-off-by: Pradyot Ranjan <99216956+pradyotRanjan@users.noreply.github.com>
@codecov-commenter

codecov-commenter commented Nov 20, 2025 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 53.78%. Comparing base (24a4e8b) to head (f143c98).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1439   +/-   ##
=======================================
  Coverage   53.78%   53.78%           
=======================================
  Files          63       63           
  Lines        5230     5230           
=======================================
  Hits         2813     2813           
  Misses       2417     2417           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@PGijsbers
PGijsbers merged commit aee8627 into openml:main Oct 16, 2026
24 checks passed
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.

3 participants