lint

Author: adamweiner

planet/cli/orders.py

@@ -137,7 +137,9 @@ def orders(ctx, base_url):
     help="Filter by orders created with the provided destination reference.")
 @click.option(
     '--user-id',
-    help="Filter by user ID. Only available to organization admins. Accepts 'all' or a specific user ID.")
+    help=
+    "Filter by user ID. Only available to organization admins. Accepts 'all' or a specific user ID."
+)
 @limit
 @pretty
 async def list(ctx,

planet/cli/subscriptions.py

@@ -112,7 +112,9 @@ def subscriptions(ctx, base_url):
 )
 @click.option(
     '--user-id',
-    help="Filter by user ID. Only available to organization admins. Accepts 'all' or a specific user ID.")
+    help=
+    "Filter by user ID. Only available to organization admins. Accepts 'all' or a specific user ID."
+)
 @limit
 @click.option('--page-size',
               type=click.INT,

planet/clients/orders.py

@@ -551,7 +551,7 @@ async def list_orders(
         if destination_ref is not None:
             params["destination_ref"] = destination_ref
         if user_id is not None:
-            params["user_id"] = user_id
+            params["user_id"] = str(user_id)
         if state:
             if state not in ORDER_STATE_SEQUENCE:
                 raise exceptions.ClientError(

planet/sync/orders.py

@@ -252,18 +252,19 @@ def wait(self,
         return self._client._call_sync(
             self._client.wait(order_id, state, delay, max_attempts, callback))
 
-    def list_orders(self,
-                    state: Optional[str] = None,
-                    limit: int = 100,
-                    source_type: Optional[str] = None,
-                    name: Optional[str] = None,
-                    name__contains: Optional[str] = None,
-                    created_on: Optional[str] = None,
-                    last_modified: Optional[str] = None,
-                    hosting: Optional[bool] = None,
-                    destination_ref: Optional[str] = None,
-                    sort_by: Optional[str] = None,
-                    user_id: Optional[Union[str, int]] = None) -> Iterator[dict]:
+    def list_orders(
+            self,
+            state: Optional[str] = None,
+            limit: int = 100,
+            source_type: Optional[str] = None,
+            name: Optional[str] = None,
+            name__contains: Optional[str] = None,
+            created_on: Optional[str] = None,
+            last_modified: Optional[str] = None,
+            hosting: Optional[bool] = None,
+            destination_ref: Optional[str] = None,
+            sort_by: Optional[str] = None,
+            user_id: Optional[Union[str, int]] = None) -> Iterator[dict]:
         """Iterate over the list of stored orders.
 
         By default, order descriptions are sorted by creation date with the last created

tests/integration/test_orders_api.py

@@ -191,9 +191,7 @@ async def test_list_orders_user_id_filtering(order_descriptions, session):
 
     # if the value of user_id doesn't get sent as a url parameter,
     # the mock will fail and this test will fail
-    assert [order1, order2] == [
-        o async for o in cl.list_orders(user_id='all')
-    ]
+    assert [order1, order2] == [o async for o in cl.list_orders(user_id='all')]
 
 
 @respx.mock

tests/integration/test_orders_cli.py

@@ -138,11 +138,7 @@ def test_cli_orders_list_user_id(invoke, order_descriptions):
 
     # if the value of user_id doesn't get sent as a url parameter,
     # the mock will fail and this test will fail
-    result = invoke([
-        'list',
-        '--user-id',
-        'all'
-    ])
+    result = invoke(['list', '--user-id', 'all'])
     assert result.exit_code == 0
     sequence = '\n'.join([json.dumps(o) for o in [order1, order2]])
     assert result.output == sequence + '\n'

tests/integration/test_subscriptions_cli.py

@@ -72,9 +72,7 @@ def _invoke(extra_args, runner=None, **kwargs):
          '--hosting=true',
          '--sort-by=name DESC'
      ],
-      2),
-     (['--user-id=all'], 100),
-     (['--user-id=12345'], 100)])
+      2), (['--user-id=all'], 100), (['--user-id=12345'], 100)])
 @api_mock
 # Remember, parameters come before fixtures in the function definition.
 def test_subscriptions_list_options(invoke, options, expected_count):