Merge pull request #474 from pre-commit/go_passes_on_windows

The golang tests pass on windows
This commit is contained in:
Anthony Sottile 2017-01-24 21:41:41 -08:00 committed by GitHub
commit cc1eac46e4

View file

@ -15,14 +15,14 @@ from pre_commit.xargs import xargs
ENVIRONMENT_DIR = 'golangenv' ENVIRONMENT_DIR = 'golangenv'
def get_env_patch(venv): # pragma: windows no cover def get_env_patch(venv):
return ( return (
('PATH', (os.path.join(venv, 'bin'), os.pathsep, Var('PATH'))), ('PATH', (os.path.join(venv, 'bin'), os.pathsep, Var('PATH'))),
) )
@contextlib.contextmanager @contextlib.contextmanager
def in_env(repo_cmd_runner): # pragma: windows no cover def in_env(repo_cmd_runner):
envdir = repo_cmd_runner.path( envdir = repo_cmd_runner.path(
helpers.environment_dir(ENVIRONMENT_DIR, 'default'), helpers.environment_dir(ENVIRONMENT_DIR, 'default'),
) )
@ -47,7 +47,7 @@ def install_environment(
repo_cmd_runner, repo_cmd_runner,
version='default', version='default',
additional_dependencies=(), additional_dependencies=(),
): # pragma: windows no cover ):
helpers.assert_version_default('golang', version) helpers.assert_version_default('golang', version)
helpers.assert_no_additional_deps('golang', additional_dependencies) helpers.assert_no_additional_deps('golang', additional_dependencies)
directory = repo_cmd_runner.path( directory = repo_cmd_runner.path(
@ -67,6 +67,6 @@ def install_environment(
cmd_output('go', 'get', './...', cwd=repo_src_dir, env=env) cmd_output('go', 'get', './...', cwd=repo_src_dir, env=env)
def run_hook(repo_cmd_runner, hook, file_args): # pragma: windows no cover def run_hook(repo_cmd_runner, hook, file_args):
with in_env(repo_cmd_runner): with in_env(repo_cmd_runner):
return xargs(helpers.to_cmd(hook), file_args) return xargs(helpers.to_cmd(hook), file_args)