Details
-
Type:
Bug
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: afw
-
Labels:None
-
Story Points:2
-
Epic Link:
-
Sprint:Science Pipelines DM-S15-4, Science Pipelines DM-S15-5
-
Team:Alert Production
Description
Running coverage.py on the AFW test suite indicated that two test classes in tests/wcs1.py are disabled. WCSTestCaseCFHT was added by Robert Lupton in 2007 but disabled during a merge a long time ago by Jim Bosch in 2010 but with no indication as to why. WCSRotateFlip appeared in 2012 (added by Simon Krughoff) but doesn't appear in the suite list at the end and so does not execute.
Similarly testSchema.py has two tests that are not run: xtestSchema and testJoin. I assume xtestSchema is deliberately disabled but could there at least be a comment in the test explaining why?
My feeling is that we should either run the tests or they should be removed. Having them their gives the impression they are doing something useful.
Less importantly, warpExposure.py has some support code for comparing masked images that was written in 2009 by Russell Owen but which is not used anywhere in the test.
The tests in testSchema.py are not intentionally disabled and should be enabled:
WCSTestCaseCFHT was pretty bitrotted, but I've managed to fix it.
I've put both of these changes on branch u/jbosch/
DM-2929.