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 #51 from pre-commit/fix_leaky_prefixed_command_runner_test

Browse source 
Fix prefixed_command_runner_test to not create real directories.
This commit is contained in:
Anthony Sottile 2014-03-30 14:05:01 -07:00
commit 02660f7c0a
3 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

6
pre_commit/prefixed_command_runner.py
View file

@ -79,4 +79,8 @@ class PrefixedCommandRunner(object):
"""Constructs a new command runner from an existing one by appending """Constructs a new command runner from an existing one by appending
`path_end` to the command runner's prefix directory. `path_end` to the command runner's prefix directory.
""" """
return cls(command_runner.path(path_end), popen=command_runner.__popen) return cls(
command_runner.path(path_end),
popen=command_runner.__popen,
makedirs=command_runner.__makedirs,
)

0
tests/hooks_workspace_test.py
View file

1
tests/prefixed_command_runner_test.py
View file

@ -107,6 +107,7 @@ def test_from_command_runner_preserves_popen(popen_mock, makedirs_mock):
stdout=subprocess.PIPE, stdout=subprocess.PIPE,
stderr=subprocess.PIPE, stderr=subprocess.PIPE,
) )
makedirs_mock.assert_called_once_with('foo/bar/')
def test_create_path_if_not_exists(tmpdir): def test_create_path_if_not_exists(tmpdir):