gnu: pre-commit: Respect #:tests?.
* gnu/packages/version-control.scm (pre-commit)[arguments]: Respect #:tests? in the 'check phase. Signed-off-by: Efraim Flashner <efraim@flashner.co.il>
This commit is contained in:
parent
a2bcf64383
commit
335f7668d4
|
@ -1552,12 +1552,13 @@ (define-public pre-commit
|
||||||
(invoke "git" "config" "--global" "user.name" "Your Name")
|
(invoke "git" "config" "--global" "user.name" "Your Name")
|
||||||
(invoke "git" "config" "--global" "user.email" "you@example.com")))
|
(invoke "git" "config" "--global" "user.email" "you@example.com")))
|
||||||
(replace 'check
|
(replace 'check
|
||||||
(lambda* (#:key inputs outputs #:allow-other-keys)
|
(lambda* (#:key inputs outputs tests? #:allow-other-keys)
|
||||||
(add-installed-pythonpath inputs outputs)
|
(add-installed-pythonpath inputs outputs)
|
||||||
;; The file below contains about 30 tests that fail because they
|
(when tests?
|
||||||
|
;; The file below contains 30+ tests that fail because they
|
||||||
;; depend on tools from multiple languages (cargo, npm, cpan,
|
;; depend on tools from multiple languages (cargo, npm, cpan,
|
||||||
;; Rscript, etc). There are other tests that pass, but it's more
|
;; Rscript, etc). Other tests are passing, but it's more
|
||||||
;; convenient to skip the whole file than list 30 tests to skip.
|
;; convenient to skip the file than list 30 tests to skip.
|
||||||
(invoke "pytest" "--ignore=tests/repository_test.py"
|
(invoke "pytest" "--ignore=tests/repository_test.py"
|
||||||
;; Ruby and Node tests require node and gem.
|
;; Ruby and Node tests require node and gem.
|
||||||
"--ignore=tests/languages/node_test.py"
|
"--ignore=tests/languages/node_test.py"
|
||||||
|
@ -1571,7 +1572,7 @@ (define-public pre-commit
|
||||||
;; be possible to fix them.
|
;; be possible to fix them.
|
||||||
"not test_install_existing_hooks_no_overwrite"
|
"not test_install_existing_hooks_no_overwrite"
|
||||||
" and not test_uninstall_restores_legacy_hooks"
|
" and not test_uninstall_restores_legacy_hooks"
|
||||||
" and not test_installed_from_venv"))))
|
" and not test_installed_from_venv")))))
|
||||||
(add-before 'reset-gzip-timestamps 'make-gz-writable
|
(add-before 'reset-gzip-timestamps 'make-gz-writable
|
||||||
(lambda* (#:key outputs #:allow-other-keys)
|
(lambda* (#:key outputs #:allow-other-keys)
|
||||||
;; Make sure .gz files are writable so that the
|
;; Make sure .gz files are writable so that the
|
||||||
|
|
Loading…
Reference in a new issue