12 Commits (08683073f9903d1925aba065d455e0d03a4db38a)

Author	SHA1	Message	Date
Lee Garrett	7e3916b767	Typo fixes and other bits and bobs (#83672) ... Co-authored-by: Lee Garrett <lgarrett@rocketjump.eu>	2 years ago
Matt Clay	18e8401edd	Remove Python 2.x compat from unit tests (#82109)	3 years ago
Matt Clay	9f899f9492	Require `from __future__ import annotations` (#81902)	3 years ago
Matt Clay	3a9a23fb1a	Use unittest.mock instead of mock. (#77883) ... This can only be done for controller tests.	4 years ago
Matt Clay	1a5853d794	Remove obsolete units.compat.mock compat layer. (#77118) ... * Remove obsolete units.compat.mock compat layer. * Update remaining units.compat.mock references.	4 years ago
Matt Clay	3033fd96b0	Move unit test compat code out of `lib/ansible/`. (#46996) ... * Move ansible.compat.tests to test/units/compat/. * Fix unit test references to ansible.compat.tests. * Move builtins compat to separate file. * Fix classification of test/units/compat/ dir.	8 years ago
Matt Martz	5c63bb0090	Add test for clean_copy preservation of keys	9 years ago
Brian Coca	01b6c7c9c6	better cleanup on task results display (#27175) ... * better cleanup on task results display callbacks get 'clean' copy of result objects moved cleanup into result object itself removed now redundant callback cleanup moved no_log tests * moved import as per feedback	9 years ago
Dag Wieers	fceb71128e	Return code is not very useful to assume a task failed ... So I thought I fixed it before, but there's still one location where the `rc` value is influential to decide whether a task failed or not. We already established in #24867 that it is up to the module to decide what the return code actually means, not the task executor. We modified the existing modules to move that logic into the module (eg. for command, shell, etc.) This relates to the integration tests of win_robocopy, where different return codes have different meanings: - 0 -- No files copied. - 1 -- Files copied successfully! (changed) - 2 -- Some Extra files or directories were detected. No files were copied. (warning) - 3 -- (2+1) Some files were copied. Additional files were present. (changed) - 4 -- Some mismatched files or directories were detected. Housekeeping might be required! (changed + warning) - 5 -- (4+1) Some files were copied. Some files were mismatched. (changed + warning) - 6 -- (4+2) Additional files and mismatched files exist. No files were copied. (warning) - 7 -- (4+1+2) Files were copied, a file mismatch was present, and additional files were present. (changed + warning) - 8 -- Some files or directories could not be copied! (changed + failed) - 9 - 15 -- Fatal error. Check log message! (failed) - 16 -- Serious Error! No files were copied! Do you have permissions to access $src and $dest? (failed) This also fixes #24652	9 years ago
Dag Wieers	4efec414e7	test/: PEP8 compliancy (#24803) ... * test/: PEP8 compliancy - Make PEP8 compliant * Python3 chokes on casting int to bytes (#24952) But if we tell the formatter that the var is a number, it works	9 years ago
nitzmahone	133395db30	add jimi-c's unit test for squashed skip results, tweaked is_skipped() logic to pass	10 years ago
James Cammarata	81788e627d	Adding unit tests for TaskResult	10 years ago