refactor: remove unnecessary newline

src/cli.rs

@@ -216,7 +216,7 @@ impl<'a> Cmd<'a> {
                 values.push(format!("{}__{}={}", VARIABLE_PREFIX, "call", fn_name));
             }
         }
-        values.join("")
+        values.join("\n").trim().to_string()
     }
     fn add_arg(&mut self, arg_data: &'a ArgData, position: &Position) -> Result<()> {
         let arg_data = WrapArgData::new(arg_data, self.positional_idx);
@@ -319,17 +319,17 @@ impl<'a> WrapArgData<'a> {
             return None;
         }
         if self.kind == ArgKind::Flag {
-            return Some(format!("{}_{}=1\n", VARIABLE_PREFIX, name));
+            return Some(format!("{}_{}=1", VARIABLE_PREFIX, name));
         }
         if self.multiple {
             return matches.values_of(self.name).map(|values| {
                 let values: Vec<String> = values.map(escape_value).collect();
-                format!("{}_{}=( {} )\n", VARIABLE_PREFIX, name, values.join(" "))
+                format!("{}_{}=( {} )", VARIABLE_PREFIX, name, values.join(" "))
             });
         }
         matches
             .value_of(self.name)
-            .map(|value| format!("{}_{}={}\n", VARIABLE_PREFIX, name, escape_value(value)))
+            .map(|value| format!("{}_{}={}", VARIABLE_PREFIX, name, escape_value(value)))
     }
     fn detect_conflict(
         &self,

tests/escape_test.rs

@@ -4,11 +4,11 @@ const SCRIPT: &str = r###"
 
 #[test]
 fn test_syntax_error() {
-    plain!(SCRIPT, &["prog", "$foo"], stdout: "argc_value=\\$foo\n",);
-    plain!(SCRIPT, &["prog", "`pwd`"], stdout: "argc_value=\\`pwd\\`\n",);
-    plain!(SCRIPT, &["prog", "$(pwd)"], stdout: "argc_value=\\$\\(pwd\\)\n",);
-    plain!(SCRIPT, &["prog", "'"], stdout: "argc_value=\\'\n",);
-    plain!(SCRIPT, &["prog", "\\1"], stdout: "argc_value=\\\\1\n",);
-    plain!(SCRIPT, &["prog", ""], stdout: "argc_value=''\n",);
-    plain!(SCRIPT, &["prog", "\n"], stdout: "argc_value='\n'\n",);
+    plain!(SCRIPT, &["prog", "$foo"], stdout: "argc_value=\\$foo",);
+    plain!(SCRIPT, &["prog", "`pwd`"], stdout: "argc_value=\\`pwd\\`",);
+    plain!(SCRIPT, &["prog", "$(pwd)"], stdout: "argc_value=\\$\\(pwd\\)",);
+    plain!(SCRIPT, &["prog", "'"], stdout: "argc_value=\\'",);
+    plain!(SCRIPT, &["prog", "\\1"], stdout: "argc_value=\\\\1",);
+    plain!(SCRIPT, &["prog", ""], stdout: "argc_value=''",);
+    plain!(SCRIPT, &["prog", "\n"], stdout: "argc_value='\n'",);
 }

tests/snapshots/integration__spec_test__spec_cmd_positional_with_choices_and_default_exec.snap

@@ -1,6 +1,6 @@
 ---
 source: tests/spec_test.rs
-assertion_line: 133
+assertion_line: 143
 expression: output
 
 ---

tests/snapshots/integration__spec_test__spec_cmd_without_any_arg.snap

@@ -1,6 +1,6 @@
 ---
 source: tests/spec_test.rs
-assertion_line: 139
+assertion_line: 156
 expression: output
 
 ---