Add tests for run command

src/cmd/build.rs

@@ -29,20 +29,19 @@ impl Execute for BuildCommand {
                         self.artifact
                     ))
                 }
-            },
-            CommandLookupResult::NotFound => {
-                Err(anyhow!(
-                    "Target <{}> not found in config file <{}>",
-                    self.artifact,
-                    context.config_path
-                ))
-            },
-            CommandLookupResult::Duplicates(duplicates) => {
+            }
+            CommandLookupResult::NotFound => Err(anyhow!(
+                "Target <{}> not found in config file <{}>",
+                self.artifact,
+                context.config_path
+            )),
+            CommandLookupResult::Duplicates(ref mut duplicates) => {
+                duplicates.sort();
                 Err(anyhow!(
                     "Target <{}> is ambiguous, possible values are <{}>, please specify the command to run using one of those names",
                     self.artifact, duplicates.join(", ")
                 ))
-            },
+            }
         }
     }
 }

src/cmd/run.rs

@@ -27,25 +27,21 @@ impl Execute for RunCommand {
                 if let Some(runner) = runner {
                     runner.run(&context, &mut outputs, cleanup_manager, self.args.clone())
                 } else {
-                    Err(anyhow!(
-                        "Target <{}> is not runnable",
-                        self.name
-                    ))
+                    Err(anyhow!("Target <{}> is not runnable", self.name))
                 }
-            },
-            CommandLookupResult::NotFound => {
-                Err(anyhow!(
-                    "Target <{}> not found in config file <{}>",
-                    self.name,
-                    context.config_path
-                ))
-            },
-            CommandLookupResult::Duplicates(duplicates) => {
+            }
+            CommandLookupResult::NotFound => Err(anyhow!(
+                "Target <{}> not found in config file <{}>",
+                self.name,
+                context.config_path
+            )),
+            CommandLookupResult::Duplicates(ref mut duplicates) => {
+                duplicates.sort();
                 Err(anyhow!(
                     "Target <{}> is ambiguous, possible values are <{}>, please specify the command to run using one of those names",
                     self.name, duplicates.join(", ")
                 ))
-            },
+            }
         }
     }
 }

tests/test_build.rs

@@ -225,6 +225,6 @@ fn test_error_when_ambiguous() {
     cmd.assert().failure();
 
     cmd.assert().stderr(predicate::str::contains(
-        "Target <copy> is ambiguous, possible values are <artifact.exec.copy, artifact.container_image.copy>",
+        "Target <copy> is ambiguous, possible values are <artifact.container_image.copy, artifact.exec.copy>",
     ));
 }

tests/test_run.rs

@@ -0,0 +1,48 @@
+use assert_cmd::prelude::*;
+use predicates::prelude::*;
+
+mod common;
+
+#[test]
+fn test_error_when_does_not_exist() {
+    let config_src = r#"
+        [command.exec.copy]
+        command = "cp hello world"
+    "#;
+
+    let test_context = common::TestContext::new();
+    test_context.write_config(config_src);
+
+    let mut cmd = test_context.get_command();
+    cmd.arg("run").arg("non_existent");
+
+    cmd.assert().failure();
+
+    cmd.assert().stderr(predicate::str::contains(
+        "Target <non_existent> not found in config file <",
+    ));
+}
+
+#[test]
+fn test_error_when_ambiguous() {
+    let config_src = r#"
+        [artifact.exec.copy]
+        command = "cp hello world"
+
+        [artifact.container_image.copy]
+        context = "."
+        tag = "latest"
+    "#;
+
+    let test_context = common::TestContext::new();
+    test_context.write_config(config_src);
+
+    let mut cmd = test_context.get_command();
+    cmd.arg("run").arg("copy");
+
+    cmd.assert().failure();
+
+    cmd.assert().stderr(predicate::str::contains(
+        "Target <copy> is ambiguous, possible values are <artifact.container_image.copy, artifact.exec.copy>",
+    ));
+}