adjust the run_hook api to no longer take Hook

tests/languages/helpers_test.py

@@ -12,7 +12,6 @@ from pre_commit import parse_shebang
 from pre_commit.languages import helpers
 from pre_commit.prefix import Prefix
 from pre_commit.util import CalledProcessError
-from testing.auto_namedtuple import auto_namedtuple
 
 
 @pytest.fixture
@@ -94,31 +93,22 @@ def test_assert_no_additional_deps():
     )
 
 
-SERIAL_FALSE = auto_namedtuple(require_serial=False)
-SERIAL_TRUE = auto_namedtuple(require_serial=True)
-
-
 def test_target_concurrency_normal():
     with mock.patch.object(multiprocessing, 'cpu_count', return_value=123):
         with mock.patch.dict(os.environ, {}, clear=True):
-            assert helpers.target_concurrency(SERIAL_FALSE) == 123
-
-
-def test_target_concurrency_cpu_count_require_serial_true():
-    with mock.patch.dict(os.environ, {}, clear=True):
-        assert helpers.target_concurrency(SERIAL_TRUE) == 1
+            assert helpers.target_concurrency() == 123
 
 
 def test_target_concurrency_testing_env_var():
     with mock.patch.dict(
             os.environ, {'PRE_COMMIT_NO_CONCURRENCY': '1'}, clear=True,
     ):
-        assert helpers.target_concurrency(SERIAL_FALSE) == 1
+        assert helpers.target_concurrency() == 1
 
 
 def test_target_concurrency_on_travis():
     with mock.patch.dict(os.environ, {'TRAVIS': '1'}, clear=True):
-        assert helpers.target_concurrency(SERIAL_FALSE) == 2
+        assert helpers.target_concurrency() == 2
 
 
 def test_target_concurrency_cpu_count_not_implemented():
@@ -126,10 +116,20 @@ def test_target_concurrency_cpu_count_not_implemented():
             multiprocessing, 'cpu_count', side_effect=NotImplementedError,
     ):
         with mock.patch.dict(os.environ, {}, clear=True):
-            assert helpers.target_concurrency(SERIAL_FALSE) == 1
+            assert helpers.target_concurrency() == 1
 
 
 def test_shuffled_is_deterministic():
     seq = [str(i) for i in range(10)]
     expected = ['4', '0', '5', '1', '8', '6', '2', '3', '7', '9']
     assert helpers._shuffled(seq) == expected
+
+
+def test_xargs_require_serial_is_not_shuffled():
+    ret, out = helpers.run_xargs(
+        ('echo',), [str(i) for i in range(10)],
+        require_serial=True,
+        color=False,
+    )
+    assert ret == 0
+    assert out.strip() == b'0 1 2 3 4 5 6 7 8 9'

tests/languages/r_test.py

@@ -23,7 +23,7 @@ def _test_r_parsing(
     repo = make_repo(tempdir_factory, 'r_hooks_repo')
     config = make_config_from_repo(repo)
     hook = _get_hook_no_install(config, store, hook_id)
-    ret = r._cmd_from_hook(hook)
+    ret = r._cmd_from_hook(hook.prefix, hook.entry, hook.args)
     expected_path = os.path.join(hook.prefix.prefix_dir, f'{hook_id}.R')
     expected = (
         'Rscript',
@@ -111,7 +111,7 @@ def test_r_parsing_file_local(tempdir_factory, store):
         }],
     }
     hook = _get_hook_no_install(config, store, 'local-r')
-    ret = r._cmd_from_hook(hook)
+    ret = r._cmd_from_hook(hook.prefix, hook.entry, hook.args)
     assert ret == (
         'Rscript',
         '--no-save', '--no-restore', '--no-site-file', '--no-environ',

tests/repository_test.py

@@ -45,7 +45,14 @@ def _norm_out(b):
 
 def _hook_run(hook, filenames, color):
     with languages[hook.language].in_env(hook.prefix, hook.language_version):
-        return languages[hook.language].run_hook(hook, filenames, color)
+        return languages[hook.language].run_hook(
+            hook.prefix,
+            hook.entry,
+            hook.args,
+            filenames,
+            require_serial=hook.require_serial,
+            color=color,
+        )
 
 
 def _get_hook_no_install(repo_config, store, hook_id):