From aa96dbe4e118d8db653289eab844d186ae315d71 Mon Sep 17 00:00:00 2001 From: Felix Fontein Date: Sat, 16 Nov 2024 22:35:15 +0100 Subject: [PATCH] Enforce FQCNs in seealso plugin/module entries. --- .../84325-validate-modules-seealso-fqcn.yml | 2 ++ .../validate-modules/validate_modules/schema.py | 15 +++++++++++++-- 2 files changed, 15 insertions(+), 2 deletions(-) create mode 100644 changelogs/fragments/84325-validate-modules-seealso-fqcn.yml diff --git a/changelogs/fragments/84325-validate-modules-seealso-fqcn.yml b/changelogs/fragments/84325-validate-modules-seealso-fqcn.yml new file mode 100644 index 00000000000..98bb102fb9e --- /dev/null +++ b/changelogs/fragments/84325-validate-modules-seealso-fqcn.yml @@ -0,0 +1,2 @@ +minor_changes: + - "validate-modules sanity test - make sure that ``module`` and ``plugin`` ``seealso`` entries use FQCNs (https://github.com/ansible/ansible/pull/84325)." diff --git a/test/lib/ansible_test/_util/controller/sanity/validate-modules/validate_modules/schema.py b/test/lib/ansible_test/_util/controller/sanity/validate-modules/validate_modules/schema.py index d44553e64ac..abbbd66c25c 100644 --- a/test/lib/ansible_test/_util/controller/sanity/validate-modules/validate_modules/schema.py +++ b/test/lib/ansible_test/_util/controller/sanity/validate-modules/validate_modules/schema.py @@ -67,6 +67,17 @@ def collection_name(v, error_code=None): return v +def fqcn(v, error_code=None): + if not isinstance(v, string_types): + raise _add_ansible_error_code( + Invalid('Module/plugin name must be a string'), error_code or 'invalid-documentation') + m = FULLY_QUALIFIED_COLLECTION_RESOURCE_RE.match(v) + if not m: + raise _add_ansible_error_code( + Invalid('Module/plugin name must be of format `..(.)*`'), error_code or 'invalid-documentation') + return v + + def deprecation_versions(): """Create a list of valid version for deprecation entries, current+4""" major, minor = [int(version) for version in __version__.split('.')[0:2]] @@ -196,11 +207,11 @@ seealso_schema = Schema( [ Any( { - Required('module'): Any(*string_types), + Required('module'): fqcn, 'description': doc_string, }, { - Required('plugin'): Any(*string_types), + Required('plugin'): fqcn, Required('plugin_type'): Any(*DOCUMENTABLE_PLUGINS), 'description': doc_string, },