Séamus Ó Ceanainn
Séamus Ó Ceanainn
@ruurtjan sorry for missing this until now. I definitely see the value in the changes you suggested, however I would suggest having this as a configurable option as opposed to...
Actually thinking about it more carefully - I was wrong to suggest that pytest-mock could be updated to solve this. Extending pytest-mock to allow patching with non-mock objects wouldn't make...
There are a lot of potential configuration options for configuring the admin-client properties, and it would be a lot of work to maintain the ability to set any of them...
@auvipy tagging you here as you had volunteered to review any contributions to the original issue (although I understand that was 18 months ago)
Okay I see the issue in tests now: 1. Celery CLI is actually configured by explicitly setting these env variables: https://github.com/celery/celery/blob/1aff856ea37a477639ef2c8883f121d6670f72e0/celery/bin/celery.py#L152-L162 2. This is causing issues in tests due to...