from __future__ import unicode_literals import mock import os import os.path import pytest from plumbum import local from pre_commit.commands.run import _get_skips from pre_commit.commands.run import _has_unmerged_paths from pre_commit.commands.run import run from pre_commit.runner import Runner from testing.auto_namedtuple import auto_namedtuple from testing.fixtures import make_consuming_repo @pytest.yield_fixture def repo_with_passing_hook(tmpdir_factory): git_path = make_consuming_repo(tmpdir_factory, 'script_hooks_repo') with local.cwd(git_path): yield git_path @pytest.yield_fixture def repo_with_failing_hook(tmpdir_factory): git_path = make_consuming_repo(tmpdir_factory, 'failing_hook_repo') with local.cwd(git_path): yield git_path def stage_a_file(): local['touch']('foo.py') local['git']('add', 'foo.py') def get_write_mock_output(write_mock): return ''.join(call[0][0] for call in write_mock.call_args_list) def _get_opts( all_files=False, color=False, verbose=False, hook=None, no_stash=False, ): return auto_namedtuple( all_files=all_files, color=color, verbose=verbose, hook=hook, no_stash=no_stash, ) def _do_run(repo, args, environ={}): runner = Runner(repo) write_mock = mock.Mock() ret = run(runner, args, write=write_mock, environ=environ) printed = get_write_mock_output(write_mock) return ret, printed def _test_run(repo, options, expected_outputs, expected_ret, stage): if stage: stage_a_file() args = _get_opts(**options) ret, printed = _do_run(repo, args) assert ret == expected_ret for expected_output_part in expected_outputs: assert expected_output_part in printed def test_run_all_hooks_failing( repo_with_failing_hook, mock_out_store_directory ): _test_run( repo_with_failing_hook, {}, ('Failing hook', 'Failed', 'Fail\nfoo.py\n'), 1, True, ) @pytest.mark.parametrize( ('options', 'outputs', 'expected_ret', 'stage'), ( ({}, ('Bash hook', 'Passed'), 0, True), ({'verbose': True}, ('foo.py\nHello World',), 0, True), ({'hook': 'bash_hook'}, ('Bash hook', 'Passed'), 0, True), ({'hook': 'nope'}, ('No hook with id `nope`',), 1, True), ( {'all_files': True, 'verbose': True}, ('foo.py'), 0, True, ), ({}, ('Bash hook', '(no files to check)', 'Skipped'), 0, False), ) ) def test_run( repo_with_passing_hook, options, outputs, expected_ret, stage, mock_out_store_directory, ): _test_run(repo_with_passing_hook, options, outputs, expected_ret, stage) @pytest.mark.parametrize( ('no_stash', 'all_files', 'expect_stash'), ( (True, True, False), (True, False, False), (False, True, False), (False, False, True), ), ) def test_no_stash( repo_with_passing_hook, no_stash, all_files, expect_stash, mock_out_store_directory, ): stage_a_file() # Make unstaged changes with open('foo.py', 'w') as foo_file: foo_file.write('import os\n') args = _get_opts(no_stash=no_stash, all_files=all_files) ret, printed = _do_run(repo_with_passing_hook, args) assert ret == 0 warning_msg = '[WARNING] Unstaged files detected.' if expect_stash: assert warning_msg in printed else: assert warning_msg not in printed @pytest.mark.parametrize(('output', 'expected'), (('some', True), ('', False))) def test_has_unmerged_paths(output, expected): mock_runner = mock.Mock() mock_runner.cmd_runner.run.return_value = (1, output, '') assert _has_unmerged_paths(mock_runner) is expected def test_merge_conflict(in_merge_conflict, mock_out_store_directory): ret, printed = _do_run(in_merge_conflict, _get_opts()) assert ret == 1 assert 'Unmerged files. Resolve before committing.' in printed def test_merge_conflict_modified(in_merge_conflict, mock_out_store_directory): # Touch another file so we have unstaged non-conflicting things assert os.path.exists('dummy') with open('dummy', 'w') as dummy_file: dummy_file.write('bar\nbaz\n') ret, printed = _do_run(in_merge_conflict, _get_opts()) assert ret == 1 assert 'Unmerged files. Resolve before committing.' in printed def test_merge_conflict_resolved(in_merge_conflict, mock_out_store_directory): local['git']('add', '.') ret, printed = _do_run(in_merge_conflict, _get_opts()) for msg in ('Checking merge-conflict files only.', 'Bash hook', 'Passed'): assert msg in printed @pytest.mark.parametrize( ('environ', 'expected_output'), ( ({}, set([])), ({'SKIP': ''}, set([])), ({'SKIP': ','}, set([])), ({'SKIP': ',foo'}, set(['foo'])), ({'SKIP': 'foo'}, set(['foo'])), ({'SKIP': 'foo,bar'}, set(['foo', 'bar'])), ({'SKIP': ' foo , bar'}, set(['foo', 'bar'])), ), ) def test_get_skips(environ, expected_output): ret = _get_skips(environ) assert ret == expected_output def test_skip_hook(repo_with_passing_hook, mock_out_store_directory): ret, printed = _do_run( repo_with_passing_hook, _get_opts(), {'SKIP': 'bash_hook'}, ) for msg in ('Bash hook', 'Skipped'): assert msg in printed def test_hook_id_not_in_non_verbose_output( repo_with_passing_hook, mock_out_store_directory ): ret, printed = _do_run(repo_with_passing_hook, _get_opts(verbose=False)) assert '[bash_hook]' not in printed def test_hook_id_in_verbose_output( repo_with_passing_hook, mock_out_store_directory ): ret, printed = _do_run(repo_with_passing_hook, _get_opts(verbose=True)) assert '[bash_hook] Bash hook' in printed