inspect.getargspec no longer available on Python 3.11 about pynose

[mdmintz]

@adiroiban Thanks for reporting this. I found a workaround: pyinvoke/invoke#833 (comment)

I'll be shipping a new release with the fix very soon.

[adiroiban]

This is the way I am patching it for nose

inspect.getargspec = lambda func: inspect.getfullargspec(func)[:4]

[mdmintz]

@adiroiban If you use:

if not hasattr(inspect, 'getargspec'):
    inspect.getargspec = inspect.getfullargspec

does that work for you? That seems to be the solution that many people liked.

[adiroiban]

As far as I remember, no. It doesn't work.

See the docs https://docs.python.org/3/library/inspect.html#inspect.getfullargspec

getfullargspec returns FullArgSpec(args, varargs, varkw, defaults, kwonlyargs, kwonlydefaults, annotations)

nose usage is args, varargs, varkw, defaults = inspect.getargspec(func)

but nose is only using the args from the return value.

---

Wouldn't it be better to have everything in the pynose namespace ?

That is, instead of patching the vanilla nose, why not continue the development as a separate project?

[mdmintz]

@adiroiban I'll start with the quick patches, and then make more changes as needed.

I'll use the following after importing inspect:

inspect.getargspec = lambda func: inspect.getfullargspec(func)[:4]

If you say that solves your current issue, I can make the necessary changes quickly and push a release within an hour.

[adiroiban]

I don't think that we need to rush.
I already have this working on my fork ... and for practical and security reasons I will always use a fork from a private PyPI server :)

I think is best to update the automated tests to reveal this issue and then push a fix.

Let me give this a try. I will push a PR and I will give you access to the branch.

[mdmintz]

@adiroiban If the only change is:

if not hasattr(inspect, "getargspec"):
    inspect.getargspec = lambda func: inspect.getfullargspec(func)[:4]

I can push that change quickly.

[adiroiban]

I am not sure that is the only change.

But most probably it is.

It would have been nice to have the full nose test available and run those tests, rather than reinventing tests for pynose

Let's check the changes in the PR.

I first would like to see the CI failing and then fix the issue :)

And also add a NEWS.md or some release notes file to keep track of latest changes in the repo :p

[mdmintz]

Resolved in 1.4.5 - https://github.com/mdmintz/pynose/releases/tag/v1.4.5