HEASARC / cfitsio

C and Fortran library for reading and writing FITS files
19 stars 7 forks source link

Expand the test suite for cfitsio. #12

Open demitri opened 7 months ago

demitri commented 7 months ago

The cfitsio standard covers a large parameter space of options. When developing new software (or just testing updates to existing software), it's extremely valuable to be able to run a battery of tests that cover this entire parameter space. I would like to request a testing suite of FITS files be created that can be used to test all of the possibilities of the format. For example, create files that have:

Images

Headers

Tables

The above is by no means comprehensive. It would be useful to compile a list of things to test that provide 100% coverage of the specification.

In addition, it is just as useful to ensure that programs that utilize CFITSIO gracefully handle improper or badly formatted files, for example:

Yes, this would be a fair amount of work. Even a partial set of test files would be extremely valuable. Any program that utilizes CFITSIO must address all of these points and more, so this effort would be beneficial to every program that uses the library. It would also provide test coverage for the library itself (I've certainly encountered cases that haven't been tested before). Finally, this will ultimately provide a better user experience for our community by identifying problems at a much earlier stage of development.

bryanirby commented 7 months ago

Thanks for the note. We agree that an expanded test suite would be useful, and will take all of this into consideration for future development.