Add regression test for always_run setting

This commit is contained in:
Anthony Sottile 2015-12-22 12:33:17 -08:00
parent 5d160e1547
commit 4f58f119b1
2 changed files with 8 additions and 2 deletions

View file

@ -184,6 +184,12 @@ def test_run(
_test_run(repo_with_passing_hook, options, outputs, expected_ret, stage)
def test_always_run(repo_with_passing_hook, mock_out_store_directory):
with modify_config() as config:
config[0]['hooks'][0]['always_run'] = True
_test_run(repo_with_passing_hook, {}, (b'Bash hook', b'Passed'), 0, False)
@pytest.mark.parametrize(
('origin', 'source', 'expect_failure'),
(

View file

@ -19,6 +19,7 @@ def manifest(store, tempdir_factory):
def test_manifest_contents(manifest):
# Should just retrieve the manifest contents
assert manifest.manifest_contents == [{
'always_run': False,
'args': [],
'description': '',
'entry': 'bin/hook.sh',
@ -30,12 +31,12 @@ def test_manifest_contents(manifest):
'minimum_pre_commit_version': '0.0.0',
'name': 'Bash hook',
'stages': [],
'always_run': False
}]
def test_hooks(manifest):
assert manifest.hooks['bash_hook'] == {
'always_run': False,
'args': [],
'description': '',
'entry': 'bin/hook.sh',
@ -47,5 +48,4 @@ def test_hooks(manifest):
'minimum_pre_commit_version': '0.0.0',
'name': 'Bash hook',
'stages': [],
'always_run': False
}