Fix ansible-test reported pep8 errors

These are indent issues, one item per line for argument_specs items
containing options dicts and missing or overflow spaces for comments
and dict delimiters.
This commit is contained in:
Thomas Woerner
2021-12-14 17:17:17 +01:00
parent 82412ef761
commit 85006d611f
11 changed files with 61 additions and 54 deletions

View File

@@ -194,19 +194,23 @@ def main():
argument_spec=dict( argument_spec=dict(
# general # general
inclusive=dict(type="list", inclusive=dict(type="list",
aliases=["automemberinclusiveregex"], default=None, aliases=["automemberinclusiveregex"],
default=None,
options=dict( options=dict(
key=dict(type="str", required=True), key=dict(type="str", required=True),
expression=dict(type="str", required=True) expression=dict(type="str", required=True)
), ),
elements="dict", required=False), elements="dict",
exclusive=dict(type="list", aliases=[ required=False),
"automemberexclusiveregex"], default=None, exclusive=dict(type="list",
aliases=["automemberexclusiveregex"],
default=None,
options=dict( options=dict(
key=dict(type="str", required=True), key=dict(type="str", required=True),
expression=dict(type="str", required=True) expression=dict(type="str", required=True)
), ),
elements="dict", required=False), elements="dict",
required=False),
name=dict(type="list", aliases=["cn"], name=dict(type="list", aliases=["cn"],
default=None, required=True), default=None, required=True),
description=dict(type="str", default=None), description=dict(type="str", default=None),

View File

@@ -181,7 +181,6 @@ def main():
# general # general
state=dict(type="str", default="present", state=dict(type="str", default="present",
choices=["present", "absent"]), choices=["present", "absent"]),
) )
) )

View File

@@ -1108,7 +1108,8 @@ def configure_module():
name=dict(type="list", aliases=["record_name"], default=None, name=dict(type="list", aliases=["record_name"], default=None,
required=False), required=False),
records=dict(type="list", default=None, records=dict(type="list",
default=None,
options=dict( options=dict(
# Here name is a simple string # Here name is a simple string
name=dict(type='str', required=True, name=dict(type='str', required=True,

View File

@@ -393,9 +393,9 @@ def main():
# If yes: modify # If yes: modify
# Also if it is a modification from nonposix to posix # Also if it is a modification from nonposix to posix
# or nonposix to external. # or nonposix to external.
if not compare_args_ipa(ansible_module, args, if not compare_args_ipa(
res_find) or \ ansible_module, args, res_find
( ) or (
not is_posix_group(res_find) and not is_posix_group(res_find) and
not is_external_group(res_find) and not is_external_group(res_find) and
(posix or external) (posix or external)

View File

@@ -824,14 +824,17 @@ def main():
# general # general
name=dict(type="list", aliases=["login"], default=None, name=dict(type="list", aliases=["login"], default=None,
required=False), required=False),
users=dict(type="list", aliases=["login"], default=None, users=dict(type="list",
aliases=["login"],
default=None,
options=dict( options=dict(
# Here name is a simple string # Here name is a simple string
name=dict(type="str", required=True), name=dict(type="str", required=True),
# Add user specific parameters # Add user specific parameters
**user_spec **user_spec
), ),
elements='dict', required=False), elements='dict',
required=False),
# deleted # deleted
preserve=dict(required=False, type='bool', default=None), preserve=dict(required=False, type='bool', default=None),

View File

@@ -56,8 +56,8 @@ param_docs = {
"no_ui_redirect": "Do not automatically redirect to the Web UI", "no_ui_redirect": "Do not automatically redirect to the Web UI",
"external_ca": "External ca setting", "external_ca": "External ca setting",
"external_cert_files": [ "external_cert_files": [
"File containing the IPA CA certificate and the external CA certificate", "File containing the IPA CA certificate and the external CA "
"chain" "certificate chain"
], ],
"reverse_zones": "The reverse DNS zones to use", "reverse_zones": "The reverse DNS zones to use",
"no_reverse": "Do not create new reverse DNS zone", "no_reverse": "Do not create new reverse DNS zone",
@@ -131,8 +131,8 @@ param_docs = {
"debug": "Turn on extra debugging", "debug": "Turn on extra debugging",
"basedn": "The basedn of the IPA server (of the form dc=example,dc=com)", "basedn": "The basedn of the IPA server (of the form dc=example,dc=com)",
"allow_repair": [ "allow_repair": [
"Allow repair of already joined hosts. Contrary to ipaclient_force_join", "Allow repair of already joined hosts. Contrary to "
"the host entry will not be changed on the server" "ipaclient_force_join the host entry will not be changed on the server"
], ],
"backup": "File to backup", "backup": "File to backup",
"fqdn": [ "fqdn": [