Merge pull request #1345 from jqnatividad/geocode-fix-now-json-output

`geocode`: when producing JSON output with the now subcommands, use c…

src/cmd/geocode.rs

@@ -829,7 +829,18 @@ async fn geocode_main(args: Args) -> CliResult<()> {
     let engine = load_engine(geocode_index_file.clone().into(), &progress).await?;
 
     let mut rdr = rconfig.reader()?;
-    let mut wtr = Config::new(&args.flag_output).writer()?;
+    let mut wtr = Config::new(&args.flag_output)
+        .quote_style(
+            // if we're doing a now subcommand with JSON output, we don't want the CSV writer
+            // to close quote the output as it will produce invalid JSON
+            if now_cmd && (args.flag_formatstr == "%json" || args.flag_formatstr == "%pretty-json")
+            {
+                csv::QuoteStyle::Never
+            } else {
+                csv::QuoteStyle::Necessary
+            },
+        )
+        .writer()?;
 
     let headers = rdr.byte_headers()?.clone();
     let sel = rconfig.selection(&headers)?;

tests/test_geocode.rs

@@ -1564,9 +1564,8 @@ fn geocode_countryinfonow_formatstr_pretty_json() {
         .arg("mx")
         .args(["--formatstr", "%pretty-json"]);
 
-    let got: Vec<Vec<String>> = wrk.read_stdout(&mut cmd);
-    let expected = vec![svec![
-        r######"{
+    let got: String = wrk.stdout(&mut cmd);
+    let expected = r######"{
   "info": {
     "iso": "MX",
     "iso3": "MEX",
@@ -1594,7 +1593,6 @@ fn geocode_countryinfonow_formatstr_pretty_json() {
   "capital_names": {
     "en": "Mexico City"
   }
-}"######
-    ]];
+}"######;
     assert_eq!(got, expected);
 }