diff --git a/tests/forbid_new_submodules_test.py b/tests/forbid_new_submodules_test.py index dc64f067..523628d6 100644 --- a/tests/forbid_new_submodules_test.py +++ b/tests/forbid_new_submodules_test.py @@ -1,7 +1,8 @@ from __future__ import absolute_import +import subprocess + import pytest -from pre_commit.util import cmd_output from pre_commit_hooks.forbid_new_submodules import main @@ -9,13 +10,13 @@ @pytest.fixture def git_dir_with_git_dir(tmpdir): with tmpdir.as_cwd(): - cmd_output('git', 'init', '.') - cmd_output( - 'git', 'commit', '-m', 'init', '--allow-empty', '--no-gpg-sign', - ) - cmd_output('git', 'init', 'foo') - cmd_output( + subprocess.check_call(('git', 'init', '.')) + subprocess.check_call(( 'git', 'commit', '-m', 'init', '--allow-empty', '--no-gpg-sign', + )) + subprocess.check_call(('git', 'init', 'foo')) + subprocess.check_call( + ('git', 'commit', '-m', 'init', '--allow-empty', '--no-gpg-sign'), cwd=tmpdir.join('foo').strpath, ) yield @@ -31,7 +32,7 @@ def git_dir_with_git_dir(tmpdir): ), ) def test_main_new_submodule(git_dir_with_git_dir, capsys, cmd): - cmd_output(*cmd) + subprocess.check_call(cmd) assert main() == 1 out, _ = capsys.readouterr() assert out.startswith('foo: new submodule introduced\n') @@ -39,5 +40,5 @@ def test_main_new_submodule(git_dir_with_git_dir, capsys, cmd): def test_main_no_new_submodule(git_dir_with_git_dir): open('test.py', 'a+').close() - cmd_output('git', 'add', 'test.py') + subprocess.check_call(('git', 'add', 'test.py')) assert main() == 0