From 606d05ae290de9ce43645515b6e9c970bff009d0 Mon Sep 17 00:00:00 2001 From: renaudjester Date: Fri, 12 Jul 2024 13:10:34 +0200 Subject: [PATCH] test: update tests --- copernicusmarine/core_functions/get.py | 1 - copernicusmarine/core_functions/subset.py | 3 +-- tests/__snapshots__/test_help_command_interface.ambr | 4 ++-- 3 files changed, 3 insertions(+), 5 deletions(-) diff --git a/copernicusmarine/core_functions/get.py b/copernicusmarine/core_functions/get.py index 553c58e4..5e71d87f 100644 --- a/copernicusmarine/core_functions/get.py +++ b/copernicusmarine/core_functions/get.py @@ -64,7 +64,6 @@ def get_function( if not get_request.dataset_id: raise ValueError("Please provide a dataset id for a get request.") request_update_dict = { - "dataset_id": dataset_id, "force_dataset_version": force_dataset_version, "output_directory": output_directory, "force_service": force_service, diff --git a/copernicusmarine/core_functions/subset.py b/copernicusmarine/core_functions/subset.py index 1ce78a47..31753f76 100644 --- a/copernicusmarine/core_functions/subset.py +++ b/copernicusmarine/core_functions/subset.py @@ -96,7 +96,6 @@ def subset_function( ) subset_request.update(motu_api_subset_request.__dict__) request_update_dict = { - "dataset_id": dataset_id, "force_dataset_version": force_dataset_version, "force_dataset_part": force_dataset_part, "variables": variables, @@ -142,7 +141,7 @@ def subset_function( raise SyntaxError("Must specify 'dataset_id' option") logger.info( "To retrieve a complete dataset, please use instead: " - f"copernicusmarine get --dataset-id {dataset_id}" + f"copernicusmarine get --dataset-id {subset_request.dataset_id}" ) raise ValueError( "Missing subset option. Try 'copernicusmarine subset --help'." diff --git a/tests/__snapshots__/test_help_command_interface.ambr b/tests/__snapshots__/test_help_command_interface.ambr index e0641458..66ccbfb9 100644 --- a/tests/__snapshots__/test_help_command_interface.ambr +++ b/tests/__snapshots__/test_help_command_interface.ambr @@ -68,7 +68,7 @@ ' downloaded if none of the --filter or --regex options is specified.', '', 'Options:', - ' -i, --dataset-id TEXT The datasetID. [required]', + ' -i, --dataset-id TEXT The datasetID.', ' --dataset-version, --force-dataset-version TEXT', ' Force the selection of a specific dataset', ' version.', @@ -213,7 +213,7 @@ ' ones from the --request-file option.', '', 'Options:', - ' -i, --dataset-id TEXT The datasetID. [required]', + ' -i, --dataset-id TEXT The datasetID.', ' --dataset-version, --force-dataset-version TEXT', ' Force the selection of a specific dataset', ' version.',