diff options
| author |   Michal Zegan <m.zegan@samsung.com> | 2019-05-22 15:52:37 +0200 |
|---|---|---|
| committer | Michal Zegan <m.zegan@samsung.com> | 2019-05-22 18:08:06 +0200 |
| commit | f2b9a04b5bc1f8c22cba4ba587f0526306d92ac8 (patch) | |
| tree | 0a7d77578803383c90e4c08b401a922a142050e4 /ansible | |
| parent | b1fe78553ea7d1caf90c1d52e195cd8f2a082008 (diff) | |
Make molecule tests for certificates independent from docker role
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>
Diffstat (limited to 'ansible')
| -rw-r--r-- | ansible/roles/certificates/molecule/default/playbook.yml | 9 | ||||
| -rw-r--r-- | ansible/roles/certificates/molecule/default/prepare.yml | 5 |
2 files changed, 6 insertions, 8 deletions
diff --git a/ansible/roles/certificates/molecule/default/playbook.yml b/ansible/roles/certificates/molecule/default/playbook.yml index 5dcd42ee..13583ebb 100644 --- a/ansible/roles/certificates/molecule/default/playbook.yml +++ b/ansible/roles/certificates/molecule/default/playbook.yml @@ -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 index 8df759c9..00000000 --- a/ansible/roles/certificates/molecule/default/prepare.yml +++ /dev/null @@ -1,5 +0,0 @@ ---- -- name: Prepare infra - hosts: all - roles: - - prepare-docker |