Skip to content

Commit 87a0f0d

Browse files
committed
fix merge conflicts: update cli orders download test to no longer expect a message
2 parents 45ab424 + 53012b0 commit 87a0f0d

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

planet/cli/orders.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,12 +117,11 @@ async def download(ctx, order_id, quiet, overwrite, dest):
117117
with planet.reporting.StateBar(order_id=order_id,
118118
disable=quiet) as bar:
119119
await cl.poll(str(order_id), report=bar.update)
120-
filenames = await cl.download_order(
120+
_ = await cl.download_order(
121121
str(order_id),
122122
directory=dest,
123123
overwrite=overwrite,
124124
progress_bar=not quiet)
125-
click.echo(f'Downloaded {len(filenames)} files.')
126125

127126

128127
def split_id_list(ctx, param, value):

tests/integration/test_orders_cli.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -237,10 +237,8 @@ def test_cli_orders_download(invoke, mock_download_response, oid):
237237
result = invoke(['download', oid], runner=runner)
238238
assert not result.exception
239239

240-
# output is progress reporting plus the message
241-
message = 'Downloaded 2 files.\n'
242-
assert message in result.output
243-
assert message != result.output
240+
# no message, output is only progress reporting
241+
assert result.output.startswith('\r00:00 - order')
244242

245243
# Check that the files were downloaded and have the correct contents
246244
f1_path = Path(folder) / 'm1.json'

0 commit comments

Comments
 (0)