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

Simplify the skip test to only test skipping

Browse source
This commit is contained in:
Anthony Sottile 2018-12-27 09:24:41 -08:00 • committed by GitHub
parent 8ffd1f69d7
commit 6d40b2a38b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

16
tests/commands/run_test.py
View file

@ -401,27 +401,15 @@ def test_skip_hook(cap_out, store, repo_with_passing_hook):
def test_skip_aliased_hook(cap_out, store, aliased_repo):
ret, printed = _do_run(
cap_out, store, aliased_repo,
run_opts(hook='bash_hook'),
{'SKIP': 'bash_hook'},
)
assert ret == 0
# Both hooks will run since they share the same ID
assert printed.count(b'Bash hook') == 2
for msg in (b'Bash hook', b'Skipped'):
assert msg in printed
ret, printed = _do_run(
cap_out, store, aliased_repo,
run_opts(hook='foo_bash'),
{'SKIP': 'foo_bash'},
)
assert ret == 0
# Only the aliased hook runs
assert printed.count(b'Bash hook') == 1
# Only the aliased hook runs and is skipped
for msg in (b'Bash hook', b'Skipped'):
assert msg in printed, printed
assert printed.count(msg) == 1
def test_hook_id_not_in_non_verbose_output(