From 726d769d26c7d1c874709fe06af86c2905892264 Mon Sep 17 00:00:00 2001
From: Emilien Mantel <emilien.mantel@triplestack.fr>
Date: Tue, 30 May 2023 12:52:46 +0200
Subject: [PATCH] :green_heart: Fix linters errors

---
 molecule/_shared/prepare.yml    | 17 +----------------
 molecule/_shared/vars/misc.yml  |  2 +-
 molecule/_shared/verify.yml     |  2 +-
 molecule/debian-10/molecule.yml |  2 +-
 tests/test.yml                  |  8 ++++----
 5 files changed, 8 insertions(+), 23 deletions(-)

diff --git a/molecule/_shared/prepare.yml b/molecule/_shared/prepare.yml
index 8bd3c10..3920a6e 100644
--- a/molecule/_shared/prepare.yml
+++ b/molecule/_shared/prepare.yml
@@ -41,21 +41,6 @@
       ansible.builtin.service:
         name: nginx
         state: started
-      when: ansible_virtualization_type != 'docker'
-
-    - name: Start nginx if testing with Docker
-      when: ansible_virtualization_type == 'docker'
-      block:
-
-        - name: COMMAND | Docker nginx status
-          ansible.builtin.command: service nginx status
-          changed_when: false
-          failed_when: false
-          register: ngs
-
-        - name: COMMAND | Docker start nginx
-          ansible.builtin.command: service nginx start
-          when: ngs.stdout.find('nginx is not running') != -1
 
     - name: FILE | Create /var/www
       ansible.builtin.file:
@@ -79,4 +64,4 @@
         content: '<?php echo ini_get("memory_limit") . "\n";'
         owner: root
         group: root
-        mode: 0644
\ No newline at end of file
+        mode: 0644
diff --git a/molecule/_shared/vars/misc.yml b/molecule/_shared/vars/misc.yml
index b6ee183..ff486cb 100644
--- a/molecule/_shared/vars/misc.yml
+++ b/molecule/_shared/vars/misc.yml
@@ -33,4 +33,4 @@ php_fpm_poold:
     php_value:
       display_errors: 'Off'
     php_admin_value:
-      memory_limit: '98M'
\ No newline at end of file
+      memory_limit: '98M'
diff --git a/molecule/_shared/verify.yml b/molecule/_shared/verify.yml
index 187845c..6d8683d 100644
--- a/molecule/_shared/verify.yml
+++ b/molecule/_shared/verify.yml
@@ -39,7 +39,7 @@
           register: c
           failed_when: c.stdout == ''
 
-    - name: SHELL | Check custom php value
+    - name: SHELL | Check custom php value  # noqa: command-instead-of-module
       ansible.builtin.shell: "curl -H 'Host: {{ vhost }}' http://127.0.0.1/ini.php 2> /dev/null"
       changed_when: false
       register: c
diff --git a/molecule/debian-10/molecule.yml b/molecule/debian-10/molecule.yml
index bb7f019..46e4107 100644
--- a/molecule/debian-10/molecule.yml
+++ b/molecule/debian-10/molecule.yml
@@ -29,4 +29,4 @@ provisioner:
         __nginx_conf: /etc/nginx/nginx.conf
     host_vars:
       debian-10-php-7.4:
-        php_version: '7.4'
\ No newline at end of file
+        php_version: '7.4'
diff --git a/tests/test.yml b/tests/test.yml
index 0400045..65fefa9 100644
--- a/tests/test.yml
+++ b/tests/test.yml
@@ -78,13 +78,13 @@
       when: ansible_virtualization_type == 'docker'
       block:
 
-        - name: COMMAND | Docker nginx status
+        - name: COMMAND | Docker nginx status  # noqa: command-instead-of-module
           ansible.builtin.command: service nginx status
           changed_when: false
           failed_when: false
           register: ngs
 
-        - name: COMMAND | Docker start nginx
+        - name: COMMAND | Docker start nginx  # noqa: command-instead-of-module no-changed-when
           ansible.builtin.command: service nginx start
           when: ngs.stdout.find('nginx is not running') != -1
 
@@ -96,7 +96,7 @@
         state: reloaded
       notify: Docker reload nginx
 
-    - name: Docker reload nginx
+    - name: Docker reload nginx  # noqa: command-instead-of-module no-changed-when
       ansible.builtin.command: service nginx reload
       notify: Docker reload nginx
       when: ansible_virtualization_type == 'docker'
@@ -146,7 +146,7 @@
       register: c
       failed_when: c.stdout == ''
 
-    - name: SHELL | Check custom php value
+    - name: SHELL | Check custom php value  # noqa: command-instead-of-module
       ansible.builtin.shell: "curl -H 'Host: {{ vhost }}' http://127.0.0.1/ini.php 2> /dev/null"
       changed_when: false
       register: c