From 118150227ef93a7d196fb41bb59ed93cfd08ec04 Mon Sep 17 00:00:00 2001
From: Rusty Russell <rusty@rustcorp.com.au>
Date: Thu, 8 Aug 2019 12:06:42 +0930
Subject: [PATCH] cli: restore 0.7.0-style whitespace printing.

@renepickhardt has a shell script we broke.  While we still produce
perfectly valid JSON, we should not gratuitously change tool output.

Plus, I prefer the missing space before the ':'.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
---
 cli/lightning-cli.c  |  2 +-
 tests/test_misc.py   | 14 ++++++++++++++
 tests/test_plugin.py |  2 +-
 3 files changed, 16 insertions(+), 2 deletions(-)

diff --git a/cli/lightning-cli.c b/cli/lightning-cli.c
index f0dd270ca..f1e62be7e 100644
--- a/cli/lightning-cli.c
+++ b/cli/lightning-cli.c
@@ -337,7 +337,7 @@ static void print_json(const char *str, const jsmntok_t *tok, const char *indent
 			else
 				printf(",\n%s", next_indent);
 			print_json(str, t, next_indent);
-			printf(" : ");
+			printf(": ");
 			print_json(str, t + 1, next_indent);
 			first = false;
 		}
diff --git a/tests/test_misc.py b/tests/test_misc.py
index 28ded1cf7..63b99cf1a 100644
--- a/tests/test_misc.py
+++ b/tests/test_misc.py
@@ -658,6 +658,20 @@ def test_cli(node_factory):
     j = json.loads(out)
     assert only_one(j['invoices'])['label'] == 'l"[]{}'
 
+    # For those using shell scripts (you know who you are Rene), make sure we're maintaining whitespace
+    lines = [l for l in out.splitlines() if '"bolt11"' not in l and '"payment_hash"' not in l and '"expires_at"' not in l]
+    assert lines == ['{',
+                     '   "invoices": [',
+                     '      {',
+                     r'         "label": "l\"[]{}",',
+                     '         "msatoshi": 123000,',
+                     '         "amount_msat": "123000msat",',
+                     '         "status": "unpaid",',
+                     r'         "description": "d\"[]{}",',
+                     '      }',
+                     '   ]',
+                     '}']
+
 
 def test_daemon_option(node_factory):
     """
diff --git a/tests/test_plugin.py b/tests/test_plugin.py
index 27f5862f5..754606854 100644
--- a/tests/test_plugin.py
+++ b/tests/test_plugin.py
@@ -330,7 +330,7 @@ def test_utf8_passthrough(node_factory, executor):
                                    .format(l1.daemon.lightning_dir),
                                    'utf8', 'ナンセンス 1杯']).decode('utf-8')
     assert '\\u' not in out
-    assert out == '{\n   "utf8" : "ナンセンス 1杯"\n}\n'
+    assert out == '{\n   "utf8": "ナンセンス 1杯"\n}\n'
 
 
 def test_invoice_payment_hook(node_factory):