From 2b62826082bf366df65c3e05afdd24c84e030229 Mon Sep 17 00:00:00 2001 From: Felix Fontein Date: Sun, 29 Oct 2023 18:04:44 +0100 Subject: [PATCH] Fix more typos (#7439) * Fix more typos in plugins/. * Fix typos in tests/unit/. * Fix typos in tests/integration/. * Fix more typos. Co-authored-by: Sebastian Gumprich --------- Co-authored-by: Sebastian Gumprich --- plugins/action/iptables_state.py | 2 +- plugins/cache/redis.py | 2 +- plugins/callback/nrdp.py | 2 +- plugins/callback/null.py | 4 ++-- plugins/connection/funcd.py | 2 +- plugins/doc_fragments/nomad.py | 2 +- plugins/inventory/lxd.py | 16 ++++++++-------- plugins/lookup/onepassword.py | 2 +- plugins/lookup/random_string.py | 6 +++--- plugins/module_utils/dimensiondata.py | 2 +- plugins/module_utils/hwc_utils.py | 6 +++--- .../module_utils/identity/keycloak/keycloak.py | 14 +++++++------- plugins/module_utils/memset.py | 2 +- plugins/module_utils/net_tools/pritunl/api.py | 2 +- plugins/module_utils/oracle/oci_utils.py | 8 ++++---- plugins/module_utils/redfish_utils.py | 6 +++--- plugins/module_utils/scaleway.py | 2 +- plugins/module_utils/vardict.py | 4 ++-- plugins/module_utils/wdc_redfish_utils.py | 2 +- plugins/modules/homectl.py | 2 +- plugins/modules/imgadm.py | 2 +- plugins/modules/jira.py | 2 +- plugins/modules/lvg.py | 2 +- plugins/modules/lxc_container.py | 2 +- plugins/modules/solaris_zone.py | 4 ++-- plugins/modules/sysrc.py | 2 +- plugins/modules/timezone.py | 2 +- .../targets/aix_filesystem/tasks/main.yml | 2 +- .../targets/apache2_module/tasks/actualtest.yml | 2 +- .../integration/targets/btrfs_subvolume/aliases | 2 +- .../targets/consul/tasks/consul_role.yml | 2 +- .../targets/filesystem/defaults/main.yml | 2 +- tests/integration/targets/gem/tasks/main.yml | 2 +- .../tasks/iso_customize_exception.yml | 2 +- .../targets/java_cert/tasks/state_change.yml | 2 +- .../readme.adoc | 2 +- .../keycloak_authz_permission/readme.adoc | 2 +- .../targets/keycloak_group/readme.adoc | 2 +- .../targets/keycloak_realm_key/readme.adoc | 2 +- .../targets/lvg/tasks/test_active_change.yml | 2 +- tests/integration/targets/mas/tasks/main.yml | 2 +- .../integration/targets/one_host/tasks/main.yml | 2 +- .../targets/pacman/tasks/yay-become.yml | 2 +- .../scaleway_compute/tasks/pagination.yml | 4 ++-- .../scaleway_container_registry/tasks/main.yml | 2 +- .../targets/scaleway_image_info/tasks/main.yml | 4 ++-- .../targets/scaleway_ip_info/tasks/main.yml | 4 ++-- .../scaleway_organization_info/tasks/main.yml | 2 +- .../scaleway_security_group_info/tasks/main.yml | 4 ++-- .../targets/scaleway_server_info/tasks/main.yml | 4 ++-- .../scaleway_snapshot_info/tasks/main.yml | 4 ++-- .../targets/scaleway_volume_info/tasks/main.yml | 4 ++-- .../targets/setup_etcd3/tasks/main.yml | 2 +- .../targets/ssh_config/tasks/options.yml | 4 ++-- .../integration/targets/terraform/tasks/main.yml | 2 +- tests/unit/plugins/become/helper.py | 2 +- .../lookup/test_bitwarden_secrets_manager.py | 2 +- tests/unit/plugins/modules/test_dnsimple_info.py | 2 +- tests/unit/plugins/modules/test_gem.py | 2 +- tests/unit/plugins/modules/test_modprobe.py | 4 ++-- tests/unit/plugins/modules/test_pagerduty.py | 6 +++--- .../unit/plugins/modules/test_rhsm_repository.py | 4 ++-- 62 files changed, 99 insertions(+), 99 deletions(-) diff --git a/plugins/action/iptables_state.py b/plugins/action/iptables_state.py index f59a7298b6..4a27ef8a01 100644 --- a/plugins/action/iptables_state.py +++ b/plugins/action/iptables_state.py @@ -44,7 +44,7 @@ class ActionModule(ActionBase): def _async_result(self, async_status_args, task_vars, timeout): ''' - Retrieve results of the asynchonous task, and display them in place of + Retrieve results of the asynchronous task, and display them in place of the async wrapper results (those with the ansible_job_id key). ''' async_status = self._task.copy() diff --git a/plugins/cache/redis.py b/plugins/cache/redis.py index 01c67fd521..c43b1dbb5e 100644 --- a/plugins/cache/redis.py +++ b/plugins/cache/redis.py @@ -150,7 +150,7 @@ class CacheModule(BaseCacheModule): # format: "localhost:26379;localhost2:26379;0:changeme" connections = uri.split(';') connection_args = connections.pop(-1) - if len(connection_args) > 0: # hanle if no db nr is given + if len(connection_args) > 0: # handle if no db nr is given connection_args = connection_args.split(':') kw['db'] = connection_args.pop(0) try: diff --git a/plugins/callback/nrdp.py b/plugins/callback/nrdp.py index c16a3c7bec..62f4a89ec8 100644 --- a/plugins/callback/nrdp.py +++ b/plugins/callback/nrdp.py @@ -14,7 +14,7 @@ DOCUMENTATION = ''' short_description: Post task results to a Nagios server through nrdp description: - This callback send playbook result to Nagios. - - Nagios shall use NRDP to recive passive events. + - Nagios shall use NRDP to receive passive events. - The passive check is sent to a dedicated host/service for Ansible. options: url: diff --git a/plugins/callback/null.py b/plugins/callback/null.py index f53a242945..6aeeba313a 100644 --- a/plugins/callback/null.py +++ b/plugins/callback/null.py @@ -15,7 +15,7 @@ DOCUMENTATION = ''' - set as main display callback short_description: Don't display stuff to screen description: - - This callback prevents outputing events to screen. + - This callback prevents outputting events to screen. ''' from ansible.plugins.callback import CallbackBase @@ -24,7 +24,7 @@ from ansible.plugins.callback import CallbackBase class CallbackModule(CallbackBase): ''' - This callback wont print messages to stdout when new callback events are received. + This callback won't print messages to stdout when new callback events are received. ''' CALLBACK_VERSION = 2.0 diff --git a/plugins/connection/funcd.py b/plugins/connection/funcd.py index 9f37f791de..219a8cccd3 100644 --- a/plugins/connection/funcd.py +++ b/plugins/connection/funcd.py @@ -70,7 +70,7 @@ class Connection(ConnectionBase): if in_data: raise AnsibleError("Internal Error: this module does not support optimized module pipelining") - # totally ignores privlege escalation + # totally ignores privilege escalation display.vvv("EXEC %s" % cmd, host=self.host) p = self.client.command.run(cmd)[self.host] return p[0], p[1], p[2] diff --git a/plugins/doc_fragments/nomad.py b/plugins/doc_fragments/nomad.py index 0c420eb8e9..1571c211c9 100644 --- a/plugins/doc_fragments/nomad.py +++ b/plugins/doc_fragments/nomad.py @@ -53,6 +53,6 @@ options: type: str token: description: - - ACL token for authentification. + - ACL token for authentication. type: str ''' diff --git a/plugins/inventory/lxd.py b/plugins/inventory/lxd.py index ccc5128c05..1539d000ab 100644 --- a/plugins/inventory/lxd.py +++ b/plugins/inventory/lxd.py @@ -84,7 +84,7 @@ DOCUMENTATION = r''' version_added: 4.2.0 prefered_instance_network_interface: description: - - If an instance has multiple network interfaces, select which one is the prefered as pattern. + - If an instance has multiple network interfaces, select which one is the preferred as pattern. - Combined with the first number that can be found e.g. 'eth' + 0. - The option has been renamed from O(prefered_container_network_interface) to O(prefered_instance_network_interface) in community.general 3.8.0. The old name still works as an alias. @@ -94,7 +94,7 @@ DOCUMENTATION = r''' - prefered_container_network_interface prefered_instance_network_family: description: - - If an instance has multiple network interfaces, which one is the prefered by family. + - If an instance has multiple network interfaces, which one is the preferred by family. - Specify V(inet) for IPv4 and V(inet6) for IPv6. type: str default: inet @@ -390,7 +390,7 @@ class InventoryModule(BaseInventoryPlugin): def get_instance_data(self, names): """Create Inventory of the instance - Iterate through the different branches of the instances and collect Informations. + Iterate through the different branches of the instances and collect Information. Args: list(names): List of instance names @@ -412,7 +412,7 @@ class InventoryModule(BaseInventoryPlugin): def get_network_data(self, names): """Create Inventory of the instance - Iterate through the different branches of the instances and collect Informations. + Iterate through the different branches of the instances and collect Information. Args: list(names): List of instance names @@ -465,9 +465,9 @@ class InventoryModule(BaseInventoryPlugin): return network_configuration def get_prefered_instance_network_interface(self, instance_name): - """Helper to get the prefered interface of thr instance + """Helper to get the preferred interface of thr instance - Helper to get the prefered interface provide by neme pattern from 'prefered_instance_network_interface'. + Helper to get the preferred interface provide by neme pattern from 'prefered_instance_network_interface'. Args: str(containe_name): name of instance @@ -577,7 +577,7 @@ class InventoryModule(BaseInventoryPlugin): else: path[instance_name][key] = value except KeyError as err: - raise AnsibleParserError("Unable to store Informations: {0}".format(to_native(err))) + raise AnsibleParserError("Unable to store Information: {0}".format(to_native(err))) def extract_information_from_instance_configs(self): """Process configuration information @@ -697,7 +697,7 @@ class InventoryModule(BaseInventoryPlugin): continue # add instance self.inventory.add_host(instance_name) - # add network informations + # add network information self.build_inventory_network(instance_name) # add os v = self._get_data_entry('inventory/{0}/os'.format(instance_name)) diff --git a/plugins/lookup/onepassword.py b/plugins/lookup/onepassword.py index 2bf1992374..b137ecab87 100644 --- a/plugins/lookup/onepassword.py +++ b/plugins/lookup/onepassword.py @@ -494,7 +494,7 @@ class OnePassCLIv2(OnePassCLIBase): if field.get("id") == field_name: return field.get("value", "") - # Look at the section data and get an indentifier. The value of 'id' is either a unique ID + # Look at the section data and get an identifier. The value of 'id' is either a unique ID # or a human-readable string. If a 'label' field exists, prefer that since # it is the value visible in the 1Password UI when both 'id' and 'label' exist. section = field.get("section", {}) diff --git a/plugins/lookup/random_string.py b/plugins/lookup/random_string.py index d810f59366..d3b29629d7 100644 --- a/plugins/lookup/random_string.py +++ b/plugins/lookup/random_string.py @@ -72,7 +72,7 @@ DOCUMENTATION = r""" type: int override_special: description: - - Overide a list of special characters to use in the string. + - Override a list of special characters to use in the string. - If set O(min_special) should be set to a non-default value. type: str override_all: @@ -89,7 +89,7 @@ DOCUMENTATION = r""" version_added: 7.5.0 similar_chars: description: - - Overide a list of characters not to be use in the string. + - Override a list of characters not to be use in the string. default: "il1LoO0" type: str version_added: 7.5.0 @@ -116,7 +116,7 @@ EXAMPLES = r""" var: lookup('community.general.random_string', base64=True) # Example result: ['NHZ6eWN5Qk0='] -- name: Generate a random string with 1 lower, 1 upper, 1 number and 1 special char (atleast) +- name: Generate a random string with 1 lower, 1 upper, 1 number and 1 special char (at least) ansible.builtin.debug: var: lookup('community.general.random_string', min_lower=1, min_upper=1, min_special=1, min_numeric=1) # Example result: ['&Qw2|E[-'] diff --git a/plugins/module_utils/dimensiondata.py b/plugins/module_utils/dimensiondata.py index 0300f6c1e9..a5201f7de0 100644 --- a/plugins/module_utils/dimensiondata.py +++ b/plugins/module_utils/dimensiondata.py @@ -39,7 +39,7 @@ except ImportError: LIBCLOUD_IMP_ERR = traceback.format_exc() HAS_LIBCLOUD = False -# MCP 2.x version patten for location (datacenter) names. +# MCP 2.x version pattern for location (datacenter) names. # # Note that this is not a totally reliable way of determining MCP version. # Unfortunately, libcloud's NodeLocation currently makes no provision for extended properties. diff --git a/plugins/module_utils/hwc_utils.py b/plugins/module_utils/hwc_utils.py index a21cc8e48f..86d29e272b 100644 --- a/plugins/module_utils/hwc_utils.py +++ b/plugins/module_utils/hwc_utils.py @@ -203,7 +203,7 @@ class Config(object): if url == "": raise HwcClientException( - 0, "Can not find the enpoint for %s" % service_type) + 0, "Cannot find the endpoint for %s" % service_type) if url[-1] != "/": url += "/" @@ -351,7 +351,7 @@ def wait_to_finish(target, pending, refresh, timeout, min_interval=1, delay=3): if pending and status not in pending: raise HwcModuleException( - "unexpect status(%s) occurred" % status) + "unexpected status(%s) occurred" % status) if not is_last_time: wait *= 2 @@ -362,7 +362,7 @@ def wait_to_finish(target, pending, refresh, timeout, min_interval=1, delay=3): time.sleep(wait) - raise HwcModuleException("asycn wait timeout after %d seconds" % timeout) + raise HwcModuleException("async wait timeout after %d seconds" % timeout) def navigate_value(data, index, array_index=None): diff --git a/plugins/module_utils/identity/keycloak/keycloak.py b/plugins/module_utils/identity/keycloak/keycloak.py index 924d2113cd..f320264fdc 100644 --- a/plugins/module_utils/identity/keycloak/keycloak.py +++ b/plugins/module_utils/identity/keycloak/keycloak.py @@ -544,7 +544,7 @@ class KeycloakAPI(object): return None def get_client_group_available_rolemappings(self, gid, cid, realm="master"): - """ Fetch the available role of a client in a specified goup on the Keycloak server. + """ Fetch the available role of a client in a specified group on the Keycloak server. :param gid: ID of the group from which to obtain the rolemappings. :param cid: ID of the client from which to obtain the rolemappings. @@ -627,7 +627,7 @@ class KeycloakAPI(object): % (rid, realm, str(e))) def add_group_rolemapping(self, gid, cid, role_rep, realm="master"): - """ Fetch the composite role of a client in a specified goup on the Keycloak server. + """ Fetch the composite role of a client in a specified group on the Keycloak server. :param gid: ID of the group from which to obtain the rolemappings. :param cid: ID of the client from which to obtain the rolemappings. @@ -1238,7 +1238,7 @@ class KeycloakAPI(object): :param realm: Realm in which the clientscope resides. :param client_id: The client in which the clientscope resides. - :return The optinal clientscopes of this realm or client + :return The optional clientscopes of this realm or client """ url = URL_OPTIONAL_CLIENTSCOPES if client_id is None else URL_CLIENT_OPTIONAL_CLIENTSCOPES return self._get_clientscopes_of_type(realm, url, 'optional', client_id) @@ -1251,7 +1251,7 @@ class KeycloakAPI(object): :param realm: Realm in which the clientscope resides. :param url_template the template for the right type - :param scope_type this can be either optinal or default + :param scope_type this can be either optional or default :param client_id: The client in which the clientscope resides. :return The clientscopes of the specified type of this realm """ @@ -1273,7 +1273,7 @@ class KeycloakAPI(object): def _decide_url_type_clientscope(self, client_id=None, scope_type="default"): """Decides which url to use. - :param scope_type this can be either optinal or default + :param scope_type this can be either optional or default :param client_id: The client in which the clientscope resides. """ if client_id is None: @@ -1520,7 +1520,7 @@ class KeycloakAPI(object): def get_subgroup_direct_parent(self, parents, realm="master", children_to_resolve=None): """ Get keycloak direct parent group API object for a given chain of parents. - To succesfully work the API for subgroups we actually dont need + To successfully work the API for subgroups we actually don't need to "walk the whole tree" for nested groups but only need to know the ID for the direct predecessor of current subgroup. This method will guarantee us this information getting there with @@ -2879,7 +2879,7 @@ class KeycloakAPI(object): groups_to_add_and_remove = self.extract_groups_to_add_to_and_remove_from_user(groups) # If group membership need to be changed if not is_struct_included(groups_to_add_and_remove['add'], user_existing_groups): - # Get available goups in the realm + # Get available groups in the realm realm_groups = self.get_groups(realm=realm) for realm_group in realm_groups: if "name" in realm_group and realm_group["name"] in groups_to_add_and_remove['add']: diff --git a/plugins/module_utils/memset.py b/plugins/module_utils/memset.py index 00c028fb09..8ddf76907b 100644 --- a/plugins/module_utils/memset.py +++ b/plugins/module_utils/memset.py @@ -79,7 +79,7 @@ def memset_api_call(api_key, api_method, payload=None): msg = "Memset API returned an error ({0}, {1})." . format(response.json()['error_type'], response.json()['error']) except urllib_error.URLError as e: has_failed = True - msg = "An URLError occured ({0})." . format(type(e)) + msg = "An URLError occurred ({0})." . format(type(e)) response.stderr = "{0}" . format(e) if msg is None: diff --git a/plugins/module_utils/net_tools/pritunl/api.py b/plugins/module_utils/net_tools/pritunl/api.py index be3bbe16f7..cc6db257d1 100644 --- a/plugins/module_utils/net_tools/pritunl/api.py +++ b/plugins/module_utils/net_tools/pritunl/api.py @@ -331,7 +331,7 @@ def pritunl_auth_request( ): """ Send an API call to a Pritunl server. - Taken from https://pritunl.com/api and adaped work with Ansible open_url + Taken from https://pritunl.com/api and adapted to work with Ansible open_url """ auth_timestamp = str(int(time.time())) auth_nonce = uuid.uuid4().hex diff --git a/plugins/module_utils/oracle/oci_utils.py b/plugins/module_utils/oracle/oci_utils.py index 717d253cec..b808d70ee2 100644 --- a/plugins/module_utils/oracle/oci_utils.py +++ b/plugins/module_utils/oracle/oci_utils.py @@ -434,7 +434,7 @@ def check_and_update_attributes( target_instance, attr_name, input_value, existing_value, changed ): """ - This function checks the difference between two resource attributes of literal types and sets the attrbute + This function checks the difference between two resource attributes of literal types and sets the attribute value in the target instance type holding the attribute. :param target_instance: The instance which contains the attribute whose values to be compared :param attr_name: Name of the attribute whose value required to be compared @@ -785,7 +785,7 @@ def _get_attributes_to_consider(exclude_attributes, model, module): attributes_to_consider = list(model.attribute_map) if "freeform_tags" in attributes_to_consider: attributes_to_consider.remove("freeform_tags") - # Temporarily removing node_count as the exisiting resource does not reflect it + # Temporarily removing node_count as the existing resource does not reflect it if "node_count" in attributes_to_consider: attributes_to_consider.remove("node_count") _debug("attributes to consider: {0}".format(attributes_to_consider)) @@ -1771,7 +1771,7 @@ def update_class_type_attr_difference( ): """ Checks the difference and updates an attribute which is represented by a class - instance. Not aplicable if the attribute type is a primitive value. + instance. Not applicable if the attribute type is a primitive value. For example, if a class name is A with an attribute x, then if A.x = X(), then only this method works. :param update_class_details The instance which should be updated if there is change in @@ -1933,7 +1933,7 @@ def get_target_resource_from_list( module, list_resource_fn, target_resource_id=None, **kwargs ): """ - Returns a resource filtered by identifer from a list of resources. This method should be + Returns a resource filtered by identifier from a list of resources. This method should be used as an alternative of 'get resource' method when 'get resource' is nor provided by resource api. This method returns a wrapper of response object but that should not be used as an input to 'wait_until' utility as this is only a partial wrapper of response object. diff --git a/plugins/module_utils/redfish_utils.py b/plugins/module_utils/redfish_utils.py index 36c14db1ac..6c470c7da3 100644 --- a/plugins/module_utils/redfish_utils.py +++ b/plugins/module_utils/redfish_utils.py @@ -327,7 +327,7 @@ class RedfishUtils(object): found in ansible.module_utils.urls, but it takes files and encodes them as Base64 strings, which is not expected by Redfish services. It also adds escaping of certain bytes in the payload, such as inserting '\r' - any time it finds a standlone '\n', which corrupts the image payload + any time it finds a standalone '\n', which corrupts the image payload send to the service. This implementation is simplified to Redfish's usage and doesn't necessarily represent an exhaustive method of building multipart requests. @@ -3703,12 +3703,12 @@ class RedfishUtils(object): rsp_uri = loc['Uri'] if vendor == 'HPE': # WORKAROUND - # HPE systems with iLO 4 will have BIOS Atrribute Registries location URI as a dictonary with key 'extref' + # HPE systems with iLO 4 will have BIOS Attribute Registries location URI as a dictionary with key 'extref' # Hence adding condition to fetch the Uri if type(loc['Uri']) is dict and "extref" in loc['Uri'].keys(): rsp_uri = loc['Uri']['extref'] if not rsp_uri: - msg = "Language 'en' not found in BIOS Atrribute Registries location, URI: %s, response: %s" + msg = "Language 'en' not found in BIOS Attribute Registries location, URI: %s, response: %s" return { "ret": False, "msg": msg % (resp_uri, str(resp_data)) diff --git a/plugins/module_utils/scaleway.py b/plugins/module_utils/scaleway.py index 43f2094800..67b821103a 100644 --- a/plugins/module_utils/scaleway.py +++ b/plugins/module_utils/scaleway.py @@ -303,7 +303,7 @@ class Scaleway(object): wait_timeout = self.module.params["wait_timeout"] wait_sleep_time = self.module.params["wait_sleep_time"] - # Prevent requesting the ressource status too soon + # Prevent requesting the resource status too soon time.sleep(wait_sleep_time) start = datetime.datetime.utcnow() diff --git a/plugins/module_utils/vardict.py b/plugins/module_utils/vardict.py index 517b112bb4..cfcce4d4d2 100644 --- a/plugins/module_utils/vardict.py +++ b/plugins/module_utils/vardict.py @@ -49,7 +49,7 @@ class _Variable(object): output (bool, optional): flag indicating whether the variable should be in the output of the module. Defaults to None. diff (bool, optional): flag indicating whether to generate diff mode output for this variable. Defaults to None. change (bool, optional): flag indicating whether to track if changes happened to this variable. Defaults to None. - fact (bool, optional): flag indicating whether the varaiable should be exposed as a fact of the module. Defaults to None. + fact (bool, optional): flag indicating whether the variable should be exposed as a fact of the module. Defaults to None. initial_value (any, optional): initial value of the variable, to be used with `change`. Defaults to NOTHING. verbosity (int, optional): level of verbosity in which this variable is reported by the module as `output`, `fact` or `diff`. Defaults to None. """ @@ -143,7 +143,7 @@ class VarDict(object): output (bool, optional): flag indicating whether the variable should be in the output of the module. Defaults to None. diff (bool, optional): flag indicating whether to generate diff mode output for this variable. Defaults to None. change (bool, optional): flag indicating whether to track if changes happened to this variable. Defaults to None. - fact (bool, optional): flag indicating whether the varaiable should be exposed as a fact of the module. Defaults to None. + fact (bool, optional): flag indicating whether the variable should be exposed as a fact of the module. Defaults to None. initial_value (any, optional): initial value of the variable, to be used with `change`. Defaults to NOTHING. verbosity (int, optional): level of verbosity in which this variable is reported by the module as `output`, `fact` or `diff`. Defaults to None. """ diff --git a/plugins/module_utils/wdc_redfish_utils.py b/plugins/module_utils/wdc_redfish_utils.py index d27e02d7b7..bc4b0c2cd0 100644 --- a/plugins/module_utils/wdc_redfish_utils.py +++ b/plugins/module_utils/wdc_redfish_utils.py @@ -182,7 +182,7 @@ class WdcRedfishUtils(RedfishUtils): :param str bundle_uri: HTTP URI of the firmware bundle. :return: Firmware version number contained in the bundle, and whether or not the bundle is multi-tenant. - Either value will be None if unable to deterine. + Either value will be None if unable to determine. :rtype: str or None, bool or None """ bundle_temp_filename = fetch_file(module=self.module, diff --git a/plugins/modules/homectl.py b/plugins/modules/homectl.py index 270c94e351..ca4c19a875 100644 --- a/plugins/modules/homectl.py +++ b/plugins/modules/homectl.py @@ -393,7 +393,7 @@ class Homectl(object): user_metadata.pop('status', None) # Let last change Usec be updated by homed when command runs. user_metadata.pop('lastChangeUSec', None) - # Now only change fields that are called on leaving whats currently in the record intact. + # Now only change fields that are called on leaving what's currently in the record intact. record = user_metadata record['userName'] = self.name diff --git a/plugins/modules/imgadm.py b/plugins/modules/imgadm.py index 8ea1bd0216..438acaa329 100644 --- a/plugins/modules/imgadm.py +++ b/plugins/modules/imgadm.py @@ -174,7 +174,7 @@ class Imgadm(object): # There is no feedback from imgadm(1M) to determine if anything # was actually changed. So treat this as an 'always-changes' operation. - # Note that 'imgadm -v' produces unparseable JSON... + # Note that 'imgadm -v' produces unparsable JSON... self.changed = True def manage_sources(self): diff --git a/plugins/modules/jira.py b/plugins/modules/jira.py index 53b0be0695..c36cf99375 100644 --- a/plugins/modules/jira.py +++ b/plugins/modules/jira.py @@ -799,7 +799,7 @@ class JIRA(StateModuleHelper): if msg: self.module.fail_json(msg=', '.join(msg)) self.module.fail_json(msg=to_native(error)) - # Fallback print body, if it cant be decoded + # Fallback print body, if it can't be decoded self.module.fail_json(msg=to_native(info['body'])) body = response.read() diff --git a/plugins/modules/lvg.py b/plugins/modules/lvg.py index df96c189ae..51045ff4e5 100644 --- a/plugins/modules/lvg.py +++ b/plugins/modules/lvg.py @@ -480,7 +480,7 @@ def main(): module.fail_json(msg="Failed to remove volume group %s" % (vg), rc=rc, err=err) else: module.fail_json(msg="Refuse to remove non-empty volume group %s without force=true" % (vg)) - # activate/inactivate existing VG + # activate/deactivate existing VG elif state == 'active': changed = activate_vg(module=module, vg=vg, active=True) elif state == 'inactive': diff --git a/plugins/modules/lxc_container.py b/plugins/modules/lxc_container.py index fa5e852a65..7ded041e93 100644 --- a/plugins/modules/lxc_container.py +++ b/plugins/modules/lxc_container.py @@ -1277,7 +1277,7 @@ class LxcContainerManagement(object): """ vg = self._get_lxc_vg() - free_space, messurement = self._get_vg_free_pe(vg_name=vg) + free_space, measurement = self._get_vg_free_pe(vg_name=vg) if free_space < float(snapshot_size_gb): message = ( diff --git a/plugins/modules/solaris_zone.py b/plugins/modules/solaris_zone.py index a8d7956c91..d9f44589dc 100644 --- a/plugins/modules/solaris_zone.py +++ b/plugins/modules/solaris_zone.py @@ -46,8 +46,8 @@ options: description: - Zone name. - A zone name must be unique name. - - A zone name must begin with an alpha-numeric character. - - The name can contain alpha-numeric characters, underscores V(_), hyphens V(-), and periods V(.). + - A zone name must begin with an alphanumeric character. + - The name can contain alphanumeric characters, underscores V(_), hyphens V(-), and periods V(.). - The name cannot be longer than 64 characters. type: str required: true diff --git a/plugins/modules/sysrc.py b/plugins/modules/sysrc.py index 76d6569e40..6780975d4f 100644 --- a/plugins/modules/sysrc.py +++ b/plugins/modules/sysrc.py @@ -222,7 +222,7 @@ def main(): # OID style names are not supported if not re.match('^[a-zA-Z0-9_]+$', name): module.fail_json( - msg="Name may only contain alpha-numeric and underscore characters" + msg="Name may only contain alphanumeric and underscore characters" ) value = module.params.pop('value') diff --git a/plugins/modules/timezone.py b/plugins/modules/timezone.py index 7998001c01..1059d44d3b 100644 --- a/plugins/modules/timezone.py +++ b/plugins/modules/timezone.py @@ -53,7 +53,7 @@ options: choices: [ local, UTC ] notes: - On SmartOS the C(sm-set-timezone) utility (part of the smtools package) is required to set the zone timezone - - On AIX only Olson/tz database timezones are useable (POSIX is not supported). + - On AIX only Olson/tz database timezones are usable (POSIX is not supported). - An OS reboot is also required on AIX for the new timezone setting to take effect. author: - Shinichi TAMURA (@tmshn) diff --git a/tests/integration/targets/aix_filesystem/tasks/main.yml b/tests/integration/targets/aix_filesystem/tasks/main.yml index 25146062d6..878088f4e7 100644 --- a/tests/integration/targets/aix_filesystem/tasks/main.yml +++ b/tests/integration/targets/aix_filesystem/tasks/main.yml @@ -90,7 +90,7 @@ size: -2G state: present -- name: Resizing /mksysb to 100G (no enought space) +- name: Resizing /mksysb to 100G (not enough space) aix_filesystem: filesystem: /mksysb size: +100G diff --git a/tests/integration/targets/apache2_module/tasks/actualtest.yml b/tests/integration/targets/apache2_module/tasks/actualtest.yml index 3301a16b15..6fd10ce572 100644 --- a/tests/integration/targets/apache2_module/tasks/actualtest.yml +++ b/tests/integration/targets/apache2_module/tasks/actualtest.yml @@ -73,7 +73,7 @@ state: absent force: true - - name: reenable autoindex + - name: re-enable autoindex community.general.apache2_module: name: autoindex state: present diff --git a/tests/integration/targets/btrfs_subvolume/aliases b/tests/integration/targets/btrfs_subvolume/aliases index 914c36ad3d..f5b6800dbd 100644 --- a/tests/integration/targets/btrfs_subvolume/aliases +++ b/tests/integration/targets/btrfs_subvolume/aliases @@ -1,4 +1,4 @@ -# Copyright (c) Ansible Projec +# Copyright (c) Ansible Project # GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt) # SPDX-License-Identifier: GPL-3.0-or-later diff --git a/tests/integration/targets/consul/tasks/consul_role.yml b/tests/integration/targets/consul/tasks/consul_role.yml index cd83929ef9..638af9b1c9 100644 --- a/tests/integration/targets/consul/tasks/consul_role.yml +++ b/tests/integration/targets/consul/tasks/consul_role.yml @@ -161,7 +161,7 @@ - result['role']['ServiceIdentities'][0]['ServiceName'] == "web" - result['role']['ServiceIdentities'][0]['Datacenters'][0] == "dc1" - result['role']['Policies'] is not defined - - result['role']['NodeIdentities'] == [] # in check mode the cleared field is returned as an emtpy array + - result['role']['NodeIdentities'] == [] # in check mode the cleared field is returned as an empty array - name: Update the role with service identity to remove the service identities consul_role: diff --git a/tests/integration/targets/filesystem/defaults/main.yml b/tests/integration/targets/filesystem/defaults/main.yml index 1e7ea36c76..ec446d2417 100644 --- a/tests/integration/targets/filesystem/defaults/main.yml +++ b/tests/integration/targets/filesystem/defaults/main.yml @@ -11,7 +11,7 @@ tested_filesystems: # - XFS: 20Mo # - Btrfs: 150Mo (50Mo when "--metadata single" is used and 100Mb when on newer Fedora versions) # - f2fs: - # - 1.2.0 requires at leat 116Mo + # - 1.2.0 requires at least 116Mo # - 1.7.0 requires at least 30Mo # - 1.10.0 requires at least 38Mo # - resizefs asserts when initial fs is smaller than 60Mo and seems to require 1.10.0 diff --git a/tests/integration/targets/gem/tasks/main.yml b/tests/integration/targets/gem/tasks/main.yml index 362c126bfd..2d615304f8 100644 --- a/tests/integration/targets/gem/tasks/main.yml +++ b/tests/integration/targets/gem/tasks/main.yml @@ -109,7 +109,7 @@ - current_gems.stdout is not search('gist\s+\([0-9.]+\)') when: ansible_user_uid == 0 - # Check cutom gem directory + # Check custom gem directory - name: Install gem in a custom directory with incorrect options gem: name: gist diff --git a/tests/integration/targets/iso_customize/tasks/iso_customize_exception.yml b/tests/integration/targets/iso_customize/tasks/iso_customize_exception.yml index b2130bb6b0..715f2fd389 100644 --- a/tests/integration/targets/iso_customize/tasks/iso_customize_exception.yml +++ b/tests/integration/targets/iso_customize/tasks/iso_customize_exception.yml @@ -59,7 +59,7 @@ failed_when: customized_result.msg.find("does not exist") == -1 # Test: filenames with whitespaces -# We report error: the user should be reponsible for the it +# We report error: the user should be responsible for the it - name: "Testcase: filenames with whitespaces" community.general.iso_customize: src_iso: "{{ test_dir }}/test.iso" diff --git a/tests/integration/targets/java_cert/tasks/state_change.yml b/tests/integration/targets/java_cert/tasks/state_change.yml index e135a60a3e..f2898ddc2e 100644 --- a/tests/integration/targets/java_cert/tasks/state_change.yml +++ b/tests/integration/targets/java_cert/tasks/state_change.yml @@ -181,7 +181,7 @@ - result_x509_changed is changed - name: | - We also want to make sure that the status doesnt change if we import the same cert + We also want to make sure that the status does not change if we import the same cert community.general.java_cert: cert_alias: test_cert cert_path: "{{ test_cert2_path }}" diff --git a/tests/integration/targets/keycloak_authz_authorization_scope/readme.adoc b/tests/integration/targets/keycloak_authz_authorization_scope/readme.adoc index 1941e54efd..8e052920c1 100644 --- a/tests/integration/targets/keycloak_authz_authorization_scope/readme.adoc +++ b/tests/integration/targets/keycloak_authz_authorization_scope/readme.adoc @@ -20,7 +20,7 @@ docker run --name mykeycloak -p 8080:8080 -e KC_HTTP_RELATIVE_PATH=/auth -e KEYC This test suite can run against a fresh unconfigured server instance (no preconfiguration required) and cleans up after itself (undoes all -its config changes) as long as it runs through completly. While its active +its config changes) as long as it runs through completely. While its active it changes the server configuration in the following ways: * creating, modifying and deleting some keycloak groups diff --git a/tests/integration/targets/keycloak_authz_permission/readme.adoc b/tests/integration/targets/keycloak_authz_permission/readme.adoc index 1941e54efd..8e052920c1 100644 --- a/tests/integration/targets/keycloak_authz_permission/readme.adoc +++ b/tests/integration/targets/keycloak_authz_permission/readme.adoc @@ -20,7 +20,7 @@ docker run --name mykeycloak -p 8080:8080 -e KC_HTTP_RELATIVE_PATH=/auth -e KEYC This test suite can run against a fresh unconfigured server instance (no preconfiguration required) and cleans up after itself (undoes all -its config changes) as long as it runs through completly. While its active +its config changes) as long as it runs through completely. While its active it changes the server configuration in the following ways: * creating, modifying and deleting some keycloak groups diff --git a/tests/integration/targets/keycloak_group/readme.adoc b/tests/integration/targets/keycloak_group/readme.adoc index 1941e54efd..8e052920c1 100644 --- a/tests/integration/targets/keycloak_group/readme.adoc +++ b/tests/integration/targets/keycloak_group/readme.adoc @@ -20,7 +20,7 @@ docker run --name mykeycloak -p 8080:8080 -e KC_HTTP_RELATIVE_PATH=/auth -e KEYC This test suite can run against a fresh unconfigured server instance (no preconfiguration required) and cleans up after itself (undoes all -its config changes) as long as it runs through completly. While its active +its config changes) as long as it runs through completely. While its active it changes the server configuration in the following ways: * creating, modifying and deleting some keycloak groups diff --git a/tests/integration/targets/keycloak_realm_key/readme.adoc b/tests/integration/targets/keycloak_realm_key/readme.adoc index 1941e54efd..8e052920c1 100644 --- a/tests/integration/targets/keycloak_realm_key/readme.adoc +++ b/tests/integration/targets/keycloak_realm_key/readme.adoc @@ -20,7 +20,7 @@ docker run --name mykeycloak -p 8080:8080 -e KC_HTTP_RELATIVE_PATH=/auth -e KEYC This test suite can run against a fresh unconfigured server instance (no preconfiguration required) and cleans up after itself (undoes all -its config changes) as long as it runs through completly. While its active +its config changes) as long as it runs through completely. While its active it changes the server configuration in the following ways: * creating, modifying and deleting some keycloak groups diff --git a/tests/integration/targets/lvg/tasks/test_active_change.yml b/tests/integration/targets/lvg/tasks/test_active_change.yml index 1b31693269..8ff88e6fbb 100644 --- a/tests/integration/targets/lvg/tasks/test_active_change.yml +++ b/tests/integration/targets/lvg/tasks/test_active_change.yml @@ -140,7 +140,7 @@ that: - "'active' == '{{ item }}'" -- name: Deactive volume group in check mode +- name: Deactivate volume group in check mode lvg: state: inactive vg: testvg diff --git a/tests/integration/targets/mas/tasks/main.yml b/tests/integration/targets/mas/tasks/main.yml index f659160dcc..839620779e 100644 --- a/tests/integration/targets/mas/tasks/main.yml +++ b/tests/integration/targets/mas/tasks/main.yml @@ -117,7 +117,7 @@ that: - install_status.stat.exists == true -- name: Unistall Rested +- name: Uninstall Rested mas: id: 421879749 state: absent diff --git a/tests/integration/targets/one_host/tasks/main.yml b/tests/integration/targets/one_host/tasks/main.yml index ffd5ac04ca..3b2c1cedf3 100644 --- a/tests/integration/targets/one_host/tasks/main.yml +++ b/tests/integration/targets/one_host/tasks/main.yml @@ -9,7 +9,7 @@ # GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt) # SPDX-License-Identifier: GPL-3.0-or-later -# ENVIRONENT PREPARACTION +# ENVIRONMENT PREPARACTION - set_fact: test_number= 0 diff --git a/tests/integration/targets/pacman/tasks/yay-become.yml b/tests/integration/targets/pacman/tasks/yay-become.yml index 30f322effe..95698d5ceb 100644 --- a/tests/integration/targets/pacman/tasks/yay-become.yml +++ b/tests/integration/targets/pacman/tasks/yay-become.yml @@ -30,7 +30,7 @@ state: present notify: Remove packages for yay-become -- name: Hack permssions for the remote_tmp_dir +- name: Hack permissions for the remote_tmp_dir ansible.builtin.file: path: "{{ remote_tmp_dir }}" mode: '0777' diff --git a/tests/integration/targets/scaleway_compute/tasks/pagination.yml b/tests/integration/targets/scaleway_compute/tasks/pagination.yml index 5a674b8014..b2429e39f9 100644 --- a/tests/integration/targets/scaleway_compute/tasks/pagination.yml +++ b/tests/integration/targets/scaleway_compute/tasks/pagination.yml @@ -23,7 +23,7 @@ commercial_type: '{{ scaleway_commerial_type }}' wait: true -- name: Get server informations of the first page +- name: Get server information of the first page scaleway_server_info: region: par1 query_parameters: @@ -37,7 +37,7 @@ that: - first_page is success -- name: Get server informations of the second page +- name: Get server information of the second page scaleway_server_info: region: par1 query_parameters: diff --git a/tests/integration/targets/scaleway_container_registry/tasks/main.yml b/tests/integration/targets/scaleway_container_registry/tasks/main.yml index 91cea20f3f..84d733a107 100644 --- a/tests/integration/targets/scaleway_container_registry/tasks/main.yml +++ b/tests/integration/targets/scaleway_container_registry/tasks/main.yml @@ -159,7 +159,7 @@ - cr_deletion_task is success - cr_deletion_task is changed -- name: Delete container regitry (Confirmation) +- name: Delete container registry (Confirmation) community.general.scaleway_container_registry: state: absent name: '{{ name }}' diff --git a/tests/integration/targets/scaleway_image_info/tasks/main.yml b/tests/integration/targets/scaleway_image_info/tasks/main.yml index 2cdf34fdd4..3dfe20f37b 100644 --- a/tests/integration/targets/scaleway_image_info/tasks/main.yml +++ b/tests/integration/targets/scaleway_image_info/tasks/main.yml @@ -8,7 +8,7 @@ # GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt) # SPDX-License-Identifier: GPL-3.0-or-later -- name: Get image informations and register it in a variable +- name: Get image information and register it in a variable scaleway_image_info: region: par1 register: images @@ -22,7 +22,7 @@ that: - images is success -- name: Get image informations from ams1 and register it in a variable +- name: Get image information from ams1 and register it in a variable scaleway_image_info: region: ams1 register: images_ams1 diff --git a/tests/integration/targets/scaleway_ip_info/tasks/main.yml b/tests/integration/targets/scaleway_ip_info/tasks/main.yml index b560b56586..6aad9b52e9 100644 --- a/tests/integration/targets/scaleway_ip_info/tasks/main.yml +++ b/tests/integration/targets/scaleway_ip_info/tasks/main.yml @@ -8,7 +8,7 @@ # GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt) # SPDX-License-Identifier: GPL-3.0-or-later -- name: Get ip informations and register it in a variable +- name: Get ip information and register it in a variable scaleway_ip_info: region: par1 register: ips @@ -22,7 +22,7 @@ that: - ips is success -- name: Get ip informations and register it in a variable +- name: Get ip information and register it in a variable scaleway_ip_info: region: ams1 register: ips_ams1 diff --git a/tests/integration/targets/scaleway_organization_info/tasks/main.yml b/tests/integration/targets/scaleway_organization_info/tasks/main.yml index 7326ca2266..247a4012b2 100644 --- a/tests/integration/targets/scaleway_organization_info/tasks/main.yml +++ b/tests/integration/targets/scaleway_organization_info/tasks/main.yml @@ -8,7 +8,7 @@ # GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt) # SPDX-License-Identifier: GPL-3.0-or-later -- name: Get organization informations and register it in a variable +- name: Get organization information and register it in a variable scaleway_organization_info: register: organizations diff --git a/tests/integration/targets/scaleway_security_group_info/tasks/main.yml b/tests/integration/targets/scaleway_security_group_info/tasks/main.yml index 8029a1e9aa..608d76409a 100644 --- a/tests/integration/targets/scaleway_security_group_info/tasks/main.yml +++ b/tests/integration/targets/scaleway_security_group_info/tasks/main.yml @@ -8,7 +8,7 @@ # GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt) # SPDX-License-Identifier: GPL-3.0-or-later -- name: Get security group informations and register it in a variable +- name: Get security group information and register it in a variable scaleway_security_group_info: region: par1 register: security_groups @@ -22,7 +22,7 @@ that: - security_groups is success -- name: Get security group informations and register it in a variable (AMS1) +- name: Get security group information and register it in a variable (AMS1) scaleway_security_group_info: region: ams1 register: ams1_security_groups diff --git a/tests/integration/targets/scaleway_server_info/tasks/main.yml b/tests/integration/targets/scaleway_server_info/tasks/main.yml index 7274e8a85b..516b2df0e9 100644 --- a/tests/integration/targets/scaleway_server_info/tasks/main.yml +++ b/tests/integration/targets/scaleway_server_info/tasks/main.yml @@ -8,7 +8,7 @@ # GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt) # SPDX-License-Identifier: GPL-3.0-or-later -- name: Get server informations and register it in a variable +- name: Get server information and register it in a variable scaleway_server_info: region: par1 register: servers @@ -22,7 +22,7 @@ that: - servers is success -- name: Get server informations and register it in a variable +- name: Get server information and register it in a variable scaleway_server_info: region: ams1 register: ams1_servers diff --git a/tests/integration/targets/scaleway_snapshot_info/tasks/main.yml b/tests/integration/targets/scaleway_snapshot_info/tasks/main.yml index 44f15d5156..f80a9ced18 100644 --- a/tests/integration/targets/scaleway_snapshot_info/tasks/main.yml +++ b/tests/integration/targets/scaleway_snapshot_info/tasks/main.yml @@ -8,7 +8,7 @@ # GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt) # SPDX-License-Identifier: GPL-3.0-or-later -- name: Get snapshot informations and register it in a variable +- name: Get snapshot information and register it in a variable scaleway_snapshot_info: region: par1 register: snapshots @@ -22,7 +22,7 @@ that: - snapshots is success -- name: Get snapshot informations and register it in a variable (AMS1) +- name: Get snapshot information and register it in a variable (AMS1) scaleway_snapshot_info: region: ams1 register: ams1_snapshots diff --git a/tests/integration/targets/scaleway_volume_info/tasks/main.yml b/tests/integration/targets/scaleway_volume_info/tasks/main.yml index 45995a54c4..2202e6bf98 100644 --- a/tests/integration/targets/scaleway_volume_info/tasks/main.yml +++ b/tests/integration/targets/scaleway_volume_info/tasks/main.yml @@ -8,7 +8,7 @@ # GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt) # SPDX-License-Identifier: GPL-3.0-or-later -- name: Get volume informations and register it in a variable +- name: Get volume information and register it in a variable scaleway_volume_info: region: par1 register: volumes @@ -22,7 +22,7 @@ that: - volumes is success -- name: Get volume informations and register it in a variable (AMS1) +- name: Get volume information and register it in a variable (AMS1) scaleway_volume_info: region: ams1 register: ams1_volumes diff --git a/tests/integration/targets/setup_etcd3/tasks/main.yml b/tests/integration/targets/setup_etcd3/tasks/main.yml index fe6b9cd028..1da52e225f 100644 --- a/tests/integration/targets/setup_etcd3/tasks/main.yml +++ b/tests/integration/targets/setup_etcd3/tasks/main.yml @@ -50,7 +50,7 @@ state: present # Check if re-installing etcd3 is required - - name: Check if etcd3ctl exists for re-use. + - name: Check if etcd3ctl exists for reuse. shell: "ETCDCTL_API=3 {{ etcd3_path }}/etcdctl --endpoints=localhost:2379 get foo" args: executable: /bin/bash diff --git a/tests/integration/targets/ssh_config/tasks/options.yml b/tests/integration/targets/ssh_config/tasks/options.yml index 406de6831d..df12675947 100644 --- a/tests/integration/targets/ssh_config/tasks/options.yml +++ b/tests/integration/targets/ssh_config/tasks/options.yml @@ -33,7 +33,7 @@ src: "{{ ssh_config_test }}" register: slurp_ssh_config -- name: "Options - Verify that nothign was added to {{ ssh_config_test }} during change mode" +- name: "Options - Verify that nothing was added to {{ ssh_config_test }} during change mode" assert: that: - "'options.example.com' not in slurp_ssh_config['content'] | b64decode" @@ -243,7 +243,7 @@ src: "{{ ssh_config_test }}" register: slurp_ssh_config -- name: "Options - Verify that nothign was added to {{ ssh_config_test }} during change mode" +- name: "Options - Verify that nothing was added to {{ ssh_config_test }} during change mode" assert: that: - "'options.example.com' not in slurp_ssh_config['content'] | b64decode" diff --git a/tests/integration/targets/terraform/tasks/main.yml b/tests/integration/targets/terraform/tasks/main.yml index 1c66990be2..d04757d8e4 100644 --- a/tests/integration/targets/terraform/tasks/main.yml +++ b/tests/integration/targets/terraform/tasks/main.yml @@ -63,5 +63,5 @@ loop_control: index_var: provider_index -- name: Test Complex Varibles +- name: Test Complex Variables ansible.builtin.include_tasks: complex_variables.yml diff --git a/tests/unit/plugins/become/helper.py b/tests/unit/plugins/become/helper.py index 9949e1befb..d2a7df97ff 100644 --- a/tests/unit/plugins/become/helper.py +++ b/tests/unit/plugins/become/helper.py @@ -12,7 +12,7 @@ from ansible.plugins.loader import become_loader, get_shell_plugin def call_become_plugin(task, var_options, cmd, executable=None): - """Helper function to call become plugin simiarly on how Ansible itself handles this.""" + """Helper function to call become plugin similarly on how Ansible itself handles this.""" plugin = become_loader.get(task['become_method']) plugin.set_options(task_keys=task, var_options=var_options) shell = get_shell_plugin(executable=executable) diff --git a/tests/unit/plugins/lookup/test_bitwarden_secrets_manager.py b/tests/unit/plugins/lookup/test_bitwarden_secrets_manager.py index 466d5697db..aaeaf79eaf 100644 --- a/tests/unit/plugins/lookup/test_bitwarden_secrets_manager.py +++ b/tests/unit/plugins/lookup/test_bitwarden_secrets_manager.py @@ -78,6 +78,6 @@ class TestLookupModule(unittest.TestCase): @patch('ansible_collections.community.general.plugins.lookup.bitwarden_secrets_manager._bitwarden_secrets_manager', new=MockBitwardenSecretsManager()) def test_bitwarden_secrets_manager_no_match(self): - # Getting a nonexistant secret id throws exception + # Getting a nonexistent secret id throws exception with self.assertRaises(AnsibleLookupError): self.lookup.run(['nonexistant_id'], bws_access_token='123') diff --git a/tests/unit/plugins/modules/test_dnsimple_info.py b/tests/unit/plugins/modules/test_dnsimple_info.py index 5806ec7726..08c5296c81 100644 --- a/tests/unit/plugins/modules/test_dnsimple_info.py +++ b/tests/unit/plugins/modules/test_dnsimple_info.py @@ -89,7 +89,7 @@ class TestDNSimple_Info(ModuleTestCase): result = exc_info.exception.args[0] # nothing should change self.assertFalse(result['changed']) - # we should return at least one item with mathing domain + # we should return at least one item with matching domain assert result['dnsimple_records_info'][0]['name'] == name @with_httmock(records_resp) diff --git a/tests/unit/plugins/modules/test_gem.py b/tests/unit/plugins/modules/test_gem.py index 92578e062d..10c03e537d 100644 --- a/tests/unit/plugins/modules/test_gem.py +++ b/tests/unit/plugins/modules/test_gem.py @@ -69,7 +69,7 @@ class TestGem(ModuleTestCase): assert result['msg'] == "install_dir requires user_install=false" def test_passes_install_dir_to_gem(self): - # XXX: This test is extremely fragile, and makes assuptions about the module code, and how + # XXX: This test is extremely fragile, and makes assumptions about the module code, and how # functions are run. # If you start modifying the code of the module, you might need to modify what this # test mocks. The only thing that matters is the assertion that this 'gem install' is diff --git a/tests/unit/plugins/modules/test_modprobe.py b/tests/unit/plugins/modules/test_modprobe.py index 18695695a4..2ad0831511 100644 --- a/tests/unit/plugins/modules/test_modprobe.py +++ b/tests/unit/plugins/modules/test_modprobe.py @@ -152,7 +152,7 @@ class TestUnloadModule(ModuleTestCase): class TestModuleIsLoadedPersistently(ModuleTestCase): def setUp(self): if (sys.version_info[0] == 3 and sys.version_info[1] < 7) or (sys.version_info[0] == 2 and sys.version_info[1] < 7): - self.skipTest('open_mock doesnt support readline in earlier python versions') + self.skipTest("open_mock doesn't support readline in earlier python versions") super(TestModuleIsLoadedPersistently, self).setUp() @@ -230,7 +230,7 @@ class TestModuleIsLoadedPersistently(ModuleTestCase): class TestPermanentParams(ModuleTestCase): def setUp(self): if (sys.version_info[0] == 3 and sys.version_info[1] < 7) or (sys.version_info[0] == 2 and sys.version_info[1] < 7): - self.skipTest('open_mock doesnt support readline in earlier python versions') + self.skipTest("open_mock doesn't support readline in earlier python versions") super(TestPermanentParams, self).setUp() self.mock_get_bin_path = patch('ansible.module_utils.basic.AnsibleModule.get_bin_path') diff --git a/tests/unit/plugins/modules/test_pagerduty.py b/tests/unit/plugins/modules/test_pagerduty.py index 5f12fda8db..75987d3df0 100644 --- a/tests/unit/plugins/modules/test_pagerduty.py +++ b/tests/unit/plugins/modules/test_pagerduty.py @@ -54,9 +54,9 @@ class PagerDutyTest(unittest.TestCase): def _assert_create_window_payload(self, module, url, headers, data=None, method=None): payload = json.loads(data) window_data = payload['maintenance_window'] - self.assertTrue('start_time' in window_data, '"start_time" is requiered attribute') - self.assertTrue('end_time' in window_data, '"end_time" is requiered attribute') - self.assertTrue('services' in window_data, '"services" is requiered attribute') + self.assertTrue('start_time' in window_data, '"start_time" is required attribute') + self.assertTrue('end_time' in window_data, '"end_time" is required attribute') + self.assertTrue('services' in window_data, '"services" is required attribute') return object(), {'status': 201} def _assert_create_window_single_service(self, module, url, headers, data=None, method=None): diff --git a/tests/unit/plugins/modules/test_rhsm_repository.py b/tests/unit/plugins/modules/test_rhsm_repository.py index 249d0495db..e822c7e844 100644 --- a/tests/unit/plugins/modules/test_rhsm_repository.py +++ b/tests/unit/plugins/modules/test_rhsm_repository.py @@ -89,7 +89,7 @@ Enabled: %s Enable the specified IDs. 'repo_ids' can be either a string or a list of strings representing - an ID (wilcard included). + an ID (wildcard included). Returns the same object, so calls to this can be chained. """ @@ -104,7 +104,7 @@ Enabled: %s Disable the specified IDs. 'repo_ids' can be either a string or a list of strings representing - an ID (wilcard included). + an ID (wildcard included). Returns the same object, so calls to this can be chained. """