Refactor sapcontrol command validation to enhance test case detail aggregation and improve readability

Author: devanshjainms

.gitignore

@@ -403,6 +403,7 @@ FodyWeavers.xsd
 .vscode
 
 # System configuration files
+WORKSPACES/*
 .ppk
 VMWPASSWORD
 .coverage.*

requirements.txt

@@ -23,7 +23,7 @@ attrs==25.3.0
     #   referencing
 azure-common==1.1.28
     # via azure-mgmt-network
-azure-core==1.33.0
+azure-core==1.34.0
     # via
     #   azure-identity
     #   azure-kusto-data
@@ -34,17 +34,17 @@ azure-identity==1.21.0
     # via
     #   -r requirements.in
     #   azure-kusto-data
-azure-kusto-data==5.0.2
+azure-kusto-data==5.0.3
     # via
     #   -r requirements.in
     #   azure-kusto-ingest
-azure-kusto-ingest==5.0.2
+azure-kusto-ingest==5.0.3
     # via -r requirements.in
 azure-mgmt-core==1.5.0
     # via azure-mgmt-network
 azure-mgmt-network==28.1.0
     # via -r requirements.in
-azure-storage-blob==12.25.1
+azure-storage-blob==12.23.0
     # via
     #   -r requirements.in
     #   azure-kusto-ingest
@@ -62,7 +62,7 @@ certifi==2025.4.26
     # via requests
 cffi==1.17.1
     # via cryptography
-charset-normalizer==3.4.1
+charset-normalizer==3.4.2
     # via requests
 click==8.1.8
     # via
@@ -72,7 +72,7 @@ coverage[toml]==7.8.0
     # via
     #   -r requirements.in
     #   pytest-cov
-cryptography==44.0.2
+cryptography==44.0.3
     # via
     #   ansible-core
     #   azure-identity
@@ -169,7 +169,7 @@ pyjwt[crypto]==2.10.1
     # via
     #   msal
     #   pyjwt
-pylint==3.3.6
+pylint==3.3.7
     # via -r requirements.in
 pytest==8.3.5
     # via
@@ -254,7 +254,7 @@ urllib3==2.4.0
     # via requests
 wcmatch==10.0
     # via ansible-lint
-yamllint==1.37.0
+yamllint==1.37.1
     # via ansible-lint
 zipp==3.21.0
     # via importlib-metadata

src/roles/ha_scs/tasks/sapcontrol-config.yml

@@ -57,18 +57,19 @@
               {% for res in sapcontrol_results.results %}
                 - {{ res.sapcontrol_method }}: {{ 'Success' if res.rc == 0 else 'Failed' }}
               {% endfor %}
+
+        - name:                         "Test Execution: Validate sapcontrol commands"
+          ansible.builtin.set_fact:
             test_case_details_from_test_case: >-
-                                              {{
-                                                [
-                                                  {
-                                                    'command':    item.cmd,
-                                                    'returnCode': item.rc,
-                                                    'stdout':     item.stdout_lines,
-                                                    'stderr':     item.stderr_lines
-                                                  }
-                                                  for item in sapcontrol_results.results
-                                                ]
-                                              }}
+                                        {{ test_case_details_from_test_case + [{
+                                            'command': sapcontrol_result.cmd,
+                                            'returnCode': sapcontrol_result.rc,
+                                            'stdout': sapcontrol_result.stdout_lines,
+                                            'stderr': sapcontrol_result.stderr_lines
+                                          }] }}
+          loop:                         "{{ sapcontrol_results.results }}"
+          loop_control:
+            loop_var:                   sapcontrol_result
 
 # /*---------------------------------------------------------------------------
 # |                          Post Validations                                 |