-
-
Notifications
You must be signed in to change notification settings - Fork 40
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.
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
Improve test coverage #44
Comments
Add docs and tests to the Kapteyn-less methods Addresses astropy#43 and astropy#44, fixes astropy#35
Add docs and tests to the Kapteyn-less methods Addresses astropy#43 and astropy#44, fixes astropy#35
Add docs and tests to the Kapteyn-less methods Addresses astropy#43 and astropy#44, fixes astropy#35
Current test coverage is at 63%: Especially Other files we should focus on are I'm assigning this to v1.3. |
Add docs and tests to the Kapteyn-less methods Addresses astropy#43 and astropy#44, fixes astropy#35
Add docs and tests to the Kapteyn-less methods Addresses astropy#43 and astropy#44, fixes astropy#35 Add Changelog entries
Add docs and tests to the Kapteyn-less methods Addresses astropy#43 and astropy#44, fixes astropy#35 Add Changelog entries
@astropy/pyregion-maintainers - We need more high-level tests for Recently @keflavich has found two serious bugs (#109, #110) that would have been caught by any test exercising the code. Ideally someone would spend an afternoon to add tests, and then cut a new release for |
The current test coverage is around 50%, so we should work on improving that:
https://coveralls.io/r/astropy/pyregion
Of course, we shouldn't try and improve coverage of e.g.
kapteyn_celetial.py
since this is an external package (and see #43 for a suggestion of removing it in favor of usingastropy.coordinates
).The text was updated successfully, but these errors were encountered: