Use FQCNs for builtins in molecule example to avoid adding additional lint configs.

geerlingguy · geerlingguy · commit fac6ffde6e19 · 2022-03-27T16:14:52.000-05:00
diff --git a/molecule/main.yml b/molecule/main.yml
@@ -9,25 +9,25 @@
 
   handlers:
     - name: restart apache
-      service:
+      ansible.builtin.service:
         name: "{{ apache_service }}"
         state: restarted
 
   pre_tasks:
     - name: Override Apache vars for Red Hat.
-      set_fact:
+      ansible.builtin.set_fact:
         apache_package: httpd
         apache_service: httpd
       when: ansible_os_family == 'RedHat'
 
   tasks:
     - name: Ensure Apache is installed.
-      package:
+      ansible.builtin.package:
         name: "{{ apache_package }}"
         state: present
 
     - name: Copy a web page.
-      copy:
+      ansible.builtin.copy:
         content: |
           <html>
           <head><title>Hello world!</title></head>
@@ -38,7 +38,7 @@
       notify: restart apache
 
     - name: Ensure Apache is running and starts at boot.
-      service:
+      ansible.builtin.service:
         name: "{{ apache_service }}"
         state: started
         enabled: true
diff --git a/molecule/molecule/default/converge.yml b/molecule/molecule/default/converge.yml
@@ -4,9 +4,9 @@
 
   tasks:
     - name: Update apt cache (on Debian).
-      apt:
+      ansible.builtin.apt:
         update_cache: true
         cache_valid_time: 3600
       when: ansible_os_family == 'Debian'
 
-- import_playbook: ../../main.yml
+- ansible.builtin.import_playbook: ../../main.yml
diff --git a/molecule/molecule/default/verify.yml b/molecule/molecule/default/verify.yml
@@ -4,6 +4,6 @@
 
   tasks:
   - name: Verify Apache is serving web requests.
-    uri:
+    ansible.builtin.uri:
       url: http://localhost/
       status_code: 200
