summaryrefslogtreecommitdiffstats
path: root/tests/languages/r_test.py
diff options
context:
space:
mode:
authorDaniel Baumann <daniel.baumann@progress-linux.org>2021-03-08 17:48:47 +0000
committerDaniel Baumann <daniel.baumann@progress-linux.org>2021-03-08 17:48:47 +0000
commit4db15c5b771323f1acdeb06e4acc671856da5ea9 (patch)
tree1cde2767753275f2f87a34a2606b2415a467ff09 /tests/languages/r_test.py
parentAdding upstream version 2.10.1. (diff)
downloadpre-commit-4db15c5b771323f1acdeb06e4acc671856da5ea9.tar.xz
pre-commit-4db15c5b771323f1acdeb06e4acc671856da5ea9.zip
Adding upstream version 2.11.0.upstream/2.11.0
Signed-off-by: Daniel Baumann <daniel.baumann@progress-linux.org>
Diffstat (limited to 'tests/languages/r_test.py')
-rw-r--r--tests/languages/r_test.py104
1 files changed, 104 insertions, 0 deletions
diff --git a/tests/languages/r_test.py b/tests/languages/r_test.py
new file mode 100644
index 0000000..5c046ef
--- /dev/null
+++ b/tests/languages/r_test.py
@@ -0,0 +1,104 @@
+import os.path
+
+import pytest
+
+from pre_commit.languages import r
+from testing.fixtures import make_config_from_repo
+from testing.fixtures import make_repo
+from tests.repository_test import _get_hook_no_install
+
+
+def _test_r_parsing(
+ tempdir_factory,
+ store,
+ hook_id,
+ expected_hook_expr={},
+ expected_args={},
+):
+ repo_path = 'r_hooks_repo'
+ path = make_repo(tempdir_factory, repo_path)
+ config = make_config_from_repo(path)
+ hook = _get_hook_no_install(config, store, hook_id)
+ ret = r._cmd_from_hook(hook)
+ expected_cmd = 'Rscript'
+ expected_opts = (
+ '--no-save', '--no-restore', '--no-site-file', '--no-environ',
+ )
+ expected_path = os.path.join(
+ hook.prefix.prefix_dir, '.'.join([hook_id, 'R']),
+ )
+ expected = (
+ expected_cmd,
+ *expected_opts,
+ *(expected_hook_expr or (expected_path,)),
+ *expected_args,
+ )
+ assert ret == expected
+
+
+def test_r_parsing_file_no_opts_no_args(tempdir_factory, store):
+ hook_id = 'parse-file-no-opts-no-args'
+ _test_r_parsing(tempdir_factory, store, hook_id)
+
+
+def test_r_parsing_file_opts_no_args(tempdir_factory, store):
+ with pytest.raises(ValueError) as excinfo:
+ r._entry_validate(['Rscript', '--no-init', '/path/to/file'])
+
+ msg = excinfo.value.args
+ assert msg == (
+ 'The only valid syntax is `Rscript -e {expr}`',
+ 'or `Rscript path/to/hook/script`',
+ )
+
+
+def test_r_parsing_file_no_opts_args(tempdir_factory, store):
+ hook_id = 'parse-file-no-opts-args'
+ expected_args = ['--no-cache']
+ _test_r_parsing(
+ tempdir_factory, store, hook_id, expected_args=expected_args,
+ )
+
+
+def test_r_parsing_expr_no_opts_no_args1(tempdir_factory, store):
+ hook_id = 'parse-expr-no-opts-no-args-1'
+ _test_r_parsing(
+ tempdir_factory, store, hook_id, expected_hook_expr=('-e', '1+1'),
+ )
+
+
+def test_r_parsing_expr_no_opts_no_args2(tempdir_factory, store):
+ with pytest.raises(ValueError) as execinfo:
+ r._entry_validate(['Rscript', '-e', '1+1', '-e', 'letters'])
+ msg = execinfo.value.args
+ assert msg == ('You can supply at most one expression.',)
+
+
+def test_r_parsing_expr_opts_no_args2(tempdir_factory, store):
+ with pytest.raises(ValueError) as execinfo:
+ r._entry_validate(
+ [
+ 'Rscript', '--vanilla', '-e', '1+1', '-e', 'letters',
+ ],
+ )
+ msg = execinfo.value.args
+ assert msg == (
+ 'The only valid syntax is `Rscript -e {expr}`',
+ 'or `Rscript path/to/hook/script`',
+ )
+
+
+def test_r_parsing_expr_args_in_entry2(tempdir_factory, store):
+ with pytest.raises(ValueError) as execinfo:
+ r._entry_validate(['Rscript', '-e', 'expr1', '--another-arg'])
+
+ msg = execinfo.value.args
+ assert msg == ('You can supply at most one expression.',)
+
+
+def test_r_parsing_expr_non_Rscirpt(tempdir_factory, store):
+ with pytest.raises(ValueError) as execinfo:
+ r._entry_validate(['AnotherScript', '-e', '{{}}'])
+
+ msg = execinfo.value.args
+ assert msg == ('entry must start with `Rscript`.',)