Ecm prep test refactor #578

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

yingli-NREL wants to merge 2 commits into master from ecm_prep_test_refactor

Collaborator

yingli-NREL commented Jan 24, 2026

Original State:

Single file: tests/ecm_prep_test.py
Size: 129,354 lines (one of the largest test files ever!)
Framework: unittest
Structure: Monolithic, all 15 test classes in one file

Final State:

15 separate test files, one per test class
Total size: 3,670 lines
Framework: pytest
Structure: Modular, organized, maintainable
Overall reduction: 97.2% (125,684 lines saved!)

Yingli added 2 commits

January 23, 2026 17:23


          first commit

9cb6092


          first commit

f7ad727

yingli-NREL self-assigned this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet