2017-08-19 02:21:11 +02:00
|
|
|
"""Sanity test for ansible-doc."""
|
|
|
|
from __future__ import absolute_import, print_function
|
|
|
|
|
2018-05-23 15:48:22 +02:00
|
|
|
import collections
|
|
|
|
import os
|
2018-01-17 19:53:03 +01:00
|
|
|
import re
|
|
|
|
|
2017-08-19 02:21:11 +02:00
|
|
|
from lib.sanity import (
|
|
|
|
SanityMultipleVersion,
|
|
|
|
SanityFailure,
|
|
|
|
SanitySuccess,
|
|
|
|
SanitySkipped,
|
2018-01-17 19:53:03 +01:00
|
|
|
SanityMessage,
|
2017-08-19 02:21:11 +02:00
|
|
|
)
|
|
|
|
|
|
|
|
from lib.util import (
|
|
|
|
SubprocessError,
|
|
|
|
display,
|
2017-10-26 09:21:46 +02:00
|
|
|
intercept_command,
|
2018-09-20 08:20:27 +02:00
|
|
|
read_lines_without_comments,
|
2017-08-19 02:21:11 +02:00
|
|
|
)
|
|
|
|
|
|
|
|
from lib.ansible_util import (
|
|
|
|
ansible_environment,
|
|
|
|
)
|
|
|
|
|
|
|
|
from lib.config import (
|
|
|
|
SanityConfig,
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
|
|
class AnsibleDocTest(SanityMultipleVersion):
|
|
|
|
"""Sanity test for ansible-doc."""
|
|
|
|
def test(self, args, targets, python_version):
|
|
|
|
"""
|
|
|
|
:type args: SanityConfig
|
|
|
|
:type targets: SanityTargets
|
|
|
|
:type python_version: str
|
2018-04-04 03:53:53 +02:00
|
|
|
:rtype: TestResult
|
2017-08-19 02:21:11 +02:00
|
|
|
"""
|
2018-09-20 08:20:27 +02:00
|
|
|
skip_file = 'test/sanity/ansible-doc/skip.txt'
|
|
|
|
skip_modules = set(read_lines_without_comments(skip_file, remove_blank_lines=True))
|
2017-08-19 02:21:11 +02:00
|
|
|
|
2019-01-24 02:03:47 +01:00
|
|
|
# This should use documentable plugins from constants instead
|
2018-05-23 15:48:22 +02:00
|
|
|
plugin_type_blacklist = set([
|
|
|
|
# not supported by ansible-doc
|
|
|
|
'action',
|
2019-01-24 02:03:47 +01:00
|
|
|
'doc_fragments',
|
2018-05-23 15:48:22 +02:00
|
|
|
'cliconf',
|
|
|
|
'filter',
|
|
|
|
'httpapi',
|
|
|
|
'netconf',
|
|
|
|
'terminal',
|
|
|
|
'test',
|
|
|
|
])
|
|
|
|
|
2017-08-19 02:21:11 +02:00
|
|
|
modules = sorted(set(m for i in targets.include_external for m in i.modules) -
|
|
|
|
set(m for i in targets.exclude_external for m in i.modules) -
|
|
|
|
skip_modules)
|
|
|
|
|
2018-05-23 15:48:22 +02:00
|
|
|
plugins = [os.path.splitext(i.path)[0].split('/')[-2:] + [i.path] for i in targets.include if os.path.splitext(i.path)[1] == '.py' and
|
|
|
|
os.path.basename(i.path) != '__init__.py' and
|
|
|
|
re.search(r'^lib/ansible/plugins/[^/]+/', i.path)
|
|
|
|
and i.path != 'lib/ansible/plugins/cache/base.py']
|
|
|
|
|
|
|
|
doc_targets = collections.defaultdict(list)
|
|
|
|
target_paths = collections.defaultdict(dict)
|
|
|
|
|
|
|
|
for module in modules:
|
|
|
|
doc_targets['module'].append(module)
|
|
|
|
|
|
|
|
for plugin_type, plugin_name, plugin_path in plugins:
|
|
|
|
if plugin_type in plugin_type_blacklist:
|
|
|
|
continue
|
|
|
|
|
|
|
|
doc_targets[plugin_type].append(plugin_name)
|
|
|
|
target_paths[plugin_type][plugin_name] = plugin_path
|
|
|
|
|
|
|
|
if not doc_targets:
|
2017-08-19 02:21:11 +02:00
|
|
|
return SanitySkipped(self.name, python_version=python_version)
|
2017-11-21 02:01:09 +01:00
|
|
|
|
2018-05-23 15:48:22 +02:00
|
|
|
target_paths['module'] = dict((t.module, t.path) for t in targets.targets if t.module)
|
2017-08-19 02:21:11 +02:00
|
|
|
|
|
|
|
env = ansible_environment(args, color=False)
|
2018-05-23 15:48:22 +02:00
|
|
|
error_messages = []
|
|
|
|
|
|
|
|
for doc_type in sorted(doc_targets):
|
|
|
|
cmd = ['ansible-doc', '-t', doc_type] + sorted(doc_targets[doc_type])
|
|
|
|
|
|
|
|
try:
|
|
|
|
stdout, stderr = intercept_command(args, cmd, target_name='ansible-doc', env=env, capture=True, python_version=python_version)
|
|
|
|
status = 0
|
|
|
|
except SubprocessError as ex:
|
|
|
|
stdout = ex.stdout
|
|
|
|
stderr = ex.stderr
|
|
|
|
status = ex.status
|
2017-08-19 02:21:11 +02:00
|
|
|
|
2018-05-23 15:48:22 +02:00
|
|
|
if stderr:
|
|
|
|
errors = stderr.strip().splitlines()
|
|
|
|
messages = [self.parse_error(e, target_paths) for e in errors]
|
2017-08-19 02:21:11 +02:00
|
|
|
|
2018-05-23 15:48:22 +02:00
|
|
|
if messages and all(messages):
|
|
|
|
error_messages += messages
|
|
|
|
continue
|
2018-01-17 19:53:03 +01:00
|
|
|
|
2018-05-23 15:48:22 +02:00
|
|
|
if status:
|
|
|
|
summary = u'%s' % SubprocessError(cmd=cmd, status=status, stderr=stderr)
|
|
|
|
return SanityFailure(self.name, summary=summary, python_version=python_version)
|
2018-01-17 19:53:03 +01:00
|
|
|
|
2018-05-23 15:48:22 +02:00
|
|
|
if stdout:
|
|
|
|
display.info(stdout.strip(), verbosity=3)
|
2017-08-19 02:21:11 +02:00
|
|
|
|
2018-05-23 15:48:22 +02:00
|
|
|
if stderr:
|
|
|
|
summary = u'Output on stderr from ansible-doc is considered an error.\n\n%s' % SubprocessError(cmd, stderr=stderr)
|
|
|
|
return SanityFailure(self.name, summary=summary, python_version=python_version)
|
2017-08-19 02:21:11 +02:00
|
|
|
|
2018-05-23 15:48:22 +02:00
|
|
|
if error_messages:
|
|
|
|
return SanityFailure(self.name, messages=error_messages, python_version=python_version)
|
2017-08-19 02:21:11 +02:00
|
|
|
|
|
|
|
return SanitySuccess(self.name, python_version=python_version)
|
2018-01-17 19:53:03 +01:00
|
|
|
|
|
|
|
@staticmethod
|
2018-05-23 15:48:22 +02:00
|
|
|
def parse_error(error, target_paths):
|
2018-01-17 19:53:03 +01:00
|
|
|
"""
|
|
|
|
:type error: str
|
2018-05-23 15:48:22 +02:00
|
|
|
:type target_paths: dict[str, dict[str, str]]
|
2018-01-17 19:53:03 +01:00
|
|
|
:rtype: SanityMessage | None
|
|
|
|
"""
|
|
|
|
# example error messages from lib/ansible/cli/doc.py:
|
|
|
|
# ERROR! module ping missing documentation (or could not parse documentation): expected string or buffer
|
|
|
|
# [ERROR]: module ping has a documentation error formatting or is missing documentation.
|
|
|
|
match = re.search(r'^[^ ]*ERROR[^ ]* (?P<type>[^ ]+) (?P<name>[^ ]+) (?P<text>.*)$', error)
|
|
|
|
|
|
|
|
if match:
|
|
|
|
groups = match.groupdict()
|
|
|
|
|
|
|
|
error_type = groups['type']
|
|
|
|
error_name = groups['name']
|
|
|
|
error_text = groups['text']
|
|
|
|
|
2018-05-23 15:48:22 +02:00
|
|
|
if error_name in target_paths.get(error_type, {}):
|
2018-01-17 19:53:03 +01:00
|
|
|
return SanityMessage(
|
|
|
|
message=error_text,
|
2018-05-23 15:48:22 +02:00
|
|
|
path=target_paths[error_type][error_name],
|
2018-01-17 19:53:03 +01:00
|
|
|
)
|
|
|
|
|
|
|
|
return None
|