Merge pull request #2347 from oda-hub/fix-degree-representation

freeze degree string representation

Author: bsipocz

astroquery/heasarc/core.py

@@ -169,9 +169,11 @@ def query_region_async(self, position: Union[coordinates.SkyCoord, str],
         kwargs['equinox'] = 2000
 
         # Generate the request
+        # Fixed string representation of coordinates ensures that request payload
+        # does not depend on python/astropy version for the same input coordinates
         request_payload = self._args_to_payload(
             mission=mission,
-            entry="{},{}".format(c.ra.degree, c.dec.degree),
+            entry=f"{c.ra.degree:.10f},{c.dec.degree:.10f}",
             radius=u.Quantity(radius),
             **kwargs
         )

astroquery/heasarc/tests/data/0f36aba7.dat renamed to astroquery/heasarc/tests/data/10f92917.dat

@@ -1,3 +1,4 @@
+import json
 import os
 import glob
 import hashlib
@@ -40,7 +41,11 @@ def fileid_for_request(url, params):
 
 def filename_for_request(url, params, output=False):
     fileid = fileid_for_request(url, params)
-    return data_path(fileid, output=output)
+
+    filename = data_path(fileid, output=output)
+    log.debug(f'constructed filename {filename} for request: ' + json.dumps(dict(url=url, params=params), sort_keys=True, indent=4))
+
+    return filename
 
 
 def get_mockreturn(session, method, url, params=None, timeout=10, **kwargs):

astroquery/heasarc/tests/data/69b39684.dat renamed to astroquery/heasarc/tests/data/6ad7a587.dat

@@ -24,6 +24,7 @@ def test_custom_args(self):
                                      time="2020-09-01 .. 2020-12-01",
                                      resultmax=10,
                                      good_isgri=">1000",
+                                     cache=False
                                      )
 
     def test_filter_custom_args(self):

astroquery/heasarc/tests/data/31b1e555.dat renamed to astroquery/heasarc/tests/data/6df5af53.dat

@@ -32,6 +32,7 @@ def test_custom_args(self):
                         time="2020-09-01 .. 2020-12-01",
                         resultmax=10,
                         good_isgri=">1000",
+                        cache=False
                     )
 
     def test_filter_custom_args(self):