-
Notifications
You must be signed in to change notification settings - Fork 23.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tests for InventoryModule error conditions #31381
Conversation
@chrismeyersfsu Greetings! Thanks for taking the time to open this pullrequest. In order for the community to handle your pullrequest effectively, we need a bit more information. Here are the items we could not find in your description:
Please set the description of this pullrequest with this template: |
The test
The test
|
CI failure in Python 3.x unit tests: https://app.shippable.com/github/ansible/ansible/runs/39840/4/tests |
@chrismeyersfsu as stated by the bot (#31381 (comment)) you should edit description of this pull request adding:
then Ansibullbot will remove |
Looking to get these unit tests in. What else needs to be done? |
rebuild_merge |
* tests for InventoryModule error conditions * modified unicode in tests to ahear to Ansible best practices * flake8 fixes (cherry picked from commit cf938e9)
SUMMARY
Fix possible
err
not defined when various error conditions are hit on Inventory import and stderr is empty from the importing inventory. Also increase test coverage.ISSUE TYPE
Bugfix Pull Request
COMPONENT NAME
script inventory plugin
ANSIBLE VERSION
2.5