Add newline at the end of JSON output (#1095)

RELEASE-NOTES.md

@@ -23,6 +23,7 @@
 ## Fixes and improvements
 * Improved support for quoted identifiers.
 * Fixed creating patches with `snow app version create` when there are 2 or more existing patches on a version
+* Using `--format=json` adds trailing new line to avoid `%` being added by some terminals to signal no new line at the end of output.
 
 # v2.3.1
 ## Backward incompatibility

src/snowflake/cli/app/printing.py

@@ -84,7 +84,9 @@ def print_structured(result: CommandResult):
     if isinstance(result, MultipleResults):
         _stream_json(result)
     else:
-        return json.dump(result, sys.stdout, cls=CustomJSONEncoder, indent=4)
+        json.dump(result, sys.stdout, cls=CustomJSONEncoder, indent=4)
+    # Adds empty line at the end
+    print()
 
 
 def _stream_json(result):

tests/output/test_printing.py

@@ -1,3 +1,4 @@
+import json
 from datetime import datetime
 from textwrap import dedent
 from typing import NamedTuple
@@ -288,18 +289,24 @@ def test_print_with_no_data_table(capsys):
 
 def test_print_with_no_data_in_query_json(capsys, _empty_cursor):
     print_result(QueryResult(_empty_cursor()), output_format=OutputFormat.JSON)
-    assert get_output(capsys) == "[]"
+    json_str = get_output(capsys)
+    json.loads(json_str)
+    assert json_str == "[]\n"
 
 
 def test_print_with_no_data_in_single_value_query_json(capsys, _empty_cursor):
     print_result(SingleQueryResult(_empty_cursor()), output_format=OutputFormat.JSON)
-    assert get_output(capsys) == "null"
+    json_str = get_output(capsys)
+    json.loads(json_str)
+    assert json_str == "null\n"
 
 
 def test_print_with_no_response_json(capsys):
     print_result(None, output_format=OutputFormat.JSON)
 
-    assert get_output(capsys) == "null"
+    json_str = get_output(capsys)
+    json.loads(json_str)
+    assert json_str == "null\n"
 
 
 @pytest.fixture