pre-commit/pre-commit-hooks
1
0
Fork 0
mirror of https://github.com/pre-commit/pre-commit-hooks.git synced 2026-04-04 19:26:52 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Improve error message when credentials file is not provided

Browse source
This commit is contained in:
Anthony Sottile 2016-02-08 17:05:39 -08:00
parent d740faeded
commit d444ab821d
2 changed files with 24 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

20
tests/detect_aws_credentials_test.py
View file

@ -12,10 +12,6 @@ TESTS = (
('ok_json.json', 0),
)
NO_CREDENTIALS_TEST = (
('with_secrets.txt', 2),
)
@pytest.mark.parametrize(('filename', 'expected_retval'), TESTS)
def test_detect_aws_credentials(filename, expected_retval):
@ -26,10 +22,16 @@ def test_detect_aws_credentials(filename, expected_retval):
assert ret == expected_retval
@pytest.mark.parametrize(('filename', 'expected_retval'), NO_CREDENTIALS_TEST)
def test_non_existent_credentials(filename, expected_retval):
def test_non_existent_credentials(capsys):
# with a non-existent credentials file
ret = main(
[get_resource_path(filename), "--credentials-file=testing/resources/credentailsfilethatdoesntexist"]
ret = main((
get_resource_path('with_secrets.txt'),
"--credentials-file=testing/resources/credentailsfilethatdoesntexist"
))
assert ret == 2
out, _ = capsys.readouterr()
assert out == (
'No aws keys were configured at '
'testing/resources/credentailsfilethatdoesntexist\n'
'Configure them with --credentials-file\n'
)
assert ret == expected_retval