majic-ansible-roles Changeset

Changeset - 7180fd36dda8

Parent rev.

Child rev.

[Not reviewed]

5 2 8

Branko Majic (branko) - 6 years ago 2017-11-26 10:45:05
branko@majic.rs

MAR-128: Upgraded tests for ROLE_NAME role:

- Switch to new Molecule configuration.
- Updated set-up playbook to use become: yes.
- Moved some preparatory steps outside of the main playbook (eases
idempotence tests).
- Updated tests to reference the yml inventory file.
- Updated tests to use new fixture (host instead of individual ones).

11 files changed with 183 insertions and 52 deletions:

roles/ldap_client/defaults/main.yml

roles/ldap_client/handlers/main.yml

roles/ldap_client/molecule.yml

roles/ldap_client/molecule/default/create.yml

roles/ldap_client/molecule/default/destroy.yml

roles/ldap_client/molecule/default/molecule.yml

roles/ldap_client/molecule/default/playbook.yml

roles/ldap_client/molecule/default/prepare.yml

roles/ldap_client/molecule/default/tests/test_default.py

roles/ldap_client/molecule/default/tests/test_mandatory.py

roles/ldap_client/molecule/default/tests/test_optional.py

0 comments (0 inline, 0 general)

roles/ldap_client/defaults/main.yml

➞

Show inline comments

 ---
 ldap_client_config: []
@@ \ No newline at end of file @@
 ldap_client_config: []

roles/ldap_client/handlers/main.yml

➞

Show inline comments

		---
		\ No newline at end of file
		---

roles/ldap_client/molecule.yml

➞

Show inline comments

deleted file

roles/ldap_client/molecule/default/create.yml

➞

Show inline comments

@@ new file 100644 @@
 ---
 - name: Create
   hosts: localhost
   connection: local
   gather_facts: False
   no_log: "{{ not lookup('env', 'MOLECULE_DEBUG') | bool }}"
   vars:
     molecule_file: "{{ lookup('env', 'MOLECULE_FILE') }}"
     molecule_instance_config: "{{ lookup('env', 'MOLECULE_INSTANCE_CONFIG') }}"
     molecule_yml: "{{ lookup('file', molecule_file) | molecule_from_yaml }}"
   tasks:
     - name: Create molecule instance(s)
       molecule_vagrant:
         instance_name: "{{ item.name }}"
         instance_interfaces: "{{ item.interfaces | default(omit) }}"
         instance_raw_config_args: "{{ item.instance_raw_config_args | default(omit) }}"
         platform_box: "{{ item.box }}"
         platform_box_version: "{{ item.box_version | default(omit) }}"
         platform_box_url: "{{ item.box_url | default(omit) }}"
         provider_name: "{{ molecule_yml.driver.provider.name }}"
         provider_memory: "{{ item.memory | default(omit) }}"
         provider_cpus: "{{ item.cpus | default(omit) }}"
         provider_raw_config_args: "{{ item.raw_config_args | default(omit) }}"
         state: up
       register: server
       with_items: "{{ molecule_yml.platforms }}"
     # Mandatory configuration for Molecule to function.
     - name: Populate instance config dict
       set_fact:
         instance_conf_dict: {
           'instance': "{{ item.Host }}",
           'address': "{{ item.HostName }}",
           'user': "{{ item.User }}",
           'port': "{{ item.Port }}",
           'identity_file': "{{ item.IdentityFile }}", }
       with_items: "{{ server.results }}"
       register: instance_config_dict
       when: server.changed | bool
     - name: Convert instance config dict to a list
       set_fact:
         instance_conf: "{{ instance_config_dict.results | map(attribute='ansible_facts.instance_conf_dict') | list }}"
       when: server.changed | bool
     - name: Dump instance config
       copy:
         # NOTE(retr0h): Workaround for Ansible 2.2.
         #               https://github.com/ansible/ansible/issues/20885
         content: "{{ instance_conf | to_json | from_json | molecule_to_yaml | molecule_header }}"
         dest: "{{ molecule_instance_config }}"
       when: server.changed | bool

roles/ldap_client/molecule/default/destroy.yml

➞

Show inline comments

@@ new file 100644 @@
 ---
 - name: Destroy
   hosts: localhost
   connection: local
   gather_facts: False
   no_log: "{{ not lookup('env', 'MOLECULE_DEBUG') | bool }}"
   vars:
     molecule_file: "{{ lookup('env', 'MOLECULE_FILE') }}"
     molecule_instance_config: "{{ lookup('env',' MOLECULE_INSTANCE_CONFIG') }}"
     molecule_yml: "{{ lookup('file', molecule_file) | molecule_from_yaml }}"
   tasks:
     - name: Destroy molecule instance(s)
       molecule_vagrant:
         instance_name: "{{ item.name }}"
         platform_box: "{{ item.box }}"
         provider_name: "{{ molecule_yml.driver.provider.name }}"
         force_stop: "{{ item.force_stop | default(True) }}"
         state: destroy
       register: server
       with_items: "{{ molecule_yml.platforms }}"
     # Mandatory configuration for Molecule to function.
     - name: Populate instance config
       set_fact:
         instance_conf: {}
     - name: Dump instance config
       copy:
         # NOTE(retr0h): Workaround for Ansible 2.2.
         #               https://github.com/ansible/ansible/issues/20885
         content: "{{ instance_conf | to_json | from_json | molecule_to_yaml | molecule_header }}"
         dest: "{{ molecule_instance_config }}"
       when: server.changed | bool

roles/ldap_client/molecule/default/molecule.yml

➞

Show inline comments

@@ new file 100644 @@
 ---
 dependency: {}
 driver:
   name: vagrant
   provider:
     name: virtualbox
 lint:
   name: yamllint
 platforms:
   - name: parameters-mandatory-jessie64
     groups:
       - parameters-mandatory
     box: debian/contrib-jessie64
     memory: 256
     cpus: 1
   - name: parameters-optional-jessie64
     groups:
       - parameters-optional
     box: debian/contrib-jessie64
     memory: 256
     cpus: 1
   - name: parameters-mandatory-stretch64
     groups:
       - parameters-mandatory
     box: debian/contrib-stretch64
     memory: 256
     cpus: 1
   - name: parameters-optional-stretch64
     groups:
       - parameters-optional
     box: debian/contrib-stretch64
     memory: 256
     cpus: 1
 provisioner:
   name: ansible
   config_options:
     ssh_connection:
       pipelining: "True"
   lint:
     name: ansible-lint
 scenario:
   name: default
 verifier:
   name: testinfra
   lint:
     name: flake8

roles/ldap_client/molecule/default/playbook.yml

➞

Show inline comments

 ---
 - hosts: all
   tasks:
     - name: Update all caches to avoid errors due to missing remote archives
       apt:
         update_cache: yes
       changed_when: False
 - hosts: parameters-mandatory
   become: yes
   roles:
     - ldap_client
 - hosts: parameters-optional
   become: yes
   roles:
     - role: ldap_client
       ldap_client_config:

roles/ldap_client/molecule/default/prepare.yml

➞

Show inline comments

@@ new file 100644 @@
 ---
 - name: Prepare
   hosts: all
   gather_facts: False
   tasks:
     - name: Install python for Ansible
       raw: test -e /usr/bin/python || (apt -y update && apt install -y python-minimal)
       become: True
       changed_when: False
 - hosts: all
   become: yes
   tasks:
     - name: Update all caches to avoid errors due to missing remote archives
       apt:
         update_cache: yes
       changed_when: False

roles/ldap_client/molecule/default/tests/test_default.py

➞

Show inline comments

 import testinfra.utils.ansible_runner
 testinfra_hosts = testinfra.utils.ansible_runner.AnsibleRunner(
     '.molecule/ansible_inventory').get_hosts('all')
+    '.molecule/ansible_inventory.yml').get_hosts('all')
-def test_installed_packages(Package):
+def test_installed_packages(host):
     """
     Tests if correct packages are installed.
     """
-    assert Package('ldap-utils').is_installed
+    assert host.package('ldap-utils').is_installed
-def test_ldap_configuration_file(File):
+def test_ldap_configuration_file(host):
     """
     Tests if LDAP configuration files has been deployed with correct
     permissions.
     """
-    config = File('/etc/ldap/ldap.conf')
+    config = host.file('/etc/ldap/ldap.conf')
     assert config.is_file
     assert config.user == 'root'

roles/ldap_client/molecule/default/tests/test_mandatory.py

➞

Show inline comments

 import testinfra.utils.ansible_runner
 testinfra_hosts = testinfra.utils.ansible_runner.AnsibleRunner(
     '.molecule/ansible_inventory').get_hosts('parameters-mandatory')
+    '.molecule/ansible_inventory.yml').get_hosts('parameters-mandatory')
-def test_ldap_configuration_file_content(File):
+def test_ldap_configuration_file_content(host):
     """
     Tests if LDAP configuration file has correct content
     """
-    config = File('/etc/ldap/ldap.conf')
+    config = host.file('/etc/ldap/ldap.conf')
     assert config.content == ""

roles/ldap_client/molecule/default/tests/test_optional.py

➞

Show inline comments

 import testinfra.utils.ansible_runner
 testinfra_hosts = testinfra.utils.ansible_runner.AnsibleRunner(
     '.molecule/ansible_inventory').get_hosts('parameters-optional')
+    '.molecule/ansible_inventory.yml').get_hosts('parameters-optional')
-def test_ldap_configuration_file_content(File):
+def test_ldap_configuration_file_content(host):
     """
     Tests if LDAP configuration file has correct content
     """
@@ @@ -18,6 +18,6 @@ URI ldaps://ldap-server/ @@
 # Base entry
 BASE dc=local"""
-    config = File('/etc/ldap/ldap.conf')
+    config = host.file('/etc/ldap/ldap.conf')
     assert config.content == expected_content

0 comments (0 inline, 0 general)