pyregion icon indicating copy to clipboard operation
pyregion copied to clipboard

Improve test coverage

Open astrofrog opened this issue 11 years ago • 2 comments

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 using astropy.coordinates).

astrofrog avatar Oct 30 '14 09:10 astrofrog

Current test coverage is at 63%: https://coveralls.io/github/astropy/pyregion

Especially pyregion/mpl_helper.py is at 0%, if someone could add even one test that does MPL plotting, that would be very valuable to have.

Other files we should focus on are pyregion/region_to_filter.py and pyregion/core.py where it's easy to test many lines with just a few high-level test that make sure key functionality is working and keeps working.

I'm assigning this to v1.3.

cdeil avatar Aug 04 '16 08:08 cdeil

@astropy/pyregion-maintainers - We need more high-level tests for pyregion to increase code coverage.

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 pyregion. Someone interested?

cdeil avatar Apr 17 '17 20:04 cdeil