Skip to content

remove automatic monkey-patching#727

Merged
mattbennett merged 8 commits intonameko:masterfrom
mattbennett:disable-auto-monkeypatch
Oct 13, 2021
Merged

remove automatic monkey-patching#727
mattbennett merged 8 commits intonameko:masterfrom
mattbennett:disable-auto-monkeypatch

Conversation

@mattbennett
Copy link
Copy Markdown
Member

@mattbennett mattbennett commented Oct 5, 2021

Resolves #703

  • removes automatic monkey-patching in the pytest plugin
  • adds nameko test command which wraps pytest, automatically applies eventlet monkeypatch before invoking pytest
  • warns users if the monkeypatch is not applied when the test session starts
  • suggests using nameko test or the pytest-eventlet plugin as an alternative

(resolves nameko#703)

* warns users if the monkeypatch is not applied
* suggests using pytest-eventlet as an alternative
+ update warning
+ move nameko run patch into cli so it is earlier and more explicit
@mattbennett mattbennett force-pushed the disable-auto-monkeypatch branch from b03bb55 to d72e851 Compare October 11, 2021 12:38
@mattbennett mattbennett force-pushed the disable-auto-monkeypatch branch from e4f56e9 to 5a5e135 Compare October 11, 2021 12:44
Copy link
Copy Markdown
Contributor

@iky iky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resolves #703

  • warns users if the monkeypatch is not applied
  • suggests using pytest-eventlet as an alternative

To complete the change description one bit is missing:

  • adds nameko test command which wraps pytest, automatically applies eventlet monkeypatch before invoking pytest

@mattbennett mattbennett merged commit 65d7a50 into nameko:master Oct 13, 2021
@mattbennett mattbennett deleted the disable-auto-monkeypatch branch October 13, 2021 16:07
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 this pull request may close these issues.

pytest plugin runs even when irrelevant

2 participants