3 Commits (0892d48ebc94210d3e0526aa425f7fe7a64d1d2f)

Author	SHA1	Message	Date
Dag Wieers	2ef0946370	Warn on incorrect use of parameter, and remove fix for now (#54336) ... * Fix wrong example, remove strictness and fix tests It was obvious that (because of an incorrect example) people were using the **xml** module incorrectly, specifying the `attribute` parameter where it was not supported (i.e. ignored). While this functionality would have been useful, it currently returns as if the information was requested from the parent, so we cannot simply make it to what would be expected. Therefor the real solution is to provide a warning when we find incorrect use, and deprecate this use. Then later we could implement this functionality correctly. While troubleshooting this issue, I found that in some cases our integration tests were not being run when we expected it. This fixes #53459 * Change warning * Fix weird sanity test error * Add a comment to the deprecate-test	6 years ago
Dag Wieers	2634ef955a	xml module: Better change detection, improved tests (#28460) ... This PR includes: - Improvements to change-detection by comparing 2 objectified XML trees - Implement better integration tests by comparing 2 files using copy	7 years ago
Dag Wieers	6874ba23ff	New module: Support manipulating XML files (#25323) ... * Import original unmodified upstream version This is another attempt to get the xml module upstream. https://github.com/cmprescott/ansible-xml/ This is the original file from upstream, without commit 1e7a3f6b6e2bc01aa9cebfd80ac5cd4555032774 * Add additional changes required for upstreaming This PR includes the following changes: - Clean up of DOCUMENTATION - Rename "ensure" parameter to "state" parameter (kept alias) - Added EXAMPLES - Remove explicit type-case using str() for formatting - Clean up AnsibleModule parameter handling - Retained Python 2.4 compatibility - PEP8 compliancy - Various fixes as suggested by abadger during first review This fixes cmprescott/ansible-xml#108 * Added original integration tests There is some room for improvement wrt. idempotency and check-mode testing. * Some tests depend on lxml v3.0alpha1 or higher We are now expecting lxml v2.3.0 or higher. We skips tests if lxml is too old. Plus small fix. * Relicense to GPLv3+ header All past contributors have agreed to relicense this module to GPLv2+, and GPLv3 specifically. See: https://github.com/cmprescott/ansible-xml/issues/113 This fixes cmprescott/ansible-xml#73 * Fix small typo in integration tests * Python 3 support This PR also includes: - Python 3 support - Documentation fixes - Check-mode fixes and improvements - Bugfix in check-mode support - Always return xmlstring, even if there's no change - Check for lxml 2.3.0 or newer * Add return values * Various fixes after review	7 years ago