diff options
author | Ludovic Courtès <ludo@gnu.org> | 2021-11-19 21:27:18 +0100 |
---|---|---|
committer | Ludovic Courtès <ludo@gnu.org> | 2021-11-19 21:27:18 +0100 |
commit | 7c9801e857d2afed6187645b2139facf61a4204b (patch) | |
tree | 0cfbbcd000a575afd187923e9ab3fae9c85b4358 /gnu/packages | |
parent | d55ed2725e5f35bb3fdbd6504daa69b9959325a6 (diff) | |
download | guix-patches-7c9801e857d2afed6187645b2139facf61a4204b.tar guix-patches-7c9801e857d2afed6187645b2139facf61a4204b.tar.gz |
gnu: python-notebook: Fix tests that rely on the gio "trash" mechanism.
Previously 'test_delete' and a couple of related tests in
'test_content_api.py' would fail:
https://lists.gnu.org/archive/html/guix-devel/2021-11/msg00083.html
Reported by Ricardo Wurmus.
* gnu/packages/python-xyz.scm (python-notebook)[arguments]: Add
'use-our-home-for-tests' phase. In 'check' phase, remove call to
'setenv' and unnecessary use of 'with-directory-excursion'.
Diffstat (limited to 'gnu/packages')
-rw-r--r-- | gnu/packages/python-xyz.scm | 23 |
1 files changed, 16 insertions, 7 deletions
diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm index e882659d4b..3f314ec143 100644 --- a/gnu/packages/python-xyz.scm +++ b/gnu/packages/python-xyz.scm @@ -12051,6 +12051,18 @@ convert an @code{.ipynb} notebook file into various static formats including: (arguments `(#:phases (modify-phases %standard-phases + (add-after 'unpack 'use-our-home-for-tests + (lambda _ + ;; The 'get_patch_env' function in this file reads: + ;; 'HOME': cls.home_dir + ;; but for some reason, that definition of HOME is not what the + ;; GLib/GIO trash mechanism honors, which would cause test + ;; failures. Instead, set 'HOME' here to an existing directory + ;; and let the tests honor it. + (substitute* "notebook/tests/launchnotebook.py" + (("'HOME': .*," all) + (string-append "# " all "\n"))) + (setenv "HOME" (getcwd)))) (replace 'check (lambda* (#:key tests? inputs outputs #:allow-other-keys) ;; These tests require a browser @@ -12060,13 +12072,10 @@ convert an @code{.ipynb} notebook file into various static formats including: ;; Some tests do not expect all files to be installed in the ;; same directory, but JUPYTER_PATH contains multiple entries. (unsetenv "JUPYTER_PATH") - ;; Some tests need HOME - (setenv "HOME" "/tmp") - (with-directory-excursion "/tmp" - (invoke "pytest" "-vv" - ;; TODO: This tests fails because nbconvert does not - ;; list "python" as a format. - "-k" "not test_list_formats")))))))) + (invoke "pytest" "-vv" + ;; TODO: This tests fails because nbconvert does not + ;; list "python" as a format. + "-k" "not test_list_formats"))))))) (propagated-inputs `(("python-argon2-cffi" ,python-argon2-cffi) ("python-ipykernel" ,python-ipykernel) |