From 54a62e545eee489c8c29eaaaac41dc51aa421d06 Mon Sep 17 00:00:00 2001
From: anna-savina <asavina@redhat.com>
Date: Tue, 16 Jul 2024 10:50:20 +0300
Subject: [PATCH] fixed always block for Manage folder tests

---
 .../tasks/main.yml                            | 103 +++++++++---------
 1 file changed, 52 insertions(+), 51 deletions(-)

diff --git a/tests/integration/targets/vmware_ops_manage_folder_test/tasks/main.yml b/tests/integration/targets/vmware_ops_manage_folder_test/tasks/main.yml
index 8c6cee49..87a84d85 100644
--- a/tests/integration/targets/vmware_ops_manage_folder_test/tasks/main.yml
+++ b/tests/integration/targets/vmware_ops_manage_folder_test/tasks/main.yml
@@ -1,64 +1,65 @@
 ---
-- name: Import common vars for tests on vCenter Environment
-  ansible.builtin.include_vars:
-    file: ../group_vars.yml
-  when: not run_on_simulator
+- name: Run Manage folder tests
+  block:
+    - name: Import common vars for tests on vCenter Environment
+      ansible.builtin.include_vars:
+        file: ../group_vars.yml
+      when: not run_on_simulator
 
-- name: Create Folder Tree
-  loop: "{{ folder_types }}"
-  ansible.builtin.include_role:
-    name: cloud.vmware_ops.manage_folder
-  vars:
-    manage_folder_state: present
-    manage_folder_folder_name: production/foo/web
-    manage_folder_folder_type: "{{ item }}"
+    - name: Create Folder Tree
+      loop: "{{ folder_types }}"
+      ansible.builtin.include_role:
+        name: cloud.vmware_ops.manage_folder
+      vars:
+        manage_folder_state: present
+        manage_folder_folder_name: production/foo/web
+        manage_folder_folder_type: "{{ item }}"
 
-- name: Create Folder Without Managing Full Tree
-  loop: "{{ folder_types }}"
-  ansible.builtin.include_role:
-    name: cloud.vmware_ops.manage_folder
-  vars:
-    manage_folder_state: present
-    manage_folder_folder_name: db
-    manage_folder_parent_folder: production/foo
-    manage_folder_folder_type: "{{ item }}"
+    - name: Create Folder Without Managing Full Tree
+      loop: "{{ folder_types }}"
+      ansible.builtin.include_role:
+        name: cloud.vmware_ops.manage_folder
+      vars:
+        manage_folder_state: present
+        manage_folder_folder_name: db
+        manage_folder_parent_folder: production/foo
+        manage_folder_folder_type: "{{ item }}"
 
-- name: Create A Folder With A Slash In It
-  loop: "{{ folder_types }}"
-  ansible.builtin.include_role:
-    name: cloud.vmware_ops.manage_folder
-  vars:
-    manage_folder_state: present
-    manage_folder_folder_name: security/syseng
-    manage_folder_parent_folder: production/foo
-    manage_folder_parse_name_as_path: false
-    manage_folder_folder_type: "{{ item }}"
+    - name: Create A Folder With A Slash In It
+      loop: "{{ folder_types }}"
+      ansible.builtin.include_role:
+        name: cloud.vmware_ops.manage_folder
+      vars:
+        manage_folder_state: present
+        manage_folder_folder_name: security/syseng
+        manage_folder_parent_folder: production/foo
+        manage_folder_parse_name_as_path: false
+        manage_folder_folder_type: "{{ item }}"
 
-- name: Get Folder Info
-  community.vmware.vmware_folder_info:
-    hostname: "{{ manage_folder_hostname }}"
-    username: "{{ manage_folder_username }}"
-    password: "{{ manage_folder_password }}"
-    datacenter: "{{ manage_folder_datacenter_name }}"
-    port: "{{ manage_folder_port }}"
-    validate_certs: false
-  delegate_to: localhost
-  register: _folder_info
+    - name: Get Folder Info
+      community.vmware.vmware_folder_info:
+        hostname: "{{ manage_folder_hostname }}"
+        username: "{{ manage_folder_username }}"
+        password: "{{ manage_folder_password }}"
+        datacenter: "{{ manage_folder_datacenter_name }}"
+        port: "{{ manage_folder_port }}"
+        validate_certs: false
+      delegate_to: localhost
+      register: _folder_info
 
-- name: Check Folders Were Created
-  ansible.builtin.assert:
-    that:
-      - _folder_info.folder_info.hostFolders.subfolders.production.subfolders.foo.subfolders['web'] is defined
-      - _folder_info.folder_info.hostFolders.subfolders.production.subfolders.foo.subfolders['db'] is defined
-      - _folder_info.folder_info.hostFolders.subfolders.production.subfolders.foo.subfolders['security%2fsyseng'] is defined
-    fail_msg: Folder structure does not match expected result.
+    - name: Check Folders Were Created
+      ansible.builtin.assert:
+        that:
+          - _folder_info.folder_info.hostFolders.subfolders.production.subfolders.foo.subfolders['web'] is defined
+          - _folder_info.folder_info.hostFolders.subfolders.production.subfolders.foo.subfolders['db'] is defined
+          - _folder_info.folder_info.hostFolders.subfolders.production.subfolders.foo.subfolders['security%2fsyseng'] is defined
+        fail_msg: Folder structure does not match expected result.
 
-- name: Run Post Test Operations
-  block:
+  always:
     - name: Start Cleanup
       ansible.builtin.debug:
         msg: "Running cleanup..."
-  always:
+
     - name: Delete The Whole Tree
       loop: "{{ folder_types }}"
       ansible.builtin.include_role: