yaml and ansible linting clean ups

roles/ensure_firewalld/handlers/main.yml

@@ -1,7 +1,7 @@
 ---
 # handlers file for ensure_firewalld
-- name: ensure_firewalld.package_facts
+- name: Ensure_firewalld.package_facts
   ansible.builtin.package_facts:
 
-- name: ensure_firewalld.service_facts
+- name: Ensure_firewalld.service_facts
   ansible.builtin.service_facts:

roles/ensure_firewalld/tasks/main.yml

@@ -1,6 +1,6 @@
 ---
 # tasks file for ensure_firewalld
-- name: include variables
+- name: Include Variables
   when: ["ansible_facts[\"system\"] == 'Linux'"]
   ansible.builtin.include_vars:
     file: '{{ lookup("first_found", findme) }}'
@@ -18,19 +18,19 @@
       paths: [../vars/]
       errors: ignore
 
-- name: package discovery
+- name: Package Discovery
   when:
     - ansible_facts["system"] == 'Linux'
     - ansible_facts["packages"] is not defined
   ansible.builtin.package_facts:
 
-- name: service discovery
+- name: Service Discovery
   when:
     - ansible_facts["system"] == 'Linux'
     - ansible_facts["services"] is not defined
   ansible.builtin.service_facts:
 
-- name: ensure packages
+- name: Ensure Packages
   when:
     - ansible_facts["system"] == 'Linux'
     - ensure_firewalld is defined
@@ -44,7 +44,7 @@
     label: '{{ item.name }} will be {{ item.state }}'
   notify: [ensure_firewalld.package_facts, ensure_firewalld.service_facts]
 
-- name: ensure services
+- name: Ensure Services
   when:
     - ansible_facts["system"] == 'Linux'
     - ensure_firewalld is defined
@@ -59,10 +59,10 @@
     label: '{{ item.name }} will be {{ item.state }}'
   notify: [ensure_firewalld.package_facts, ensure_firewalld.service_facts]
 
-- name: force handlers to run for firewalld
+- name: Force Handlers To Run For Firewalld
   ansible.builtin.meta: flush_handlers
 
-- name: ensure default firewall rules
+- name: Ensure Default Firewall Rules
   when:
     - ansible_facts["system"] == 'Linux'
     - ansible_facts.packages["firewalld"] is defined
@@ -78,5 +78,5 @@
   loop_control:
     label: '{{ item.service }} will be {{ item.state }}'
 
-- name: flush handlers
+- name: Flush Handlers
   ansible.builtin.meta: flush_handlers

roles/ensure_firewalld/tests/inventory

@@ -1,3 +0,0 @@
-#SPDX-License-Identifier: MIT-0
-localhost
-

roles/ensure_firewalld/tests/test.yml

@@ -1,6 +0,0 @@
-#SPDX-License-Identifier: MIT-0
----
-- hosts: localhost
-  remote_user: root
-  roles:
-    - ensure_firewalld