Move platform() into GranuleCollectionBaseQuery

README.md

@@ -111,6 +111,9 @@ api.downloadable()
 # only include granules that are unavailable for download
 api.online_only()
 
+# filter by specific satellite platform
+api.platform("Terra")
+
 # search for collections/granules associated with or identified by concept IDs
 # note: often the ECHO collection ID can be used here as well
 # note: when using CollectionQuery, only collection concept IDs can be passed
@@ -143,9 +146,8 @@ api.day_night_flag("day")
 # filter by cloud cover percentage range
 api.cloud_cover(25, 75)
 
-# filter by specific instrument or platform
+# filter by specific instrument
 api.instrument("MODIS")
-api.platform("Terra")
 
 # filter by a sort_key note: sort_keys are require some other fields to find
 # some existing granules before they can be sorted

cmr/queries.py

@@ -717,6 +717,20 @@ def entry_title(self, entry_title: str) -> Self:
 
  return self
 
+ def platform(self, platform: str) -> Self:
+ """
+ Filter by the satellite platform the granule came from.
+
+ :param platform: name of the satellite
+ :returns: self
+ """
+
+ if not platform:
+ raise ValueError("Please provide a value for platform")
+
+ self.params['platform'] = platform
+ return self
+
 
 class GranuleQuery(GranuleCollectionBaseQuery):
  """
@@ -813,20 +827,6 @@ def instrument(self, instrument: str) -> Self:
  self.params['instrument'] = instrument
  return self
 
- def platform(self, platform: str) -> Self:
- """
- Filter by the satellite platform the granule came from.
-
- :param platform: name of the satellite
- :returns: self
- """
-
- if not platform:
- raise ValueError("Please provide a value for platform")
-
- self.params['platform'] = platform
- return self
-
  def sort_key(self, sort_key: str) -> Self:
  """
  See

tests/test_collection.py

@@ -122,6 +122,20 @@ def test_invalid_cloud_hosted(self):
  with self.assertRaises(TypeError):
  query.cloud_hosted("Test_string_for_cloud_hosted_param") # type: ignore[arg-type]
 
+ def test_platform(self):
+ query = CollectionQuery()
+
+ query.platform("1B")
+
+ self.assertIn("platform", query.params)
+ self.assertEqual(query.params["platform"], "1B")
+
+ def test_empty_platform(self):
+ query = CollectionQuery()
+
+ with self.assertRaises(ValueError):
+ query.platform(None) # type: ignore[arg-type]
+
  def test_revision_date(self):
  query = CollectionQuery()
  collections = query.short_name("SWOT_L2_HR_RiverSP_reach_2.0").revision_date("2022-05-16", "2024-06-30").get_all()