Merge pull request spacetelescope#2589 from pllim/fix-ann-reg-test

TST: Fix ds9_annulus_01.reg test case

jdaviz/configs/imviz/tests/test_regions.py

@@ -3,6 +3,7 @@
 from astropy import units as u
 from astropy.coordinates import SkyCoord, Angle
 from astropy.utils.data import get_pkg_data_filename
+from numpy.testing import assert_allclose
 from photutils.aperture import CircularAperture, SkyCircularAperture
 from regions import (PixCoord, CircleSkyRegion, RectanglePixelRegion, CirclePixelRegion,
                      EllipsePixelRegion, PointSkyRegion, PolygonPixelRegion,
@@ -154,11 +155,16 @@ def test_regions_annulus_from_load_data(self):
         self.imviz.load_data(regfile)
         assert len(self.imviz.app.data_collection) == 2  # Make sure not loaded as data
 
+        # Test data is set up such that 1 pixel is 1 arcsec.
+        subset_radii = {"Subset 1": [0.5, 1], "Subset 2": [1, 3]}
+
         subsets = self.imviz.get_interactive_regions()
-        subset_names = list(subsets.keys())
+        subset_names = sorted(subsets.keys())
         assert subset_names == ['Subset 1', 'Subset 2']
         for n in subset_names:
             assert isinstance(subsets[n], CircleAnnulusPixelRegion)
+            assert_allclose(subsets[n].center.xy, 4.5, rtol=5e-6)
+            assert_allclose([subsets[n].inner_radius, subsets[n].outer_radius], subset_radii[n])
 
     def test_photutils_pixel(self):
         my_aper = CircularAperture((5, 5), r=2)