Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Please migrate away from nose for testing #7

Closed
llimeht opened this issue Oct 25, 2021 · 2 comments · Fixed by #9 · May be fixed by #8
Closed

Please migrate away from nose for testing #7

llimeht opened this issue Oct 25, 2021 · 2 comments · Fixed by #9 · May be fixed by #8
Assignees

Comments

@llimeht
Copy link
Contributor

llimeht commented Oct 25, 2021

Apparently nose will soon cease to work for setuptools; while that doesn't directly affect i18nspector, the end is nigh for it in Debian as well, and it's obviously my preference to keep running build-time and autopkgtest tests on i18nspector.

https://lists.debian.org/debian-python/2021/10/msg00058.html

Is there a plan to migrate to a different test harness? The migration looks moderately painful but certainly doable to either unittest or py.test (the latter probably being easier).

@jwilk
Copy link
Owner

jwilk commented Oct 25, 2021

Apparently nose will soon cease to work for setuptools; while that doesn't directly affect i18nspector, the end is nigh for it in Debian as well, and it's obviously my preference to keep running build-time and autopkgtest tests on i18nspector.

https://lists.debian.org/debian-python/2021/10/msg00058.html

To clarify:

Is there a plan to migrate to a different test harness? The migration looks moderately painful but certainly doable to either unittest or py.test (the latter probably being easier).

Maintaining a nose fork (in a life-support style) would be probably less work than migrating away from it, but I'm not excited about either.

For now, I sent patches to the Debian bug. Let's see how it goes.

@jwilk
Copy link
Owner

jwilk commented Aug 11, 2022

Fixed in 0.27.1.

@jwilk jwilk closed this as completed Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants