Description

This PR adds a missing unit test export_saved_model_test.py to verify that ExportFileManager.clean_up correctly handles negative max_to_keep values (early return).

So I added test_export_file_manager_negative_max_to_keep to verify that no files are deleted when max_to_keep is negative.

Type of change

For a new feature or function, please create an issue first to discuss it
with us before submitting a pull request.

Note: Please delete options that are not relevant.

[x] Other (Test coverage improvement)

Tests

I verified the changes by running the unit tests and generating a coverage report.

Test Configuration:

OS: Windows 11
Python Version: 3.10
Command:
python -m orbit.actions.export_saved_model_test
coverage run --source=orbit -m orbit.actions.export_saved_model_test

Checklist

• I have signed the Contributor License Agreement.
• I have read guidelines for pull request.
• My code follows the coding guidelines.
• I have performed a self code review of my own code.
• I have commented my code, particularly in hard-to-understand areas.
• I have made corresponding changes to the documentation.
• My changes generate no new warnings.
• I have added tests that prove my fix is effective or that my feature works.