Remove dependency on nose (port to pytest)

[mcepl]

I am a maintainer of Python packages in openSUSE, and I am on my crusade of eliminating nose1 from our distribution. When I look at its repository on https://github.com/nose-devs/nose, the last release 1.3.7 was on 2 Jun 2015, and even the last commit on the master branch was on 4 Mar 2016.

This patch eliminates dependency on nose, which has been unnecessary for a long time. Resulting test suite depends on pytest, which is currently perfectly developed and maintained.

[Carreau]

Many thanks for working on this. I'll try to integrate this and might do a release. Thought it may take some time. Another possibility would be to finish removing usage of this package from the dependees.

[mcepl]

I work for SUSE, which means we are working on hundreds of packages, so I cannot concentrate on one particular package for too long … my current enemy is nose, and so I eliminate it. Concerning dependees, given this whole package has been already deprecated, it seems weird how stubbornly it refuses to die.

[Carreau]

Concerning dependees, given this whole package has been already deprecated, it seems weird how stubbornly it refuses to die.

Sorry, this was not a comment for you to do this; I am a maintainer of (all?) the direct dependees that use ipython_genutils, and more a comment that I might go the route of removing ipython_genutils as a dependencies from those (so you can drop ipython_genutils from your list of packages) instead of publishing a new release if ipython_genutils. I might also do both depending on the timeline.

Updating the dependees might also be a better route as some of them still require nose1 IIRC (which is one of the things i've also been getting rid of progressively).

To be clearer, I have commit rights to remove and publish new version of the following packages to not use ipython_genutils:

ipython
ipykernel
ipyparallel
ipython_genutils
jupyter_client
jupyter_console
jupyter_core
jupyter_server
nbconvert
nbformat
notebook
qtconsole
traitlets

A few of those also directly use nose, and I'll try to remove nose usage as well, but will take time.

[mcepl]

Looking through openSUSE packages and it seems to me that some packages you mentioned really don’t require nose:

• python-ipython testing still depends on nose
• python-ipykernel testing still depends on nose
• python-ipyparallel runs pytest without any patch
• python-ipython_genutils pytest with this patch
• python-jupyter_client pytest without any patch
• python-jupyter_console pytest with the patch from do not depend on nose in test_console.py jupyter/jupyter_console#230
• python-jupyter_core running with pytest
• python-jupyter_server running with pytest
• python-nbconvert runs with pytest, with patch from Mark tests requiring network access jupyter/nbconvert#1526
• python-nbformat uses pytest even without any patching (we have excluded two test classes, but hopefully at least partially meaningful test suite remains)
• python-notebook using pytest with patch sent upstream as do not require nose for testing jupyter/notebook#5826
• python-qtconsole pytest w/o patch (with some tests skipped according to Python segmentation fault with test_scroll and test_debug jupyter/qtconsole#443)
• python-traitlets ditto (running pushd doc && pytest -v ../traitlets/tests)

[Carreau]

Yeah, my concern is that some of the test are skipped on pytest. I'll see what I can do to help with this migration. Feel free to drop links to upstream issue I'm happy to mention there when the upstream patches have been merged or releases have been done.

[mcepl]

@Carreau ipython/ipython#12840

[Carreau]

jupyter/jupyter_client#622 might also help to remove genutils as a dependencies of Jupyter_client.

[opoplawski]

Any more progress here? We're probably not going to introduce nose into EPEL9. Still a number of packages using ipython_genutils.

[mcepl]

Any more progress here? We're probably not going to introduce nose into EPEL9. Still a number of packages using ipython_genutils.

https://build.opensuse.org/package/view_file/devel:languages:python:jupyter/python-ipython_genutils/denose.patch?expand=1
works still just fine for us.

Our ipython packages don’t need nose anymore.

[hroncok]

https://build.opensuse.org/package/view_file/devel:languages:python:jupyter/python-ipython_genutils/denose.patch?expand=1 works still just fine for us.

This runs fewer tests (16+1 skip -> 11+1 skip).

I get:

ipython_genutils/tests/test_path.py:33: PytestCollectionWarning: cannot collect test class 'TestLinkOrCopy' because it has a __init__ constructor (from: ipython_genutils/tests/test_path.py)

This fixes it:

diff --git a/ipython_genutils/tests/test_path.py b/ipython_genutils/tests/test_path.py
index cecabc6..0f1a0bb 100644
--- a/ipython_genutils/tests/test_path.py
+++ b/ipython_genutils/tests/test_path.py
@@ -31,17 +31,13 @@ def test_ensure_dir_exists():
 
 
 class TestLinkOrCopy(object):
-    def __init__(self):
-        self.tempdir = None
-        self.src = None
-
-    def setUp(self):
+    def setup_method(self):
         self.tempdir = tempfile.TemporaryDirectory()
         self.src = self.dst("src")
         with open(self.src, "w") as f:
             f.write("Hello, world!")
 
-    def tearDown(self):
+    def teardown_method(self):
         self.tempdir.cleanup()
 
     def dst(self, *args):

[Carreau]

Which package are you using which depends in ipython_genutil ? it is likely it is still mark as a dependency but should not be for ~7 years.

[mcepl]

• https://github.com/matplotlib/ipympl, but it may get fixed as well soon (matplotlib/ipympl@564cf6d).
• https://github.com/jupyter/nbclassic and there the dependency is doing just fine, apparently.

[Carreau]

It looks like sage and spyder (in a vendored dependency that needs update) still reference it.

[Carreau]

$ conda repoquery whoneeds  -c conda-forge ipython_genutils

Can help find the current conda packages that list ipytho_genutils as a dependency; it is worth looking as wether they are actual dependencies, and wether the latest versions still use it.