feat: pass runner commands as strings

use shlex to parse a user defined string into shell args that can be
passed to duct. Main driver is a better ux

Author: joe-p

Cargo.lock

@@ -37,6 +37,7 @@ regex = "1.11.1"
 
 # Used to easily deserialize the TOML configuration file
 serde = { version = "1.0.216", features = ["derive"] }
+shlex = "1.3.0"
 
 # Used to parse the TOML configuration file
 toml = "0.8.19"

Cargo.toml

@@ -154,11 +154,7 @@ See [templates documentation](./templates.md) for more information on the expect
 
 #### command
 
-The `command` field is used to define the command to run the test. This should just be the name of the executable.
-
-#### args
-
-The `args` field is used to define the arguments to pass to the test command.
+The `command` field is used to define the command to run the test.
 
 #### fail_regex_template
 

docs/configuration.md

@@ -84,8 +84,7 @@ template_dir = "./templates/minitest_unit"
 fail_regex_template = "Test{{ suite_name | convert_case('Pascal') }}#test_{{ test_name }} = \\d+\\.\\d+ s = (F|E)"
 pass_regex_template = "Test{{ suite_name | convert_case('Pascal') }}#test_{{ test_name }} = \\d+\\.\\d+ s = \\."
 
-command = "bundle"
-args = ["exec", "rake", "test", "A='--verbose'"]
+command = "bundle exec rake test A='--verbose'"
 
 # Custom Document
 

examples/shapes/polytest.toml

@@ -87,7 +87,6 @@ impl ConfigMeta {
 #[derive(Deserialize, Debug, Clone)]
 struct RunnerConfig {
     command: String,
-    args: Vec<String>,
     fail_regex_template: String,
     pass_regex_template: String,
     env: Option<HashMap<String, String>>,
@@ -96,7 +95,6 @@ struct RunnerConfig {
 #[derive(Deserialize, Debug, Clone)]
 struct Runner {
     command: String,
-    args: Vec<String>,
     fail_regex_template: String,
     pass_regex_template: String,
     env: Option<HashMap<String, String>>,
@@ -106,7 +104,6 @@ impl Runner {
     fn from_config(config: &RunnerConfig) -> Self {
         Self {
             command: config.command.clone(),
-            args: config.args.clone(),
             fail_regex_template: "(?m)".to_owned() + config.fail_regex_template.as_str(),
             pass_regex_template: "(?m)".to_owned() + config.pass_regex_template.as_str(),
             env: config.env.clone(),
@@ -200,8 +197,7 @@ impl Target {
                             .to_string(),
                         runner: Runner {
                             env: None,
-                            command: "pytest".to_string(),
-                            args: vec!["-v".to_string()],
+                            command: "pytest -v".to_string(),
                             fail_regex_template:
                                 r"(?m){{ file_name }}::test_{{ test_name }} FAILED".to_string(),
                             pass_regex_template:
@@ -223,8 +219,7 @@ impl Target {
                     test_template: include_str!("../templates/bun/test.ts.jinja").to_string(),
                     runner: Runner {
                             env: None,
-                            command: "bun".to_string(),
-                            args: vec!["test".to_string()],
+                            command: "bun test".to_string(),
                             fail_regex_template: r"(?m)\(fail\) {{ suite_name }} > {{ group_name }} > {{ test_name }}( \[\d+\.\d+ms])*$".to_string(),
                             pass_regex_template: r"(?m)\(pass\) {{ suite_name }} > {{ group_name }} > {{ test_name }}( \[\d+\.\d+ms])*$".to_string(),
                         },
@@ -548,12 +543,10 @@ fn main() -> Result<()> {
             for target in &targets {
                 let runner = target.runner.clone();
 
-                println!(
-                    "Running {}: {} {:?}",
-                    target.id, runner.command, runner.args
-                );
+                println!("Running {}: {}", target.id, runner.command);
 
-                let mut runner_cmd = cmd(runner.command, &runner.args[..]).dir(&target.out_dir);
+                let parsed_cmd: Vec<String> = shlex::Shlex::new(runner.command.as_str()).collect(); 
+                let mut runner_cmd = cmd(&parsed_cmd[0], &parsed_cmd[1..]).dir(&target.out_dir);
                 if let Some(env) = runner.env {
                     for (key, value) in env {
                         runner_cmd = runner_cmd.env(key, value);