Make molecule tests for certificates independent from docker role 47/88247/2
authorMichal Zegan <m.zegan@samsung.com>
Wed, 22 May 2019 13:52:37 +0000 (15:52 +0200)
committerMichal Zegan <m.zegan@samsung.com>
Wed, 22 May 2019 16:08:06 +0000 (18:08 +0200)
Currently molecule tests for certificates role call the
docker role just to make one handler available.
It actually breaks best practices of unit testing,
the docker role shouldn't be called nor tested in
unit test of certificates role.
This commit makes certificates role independent by just
creating fake handlers.

Change-Id: Iec7934c7c1f4e9b37ff523e1d86a3beaaa88bdcf
Issue-ID: OOM-1864
Signed-off-by: Michal Zegan <m.zegan@samsung.com>
ansible/roles/certificates/molecule/default/playbook.yml
ansible/roles/certificates/molecule/default/prepare.yml [deleted file]

index 5dcd42e..13583eb 100644 (file)
@@ -1,14 +1,17 @@
 ---
 - name: Infra
   hosts: infrastructure
+  handlers:
+    - name: Restart Docker
+      debug: msg="Docker restart called"
   roles:
     - certificates
-    - docker  # docker role needed here just because of docker restart handler
 
 - name: Kube
   hosts: kubernetes
-  roles:
-    - docker  # docker role needed here just because of docker restart handler
+  handlers:
+    - name: Restart Docker
+      debug: msg="Docker restart called"
   tasks:
     - include_role:
         name: certificates
diff --git a/ansible/roles/certificates/molecule/default/prepare.yml b/ansible/roles/certificates/molecule/default/prepare.yml
deleted file mode 100644 (file)
index 8df759c..0000000
+++ /dev/null
@@ -1,5 +0,0 @@
----
-- name: Prepare infra
-  hosts: all
-  roles:
-    - prepare-docker