Add --quite flag to 'dtool copy' command

CHANGELOG.rst

@@ -12,6 +12,7 @@ Added
 ^^^^^
 
 - ``dtool readme show`` command that returns the readme content
+- ``--quiet`` flag to ``dtool copy`` command
 
 
 Changed

dtool_create/dataset.py

@@ -277,10 +277,11 @@ def freeze(proto_dataset_uri):
 
 
 @click.command()
+@click.option("--quiet", "-q", is_flag=True)
 @dataset_uri_argument
 @click.argument("prefix", default="")
 @click.argument("storage", default="file", callback=storagebroker_validation)
-def copy(dataset_uri, prefix, storage):
+def copy(quiet, dataset_uri, prefix, storage):
     """Copy a dataset to a different location."""
     # Check if the destination URI is already a dataset
     # and exit gracefully if true.
@@ -302,14 +303,22 @@ def copy(dataset_uri, prefix, storage):
                 "Path already exists: {}".format(parsed_uri.path))
 
     # Finally do the copy
-    num_items = len(list(src_dataset.identifiers))
-    with click.progressbar(length=num_items*2,
-                           label="Copying dataset") as progressbar:
+    if quiet:
         dest_uri = dtoolcore.copy(
             dataset_uri,
             prefix,
             storage,
-            CONFIG_PATH,
-            progressbar)
-
-    click.secho("Dataset copied to {}".format(dest_uri))
+            CONFIG_PATH)
+        click.secho(dest_uri)
+    else:
+        num_items = len(list(src_dataset.identifiers))
+        with click.progressbar(length=num_items*2,
+                               label="Copying dataset") as progressbar:
+            dest_uri = dtoolcore.copy(
+                dataset_uri,
+                prefix,
+                storage,
+                CONFIG_PATH,
+                progressbar)
+
+        click.secho("Dataset copied to:\n{}".format(dest_uri))

tests/test_dtool_dataset_copy.py

@@ -5,6 +5,7 @@
 from click.testing import CliRunner
 
 from dtoolcore import DataSet, ProtoDataSet
+from dtoolcore.compare import diff_content
 
 from . import chdir_fixture, tmp_dir_fixture  # NOQA
 
@@ -53,3 +54,17 @@ def test_dataset_copy_functional(chdir_fixture):  # NOQA
     result = runner.invoke(copy, [dataset_uri, copy_directory])
     assert result.exit_code != 0
     assert result.output.find("Error: Dataset already exists") != -1
+
+    # Create another directory to copy the dataset to.
+    copy_directory_2 = os.path.abspath("copy_dir_2")
+    os.mkdir(copy_directory_2)
+
+    # Test the quite flag.
+    result = runner.invoke(copy, ["--quiet", dataset_uri, copy_directory_2])
+    assert result.exit_code == 0
+    expected_uri = "file://" + os.path.join(copy_directory_2, dataset_name)
+    assert result.output.strip() == expected_uri
+
+    # Compare the content of the two datasets.
+    copied_dataset = DataSet.from_uri(expected_uri)
+    assert len(diff_content(dataset, copied_dataset)) == 0