Refactor package hold verification tests (#25)

nephelaiio · Mar 27, 2024 · 794e082 · 794e082
1 parent 440282e
commit 794e082
Showing 1 changed file with 14 additions and 10 deletions.
diff --git a/molecule/hold/verify.yml b/molecule/hold/verify.yml
@@ -63,27 +63,31 @@
       ansible.builtin.package_facts:
 
     - name: Verify package versions
-      ansible.builtin.fail:
-        msg: "Expected installed version for package {{ _package }} to be {{ _version }}, found {{ _installed }}"
+      ansible.builtin.assert:
+        that:
+          - _package in packages
+          - _version == _installed
+        fail_msg: "Expected {{ _package }}={{ _version }}, found {{ _package }}={{ _installed }}"
+        success_msg: "Found {{ _package }}={{ _version }}"
       vars:
         _package: "{{ item.split('=') | first }}"
         _version: "{{ item.split('=') | last }}"
         _installed: "{{ (packages[_package] | default([{'version': 'None'}]))[0].version }}"
       loop: "{{ mongos_package_list }}"
-      when:
-        - ansible_os_family == "Debian"
-        - _package not in packages or _version != _installed
+      when: ansible_os_family == "Debian"
 
     - name: Verify package versions
-      ansible.builtin.fail:
-        msg: "Expected installed version for package {{ _package }} to be {{ _version }}, found {{ _installed }}"
+      ansible.builtin.assert:
+        that:
+          - _package in packages
+          - _version == _installed
+        fail_msg: "Expected {{ _package }}={{ _version }}, found {{ _package }}={{ _installed }}"
+        success_msg: "Found {{ _package }}={{ _version }}"
       vars:
         _prefix: "^{{ _package }}-"
         _suffix: "-.*$"
         _package: "{{ item.0 }}"
         _version: "{{ item.1 | regex_replace(_prefix, '') | regex_replace(_suffix, '') }}"
         _installed: "{{ (packages[_package] | default([{'version': 'None'}]))[0].version }}"
       loop: "{{ mongos_packages | zip(mongos_package_list) }}"
-      when:
-        - ansible_os_family == "RedHat"
-        - _package not in packages or _version != _installed
+      when: ansible_os_family == "RedHat"