third-party/pre-commit
1
0
Fork 0
mirror of https://github.com/pre-commit/pre-commit.git synced 2026-02-17 08:14:42 +04:00
Code Issues Projects Releases Packages Wiki Activity Actions

pre-commit migrate-config should not return nonzero when successful

Browse source
This commit is contained in:
Anthony Sottile 2017-09-07 09:23:36 -07:00
parent 04018ad4e7
commit a78f5d5c24
4 changed files with 5 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

2
tests/commands/autoupdate_test.py
View file

@ -306,7 +306,7 @@ def test_updates_old_format_to_new_format(tmpdir, capsys):
' language: script\n',
)
ret = autoupdate(Runner(tmpdir.strpath, C.CONFIG_FILE), tags_only=True)
assert ret == 1
assert ret == 0
contents = cfg.read()
assert contents == (
'repos:\n'

6
tests/commands/migrate_config_test.py
View file

@ -33,7 +33,7 @@ def test_migrate_config_normal_format(tmpdir, capsys):
' entry: ./bin/foo.sh\n'
' language: script\n',
)
assert migrate_config(Runner(tmpdir.strpath, C.CONFIG_FILE)) == 1
assert not migrate_config(Runner(tmpdir.strpath, C.CONFIG_FILE))
out, _ = capsys.readouterr()
assert out == 'Configuration has been migrated.\n'
contents = cfg.read()
@ -61,7 +61,7 @@ def test_migrate_config_document_marker(tmpdir):
' entry: ./bin/foo.sh\n'
' language: script\n',
)
assert migrate_config(Runner(tmpdir.strpath, C.CONFIG_FILE)) == 1
assert not migrate_config(Runner(tmpdir.strpath, C.CONFIG_FILE))
contents = cfg.read()
assert contents == (
'# comment\n'
@ -88,7 +88,7 @@ def test_migrate_config_list_literal(tmpdir):
' }]\n'
'}]',
)
assert migrate_config(Runner(tmpdir.strpath, C.CONFIG_FILE)) == 1
assert not migrate_config(Runner(tmpdir.strpath, C.CONFIG_FILE))
contents = cfg.read()
assert contents == (
'repos:\n'