Follow-up from PR #942 (review items R1/R2, #942 (review)). Lowest priority of the follow-ups — pure internal cleanup, no behavior change.
ccdproc/tests/_escape_triage.py currently mixes three separable concerns:
- failure-triage grouping (collapse backend test failures by escape site),
- the live escape-log tally fed by the logger, and
- baseline load/write/enforce (the ratchet),
while the np.asarray/np.asanyarray/np.ma.asanyarray monkeypatching itself lives in ccdproc/conftest.py.
Suggested refactor:
- Split into e.g.
_escape_log.py (logger + tally) and _escape_ratchet.py (baseline load/write/enforce), keeping the failure-triage summary where it fits best.
- Register the pytest hooks via
pytest_plugins = ["ccdproc.tests..."] in conftest.py instead of the current load-bearing # noqa: F401 hook imports.
Follow-up from PR #942 (review items R1/R2, #942 (review)). Lowest priority of the follow-ups — pure internal cleanup, no behavior change.
ccdproc/tests/_escape_triage.pycurrently mixes three separable concerns:while the
np.asarray/np.asanyarray/np.ma.asanyarraymonkeypatching itself lives inccdproc/conftest.py.Suggested refactor:
_escape_log.py(logger + tally) and_escape_ratchet.py(baseline load/write/enforce), keeping the failure-triage summary where it fits best.pytest_plugins = ["ccdproc.tests..."]inconftest.pyinstead of the current load-bearing# noqa: F401hook imports.