diff options
| author | Felix Fontein <felix@fontein.de> | 2022-11-01 16:12:59 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-11-01 11:12:59 -0400 |
| commit | 6e379e5d656089cdd4bc35c30c5911669e8d59eb (patch) | |
| tree | a199077bc1d096077a4f3cc590542e8aefa49cc0 /test/lib | |
| parent | 6674c43edd203d1241507c4d857f555a7c844c18 (diff) | |
| download | ansible-6e379e5d656089cdd4bc35c30c5911669e8d59eb.tar.gz | |
validate-modules: do not treat falsy non-`False` defaults as `None` (#79267)
* Do not treat falsy non-False defaults as None.
* Fix various instances of this in modules.
* Add changelog fragment.
Diffstat (limited to 'test/lib')
| -rw-r--r-- | test/lib/ansible_test/_util/controller/sanity/validate-modules/validate_modules/main.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/test/lib/ansible_test/_util/controller/sanity/validate-modules/validate_modules/main.py b/test/lib/ansible_test/_util/controller/sanity/validate-modules/validate_modules/main.py index 9513ed30b1..ce440bca1f 100644 --- a/test/lib/ansible_test/_util/controller/sanity/validate-modules/validate_modules/main.py +++ b/test/lib/ansible_test/_util/controller/sanity/validate-modules/validate_modules/main.py @@ -79,7 +79,7 @@ from .module_args import AnsibleModuleImportError, AnsibleModuleNotInitialized, from .schema import ansible_module_kwargs_schema, doc_schema, return_schema -from .utils import CaptureStd, NoArgsAnsibleModule, compare_unordered_lists, is_empty, parse_yaml, parse_isodate +from .utils import CaptureStd, NoArgsAnsibleModule, compare_unordered_lists, parse_yaml, parse_isodate if PY3: @@ -1789,7 +1789,7 @@ class ModuleValidator(Validator): ) arg_default = None - if 'default' in data and not is_empty(data['default']): + if 'default' in data and data['default'] is not None: try: with CaptureStd(): arg_default = _type_checker(data['default']) @@ -1830,7 +1830,7 @@ class ModuleValidator(Validator): try: doc_default = None - if 'default' in doc_options_arg and not is_empty(doc_options_arg['default']): + if 'default' in doc_options_arg and doc_options_arg['default'] is not None: with CaptureStd(): doc_default = _type_checker(doc_options_arg['default']) except (Exception, SystemExit): |