Fix multi user tests

The tests test_users_present.yml and test_users_absent.yml has been
merged into test_users_present_absent.yml to fix the independent test
order execution.

The test test_users_present_slice.yml has been renamed to
test_users_present_absent_slice.yml.

Both tests are cleaning up possible exiting entries before running the tests
and are verifying the present and absent task result.

tests/user/test_users_present.yml

----
-- name: Include create_users_json.yml
-  ansible.builtin.import_playbook: create_users_json.yml
-
-- name: Test users present
-  hosts: ipaserver
-  become: true
-  gather_facts: false
-
-  tasks:
-  - name: Include users.json
-    ansible.builtin.include_vars:
-      file: users.json  # noqa missing-import
-
-  - name: Users present len:{{ users | length }}
-    ipauser:
-      ipaadmin_password: SomeADMINpassword
-      users: "{{ users }}"

tests/user/test_users_absent.yml → tests/user/test_users_present_absent.yml

@@ -2,7 +2,7 @@
 - name: Include create_users_json.yml
   ansible.builtin.import_playbook: create_users_json.yml
 
-- name: Test users absent
+- name: Test users present and absent
   hosts: ipaserver
   become: true
   gather_facts: false
@@ -17,8 +17,23 @@
       user_names: "{{ user_names | default([]) + [{'name': item.name}] }}"
     loop: "{{ users }}"
 
+  - name: Cleanup test users len:{{ users | length }}
+    ipauser:
+      ipaadmin_password: SomeADMINpassword
+      users: "{{ user_names }}"
+      state: absent
+
+  - name: Users present len:{{ users | length }}
+    ipauser:
+      ipaadmin_password: SomeADMINpassword
+      users: "{{ users }}"
+    register: result
+    failed_when: not result.changed or result.failed
+
   - name: Users absent len:{{ users | length }}
     ipauser:
       ipaadmin_password: SomeADMINpassword
       users: "{{ user_names }}"
       state: absent
+    register: result
+    failed_when: not result.changed or result.failed

tests/user/test_users_present_slice.yml → tests/user/test_users_present_absent_slice.yml

@@ -2,22 +2,47 @@
 - name: Include create_users_json.yml
   ansible.builtin.import_playbook: create_users_json.yml
 
-- name: Test users present slice
+- name: Test users present and absent with slice
   hosts: ipaserver
   become: true
   gather_facts: false
 
   vars:
-    slice_size: 500
+    slice_size: 100
   tasks:
   - name: Include users.json
     ansible.builtin.include_vars:
       file: users.json  # noqa missing-import
+
+  - name: Create dict with user names
+    ansible.builtin.set_fact:
+      user_names: "{{ user_names | default([]) + [{'name': item.name}] }}"
+    loop: "{{ users }}"
+
   - name: Size of users slice.
     ansible.builtin.debug:
       msg: "{{ users | length }}"
+
+  - name: Cleanup test users
+    ipauser:
+      ipaadmin_password: SomeADMINpassword
+      users: "{{ user_names[item : item + slice_size] }}"
+      state: absent
+    loop: "{{ range(0, user_names | length, slice_size) | list }}"
+
   - name: Users present
     ipauser:
       ipaadmin_password: SomeADMINpassword
       users: "{{ users[item : item + slice_size] }}"
     loop: "{{ range(0, users | length, slice_size) | list }}"
+    register: result
+    failed_when: not result.changed or result.failed
+
+  - name: Users absent
+    ipauser:
+      ipaadmin_password: SomeADMINpassword
+      users: "{{ user_names[item : item + slice_size] }}"
+      state: absent
+    loop: "{{ range(0, user_names | length, slice_size) | list }}"
+    register: result
+    failed_when: not result.changed or result.failed