Processed feedback

handlers/main.yml

@@ -1,13 +1,4 @@
 ---
-- name: Run postmap
-  command: postmap /etc/postfix/{{ file['name'] }}
-  when: file['postmap'] | default(false) | bool
-  no_log: true
-  loop: "{{ postfix_files }}"
-  loop_control:
-    loop_var: file
-  changed_when: false
-
 - name: Check postfix
   command: postfix check
   when: postfix_check

tasks/main.yml

@@ -108,16 +108,27 @@
     dest: /etc/postfix/{{ file['name'] }}
     owner: root
     group: root
-    mode: '0644'
+    mode: '0640'
   loop: "{{ postfix_files }}"
+  register: __postfix_postmap_files
   no_log: true
   loop_control:
     loop_var: file
   notify:
-    - Run postmap
     - Check postfix
     - Restart postfix
 
+- name: Postmap files
+  command: postmap /etc/postfix/{{ file['name'] }}
+  when:
+    - file['changed'] | bool
+    - file['file']['postmap'] is defined
+    - file['file']['postmap'] | bool
+  no_log: true
+  loop: "{{ __postfix_postmap_files['results'] }}"
+  loop_control:
+    loop_var: file
+
 - name: Apply changes
   when: __postfix_has_config_changed | d("") is search("True")
   block:

tests/tests_set_file.yml

@@ -23,3 +23,11 @@
     - name: Assert file is present
       assert:
         that: test_file.stat.exists
+
+    - name: Clean up test files
+      file:
+        path: "{{ item }}"
+        state: absent
+      loop:
+        - /etc/postfix/test
+        - /etc/postfix/test.db