Print file line stderr
diff --git a/pre_commit_hooks/pretty_format_json.py b/pre_commit_hooks/pretty_format_json.py index 8bf61c0..b9c0aa4 100644 --- a/pre_commit_hooks/pretty_format_json.py +++ b/pre_commit_hooks/pretty_format_json.py
@@ -120,7 +120,10 @@ ) if contents != pretty_contents: - print('File {} is not pretty-formatted'.format(json_file)) + print( + 'File {} is not pretty-formatted'.format(json_file), + file=sys.stderr, + ) if args.autofix: _autofix(json_file, pretty_contents)
diff --git a/tests/pretty_format_json_test.py b/tests/pretty_format_json_test.py index d95465d..9c380a0 100644 --- a/tests/pretty_format_json_test.py +++ b/tests/pretty_format_json_test.py
@@ -108,10 +108,8 @@ def test_diffing_output(capsys): - resource_path = get_resource_path('not_pretty_formatted_json.json') expected_retval = 1 - expected_diff = ''' - { + expected_out = ''' { - "foo": - "bar", - "alist": [2, 34, 234], @@ -126,17 +124,10 @@ + "foo": "bar" } - ''' - # output should include a line with the filepath, build it here - file_output_line = 'File {} is not pretty-formatted'.format(resource_path) - # prepend the above line to the diff - expected_output = file_output_line + expected_diff - actual_retval = main([resource_path]) - actual_output = capsys.readouterr() + actual_retval = main([get_resource_path('not_pretty_formatted_json.json')]) + out, err = capsys.readouterr() assert actual_retval == expected_retval - - actual_output = '\n'.join(actual_output) - assert actual_output == expected_output + assert out == expected_out