mirror of https://github.com/ansible/ansible.git
You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
e2a57414f4
As per: https://github.com/pytest-dev/pytest-mock#note-about-usage-as-context-manager pytest-mock is not meant to be used within a `with` context or as a decorator. Instead, pytest-mock will automatically unpatch the mocked methods when each test is complete. In newer pytest-mock, this use actually throws an exception and causes the tests to fail. This hasn't been hit in Ansible's CI yet, because the docker image that the tests run in uses an older version of pytest-mock. However, there is no constraint on the upper bound of pytest-mock in test/lib/ansible_test/_data/requirements/constraints.txt which means that when running the tests locally, outside of that docker image, the tests never pass. This patch removes the `with` context in each such case. Signed-off-by: Rick Elrod <rick@elrod.me> |
5 years ago | |
|---|---|---|
| .. | ||
| cli | 5 years ago | |
| compat | 6 years ago | |
| config | 5 years ago | |
| contrib | 6 years ago | |
| errors | 5 years ago | |
| executor | 5 years ago | |
| galaxy | 5 years ago | |
| inventory_test_data/group_vars | ||
| mock | 5 years ago | |
| module_utils | 5 years ago | |
| modules | 5 years ago | |
| parsing | 5 years ago | |
| playbook | 5 years ago | |
| plugins | 5 years ago | |
| regex | 6 years ago | |
| template | 5 years ago | |
| utils | 5 years ago | |
| vars | 5 years ago | |
| __init__.py | ||
| requirements.txt | 5 years ago | |
| test_constants.py | ||
| test_context.py | 5 years ago | |