[siliataider]

This Pull request:

Changes:

Moves the RDataLoader tests from unittest to pytest to make them shorter and easier to read and to extend.
The test logic is unchanged.

• Much shorter tests: 8341 --> 5152 lines (-38%), with 115 tests in 89 test functions instead of 122
• One file per test class: ml_dataloader.py is split into ml_dataloader_single_dataframe.py, ml_dataloader_multiple_dataframes.py, ml_dataloader_undersampling.py, ml_dataloader_oversampling.py, ml_dataloader_save.py, ml_dataloader_implicit_mt.py and ml_dataloader_from_files.py
• Fixtures for the input data: each test can directly ask for a common input built with the fixtures instead of writing its own
  • The helpers and the shared fixtures are in the existing conftest.py
• No more file cleanup code: each test runs in its own tmp_path, created and deleted by pytest
  • This removes all the try/except/finally blocks and the os.removes that existed only to delete temporary files
  • Tests can't clash on file names anymore
• One test class for both lazy and eager loading: the copy-pasted lazy/eager classes are merged into TestDataLoaderSingleDataframe and TestDataLoaderMultipleDataframes, which run all of their tests twice (for both loading modes) using a parametrized fixture

Checklist:

• tested changes locally

[github-actions]

Test Results

    24 files      24 suites   3d 22h 50m 58s ⏱️
 3 889 tests  3 887 ✅ 0 💤 2 ❌
83 773 runs  83 771 ✅ 0 💤 2 ❌

For more details on these failures, see this check.

Results for commit 74e1241.

♻️ This comment has been updated with latest results.