Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix TestInventory Unit Tests #6319

Merged
merged 4 commits into from Mar 6, 2014

Conversation

Projects
None yet
2 participants
Contributor

risaacson commented Mar 6, 2014

Uncomment test_dir_inventory and add commits from g-k-r.

Minor updating made to inventory parsing to get the test cases to work. See 16fe09e for more details.

risaacson and others added some commits Mar 6, 2014

Fixes related to uncommenting test_dir_inventory in TestInventory.
0. Uncomment the test.
1. Test fails.
2. Make vars unique per file in test inventory files.
3. Modify token addition to not ast.literal_eval(v) a variable containing a hash.
4. Modify vars to have an escape in test inventory file.
5. Catch exceptions explicitly. Any unknown exceptions should be a bug.
6. Test passes.
add test same host in different files in dir
tests issue #5749
 same host defined in different groups which in turn are defined
 in different ini files in an inventory directory

Conflicts:
	test/units/TestInventory.py
modifed test to use get_hosts instead of get_groups
closes #5749

Conflicts:
	test/units/TestInventory.py
Formatting cleanup.
Post merging of GH-5837 via GH-5749.

risaacson added a commit that referenced this pull request Mar 6, 2014

Merge pull request #6319 from risaacson/fix_test_dir_inventory
Fix TestInventory Unit Tests

Tested clean.

@risaacson risaacson merged commit 119434d into ansible:devel Mar 6, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment