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.
ansible/test/units/plugins/lookup
Rick Elrod e2a57414f4 Remove `with` statement for pytest-mock unit tests
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
..
fixtures Adding Avi ansible lookup module (#58667) 5 years ago
__init__.py
test_avi.py Adding Avi ansible lookup module (#58667) 5 years ago
test_aws_secret.py Remove `with` statement for pytest-mock unit tests 5 years ago
test_aws_ssm.py Remove `with` statement for pytest-mock unit tests 5 years ago
test_conjur_variable.py
test_env.py Fix env lookup plugin error on utf8 values (#65541) 5 years ago
test_ini.py Remove empty setUp/tearDown/tearDownClass methods in test classes. 5 years ago
test_laps_password.py missing_required_lib: add information how to change Python interpreter (#60736) 5 years ago
test_lastpass.py Remove empty setUp/tearDown/tearDownClass methods in test classes. 5 years ago
test_manifold.py
test_onepassword.py
test_password.py Replace TestCase.assertEquals with TestCase.assertEqual. 5 years ago