address feedback

Author: asonnenschein

planet/cli/orders.py

@@ -367,7 +367,8 @@ async def create(ctx, request, pretty, **kwargs):
 @click.option(
     '--fallback-bundle',
     required=False,
-    help='Fallback bundle type(s) for the item if bundle is not available.',
+    help="""Fallback bundle type(s) for the item if bundle is not available.
+    Multiple fallback bundles can be specified as a comma-separated string.""",
     type=str,
     callback=check_bundle)
 @click.option('--name',

planet/order_request.py

@@ -115,9 +115,11 @@ def product(item_ids: List[str],
         product_bundle: Set of asset types for the catalog items.
         item_type: The class of spacecraft and processing characteristics
             for the catalog items.
-        fallback_bundle: In case product_bundle not having
+        fallback_bundle: In case of product_bundle not having
             all asset types available, which would result in failed
-            delivery, try a fallback bundle
+            delivery, try one or more fallback bundles. Multiple
+            fallback bundles may be provided as a list or a comma
+            separated string.
 
     Raises:
         planet.specs.SpecificationException: If bundle or fallback bundle

tests/unit/test_order_request.py

@@ -411,13 +411,18 @@ def test_sentinel_hub_collection_configuration():
     [
         ('analytic_8b_udm2', 'analytic_udm2'),
         ('analytic_8b_udm2', 'analytic_udm2,analytic_3b_udm2'),
+        ('analytic_8b_udm2', ['analytic_udm2', 'analytic_3b_udm2']),
     ])
 def test_fallback_bundle_success(bundle, fallback_bundle):
     product = order_request.product(["20250130_035211_69_2516"],
                                     bundle,
                                     "PSScene",
                                     fallback_bundle=fallback_bundle)
 
+    # Ensure fallback_bundle is a string for the expected output
+    fallback_bundle = fallback_bundle if isinstance(
+        fallback_bundle, str) else ','.join(fallback_bundle)
+
     expected = {
         "item_ids": ["20250130_035211_69_2516"],
         "item_type": "PSScene",
@@ -432,6 +437,8 @@ def test_fallback_bundle_success(bundle, fallback_bundle):
         ('analytic_8b_udm2', ''),
         ('analytic_8b_udm2', 'analytic_udm2;analytic_3b_udm2'),
         ('analytic_8b_udm2', 'analytic_udm2,analytic_3b_udm2,fake_bundle'),
+        ('analytic_8b_udm2',
+         ['analytic_udm2', 'analytic_3b_udm2', 'fake_bundle']),
     ])
 def test_fallback_bundle_invalid(bundle, fallback_bundle):
     with pytest.raises(specs.SpecificationException):