Fix nginx_config_cleanup_paths not working as intended (#99)

Author: alessfg

CHANGELOG.md

@@ -58,6 +58,7 @@ BUG FIXES:
 *   Add `state` parameter to package module in Molecule verification tests.
 *   In NGINX App Protect environments on SELinux enforced systems, the `nginx -t` handler fails when run from a directory that the NGINX process' user does not have access to.
 *   Fix missing GRPC boolean check in GRPC template.
+*   Fix `nginx_config_cleanup_paths` not working as intended.
 
 ## 0.3.3 (January 28, 2021)
 

molecule/cleanup_module/prepare.yml

@@ -8,3 +8,8 @@
       vars:
         nginx_modules:
           - njs
+
+    - name: Create mock NGINX configuration file
+      file:
+        path: /etc/nginx/conf.d/mock.conf
+        state: touch

molecule/cleanup_module/verify.yml

@@ -25,3 +25,10 @@
       check_mode: yes
       register: stat_result
       failed_when: stat_result.stat.exists
+
+    - name: Check mock.conf does not exist
+      stat:
+        path: /etc/nginx/conf.d/mock.conf
+      check_mode: yes
+      register: stat_result
+      failed_when: stat_result.stat.exists

tasks/config/cleanup-config.yml

@@ -12,6 +12,6 @@
   file:
     path: "{{ item }}"
     state: absent
-  loop: "{{ nginx_config_files.files | default('') | map(attribute='path') |
-        list + nginx_config_cleanup_files | default('') | list }}"
+  loop: "{{ nginx_config_files.results | map(attribute='files') | sum(start=[]) | map(attribute='path') | list
+        + nginx_config_cleanup_files | default('') | list }}"
   notify: (Handler - NGINX Config) Run NGINX