Merge pull request #1039 from amarvin/patch-1

Continues processing JSONs even if hook fails (fixes #1038)

pre_commit_hooks/pretty_format_json.py

@@ -115,13 +115,17 @@ def main(argv: Sequence[str] | None = None) -> int:
                 f'Input File {json_file} is not a valid JSON, consider using '
                 f'check-json',
             )
-            return 1
-
+            status = 1
+        else:
             if contents != pretty_contents:
                 if args.autofix:
                     _autofix(json_file, pretty_contents)
                 else:
-                diff_output = get_diff(contents, pretty_contents, json_file)
+                    diff_output = get_diff(
+                        contents,
+                        pretty_contents,
+                        json_file,
+                    )
                     sys.stdout.buffer.write(diff_output.encode())
 
                 status = 1

tests/pretty_format_json_test.py

@@ -82,6 +82,24 @@ def test_autofix_main(tmpdir):
     assert ret == 0
 
 
+def test_invalid_main(tmpdir):
+    srcfile1 = tmpdir.join('not_valid_json.json')
+    srcfile1.write(
+        '{\n'
+        '  // not json\n'
+        '  "a": "b"\n'
+        '}',
+    )
+    srcfile2 = tmpdir.join('to_be_json_formatted.json')
+    srcfile2.write('{ "a": "b" }')
+
+    # it should have skipped the first file and formatted the second one
+    assert main(['--autofix', str(srcfile1), str(srcfile2)]) == 1
+
+    # confirm second file was formatted (shouldn't trigger linter again)
+    assert main([str(srcfile2)]) == 0
+
+
 def test_orderfile_get_pretty_format():
     ret = main((
         '--top-keys=alist', get_resource_path('pretty_formatted_json.json'),