Sorting table based on time (#96)

sunpy · Sep 11, 2023 · 4ca6c91 · 4ca6c91
1 parent e362e92
commit 4ca6c91
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/changelog/96.feature.rst b/changelog/96.feature.rst
@@ -0,0 +1 @@
+Now query returns time sorted table of results.
diff --git a/sunpy_soar/client.py b/sunpy_soar/client.py
@@ -95,7 +95,7 @@ def _do_search(query):
             info["begin_time"] = parse_time(info["begin_time"]).iso
             info["end_time"] = parse_time(info["end_time"]).iso
 
-        return astropy.table.QTable(
+        result_table = astropy.table.QTable(
             {
                 "Instrument": info["instrument"],
                 "Data product": info["descriptor"],
@@ -108,6 +108,8 @@ def _do_search(query):
                 "SOOP Name": info["soop_name"],
             },
         )
+        result_table.sort("Start time")
+        return result_table
 
     def fetch(self, query_results, *, path, downloader, **kwargs):
         """

diff --git a/sunpy_soar/tests/test_sunpy_soar.py b/sunpy_soar/tests/test_sunpy_soar.py
@@ -19,7 +19,7 @@ def test_search():
     res = Fido.search(instrument, time, level, product)
     assert len(res) == 1
     assert len(res[0]) == 37
-    assert u.allclose(res[0, 0]["Filesize"], 3.47 * u.Mbyte)
+    assert u.allclose(res[0, 0]["Filesize"], 3.45 * u.Mbyte)
 
     # check passing upper case descriptor
     product = a.soar.Product("EUI-FSI174-IMAGE")
@@ -29,7 +29,9 @@ def test_search():
     files = Fido.fetch(res[0, 0])
     assert len(files) == 1
     fname = files[0]
+
     assert u.allclose(Path(fname).stat().st_size * u.byte, res[0, 0]["Filesize"], atol=1e-3 * u.Mbyte)
+
     # Smoke test that we can read this into a map
     sunpy.map.Map(fname)