Changes, style, test fixes

Author: snbianco

CHANGES.rst

@@ -39,6 +39,8 @@ mast
 
 - Get the keyword corresponding to the dataset ID for a specific mission with ``mast.MastMissions.get_dataset_kwd``. [#3155]
 
+- Handle coordinates that are not in the ICRS frame in query functions. [#3164]
+
 mocserver
 ^^^^^^^^^
 

astroquery/mast/missions.py

@@ -19,7 +19,7 @@
 from requests import HTTPError, RequestException
 
 from astroquery import log
-from astroquery.utils import commons, async_to_sync
+from astroquery.utils import async_to_sync
 from astroquery.utils.class_or_instance import class_or_instance
 from astroquery.exceptions import InvalidQueryError, MaxResultsWarning, InputWarning, NoResultsWarning
 

astroquery/mast/tests/test_mast_remote.py

@@ -1262,7 +1262,7 @@ def test_tesscut_get_sectors_mt(self):
         assert sector_table['ccd'][0] == 1
 
         error_noname = ("Please specify the object name or ID (as understood by the "
-                        "`JPL ephemerides service <https://ssd.jpl.nasa.gov/horizons.cgi>`__) "
+                        "`JPL ephemerides service <https://ssd.jpl.nasa.gov/horizons/app.html>`__) "
                         "of a moving target such as an asteroid or comet.")
         error_nameresolve = f"Could not resolve {moving_target_name} to a sky position."
         error_mt_coord = "Only one of moving_target and coordinates may be specified."
@@ -1345,7 +1345,7 @@ def test_tesscut_download_cutouts_mt(self, tmpdir):
             assert os.path.isfile(row['Local Path'])
 
         error_noname = ("Please specify the object name or ID (as understood by the "
-                        "`JPL ephemerides service <https://ssd.jpl.nasa.gov/horizons.cgi>`__) of "
+                        "`JPL ephemerides service <https://ssd.jpl.nasa.gov/horizons/app.html>`__) of "
                         "a moving target such as an asteroid or comet.")
         error_nameresolve = f"Could not resolve {moving_target_name} to a sky position."
         error_mt_coord = "Only one of moving_target and coordinates may be specified."
@@ -1426,7 +1426,7 @@ def test_tesscut_get_cutouts_mt(self):
         assert isinstance(cutout_hdus_list[0], fits.HDUList)
 
         error_noname = ("Please specify the object name or ID (as understood by the "
-                        "`JPL ephemerides service <https://ssd.jpl.nasa.gov/horizons.cgi>`__) of "
+                        "`JPL ephemerides service <https://ssd.jpl.nasa.gov/horizons/app.html>`__) of "
                         "a moving target such as an asteroid or comet.")
         error_nameresolve = f"Could not resolve {moving_target_name} to a sky position."
         error_mt_coord = "Only one of moving_target and coordinates may be specified."