Kyle Altendorf

Results 572 comments of Kyle Altendorf

Oh, I forgot about https://github.com/pytest-dev/pytest-twisted/issues/4#issuecomment-360888462. Not as a full solution but rather as existing code related to identifying forgotten things.

I'm not sure what to call it or how `@decorator.decorator` works but... maybe this is a start? ```python3 @decorator.decorator def asyncCallbacks(f, *args, **kwargs): @pytest.inlineCallbacks def wrapper(*args, **kwargs): yield twisted.internet.defer.ensureDeferred(f()) return...

Drop the `@decorator.decorator` stuff. That was a false positive working... Without it it actually runs the tests and passes/fails appropriately.

Then of course the option most like the existing `inlineCallbacks` actually ends up working with fixtures as well. ```python @decorator.decorator def asyncCallbacks(fun, *args, **kw): return twisted.internet.defer.ensureDeferred(fun(*args, **kw)) ``` (someday I'll...

So I think that `ValueError: signal only works in main thread of the main interpreter` is the actually 'interesting' error here. It's funny because lately I've been dealing with this...

Could you try out #154? I think we may also need to tell Twisted to not configure the signals. You should be able to install it directly such as `venv/bin/pip...

``` INTERNALERROR> E File "/Users/andyfreeland/src/pilot/zapgram/.ve/lib/python3.10/site-packages/pytest_twisted.py", line 204, in init_twisted_greenlet INTERNALERROR> E signal.signal( INTERNALERROR> E File "/Users/andyfreeland/.asdf/installs/python/3.10.4/lib/python3.10/signal.py", line 56, in signal INTERNALERROR> E handler = _signal.signal(_enum_to_int(signalnum), _enum_to_int(handler)) INTERNALERROR> E ValueError: signal...

Thanks. That is an intentional warning. pytest can be set to ignore specific warnings in case you like to make all warnings errors. But, this is a situation that nobody...

Should be released in https://pypi.org/project/pytest-twisted/1.14.0/. Sorry for the wait and thanks for the patience.

So at least with `tryFirst=True`, a user provided `pytest_configure()` seems able to effect the change. Despite the exception being painfully generic (each place it is used should inherit from it...