Close FITS files after loading

[svank]

I noticed a comment pointing out that when FITS file names are passed for the input data, the file is opened but never closed. Changing that looked like a quick fix, and should be safe---after the file is opened, parse_input_data calls itself a few times before eventually extracting the .data and .header attributes of the selected HDU and returning those, so the HDUL object never leaves parse_input_data. (Not sure if the garbage collector would have eventually closed everything anyway.)

While checking that this was covered by existing tests, I saw that the parse_input_data tests were checking the data array that gets returned but not the corresponding WCS, so I augmented those tests to check both.

(The CI failure appears to be pre-existing)

[codecov]

Codecov Report

Merging #330 (6fbceb0) into main (aa23eb1) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #330      +/-   ##
==========================================
+ Coverage   91.75%   91.77%   +0.01%     
==========================================
  Files          24       24              
  Lines         716      717       +1     
==========================================
+ Hits          657      658       +1     
  Misses         59       59              

Impacted Files	Coverage Δ
reproject/utils.py	87.83% <100.00%> (+0.08%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[astrofrog]

Thanks for the fix!