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

Merge pull request #135 from pre-commit/install_hooks

Browse source 
Support --install-hooks as an option for pre-commit install
This commit is contained in:
Anthony Sottile 2014-07-03 19:38:56 -07:00
commit 76ffb3665a
3 changed files with 43 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

28
tests/commands/install_uninstall_test.py
View file

@ -76,11 +76,12 @@ def test_uninstall(tmpdir_factory):
assert not os.path.exists(runner.pre_commit_path)
def _get_commit_output(tmpdir_factory, touch_file='foo'):
def _get_commit_output(tmpdir_factory, touch_file='foo', home=None):
local['touch'](touch_file)
local['git']('add', touch_file)
# Don't want to write to home directory
env = dict(os.environ, **{'PRE_COMMIT_HOME': tmpdir_factory.get()})
home = home or tmpdir_factory.get()
env = dict(os.environ, **{'PRE_COMMIT_HOME': home})
return local['git'].run(
['commit', '-m', 'Commit!', '--allow-empty'],
# git commit puts pre-commit to stderr
@ -337,3 +338,26 @@ def test_uninstall_doesnt_remove_not_our_hooks(tmpdir_factory):
assert uninstall(runner) == 0
assert os.path.exists(runner.pre_commit_path)
PRE_INSTALLED = re.compile(
r'Bash hook\.+Passed\n'
r'\[master [a-f0-9]{7}\] Commit!\n' +
FILES_CHANGED +
r' create mode 100644 foo\n$'
)
def test_installs_hooks_with_hooks_True(
tmpdir_factory,
mock_out_store_directory,
):
path = make_consuming_repo(tmpdir_factory, 'script_hooks_repo')
with local.cwd(path):
install(Runner(path), hooks=True)
ret, output = _get_commit_output(
tmpdir_factory, home=mock_out_store_directory,
)
assert ret == 0
assert PRE_INSTALLED.match(output)