Add missing nfs_provider parameter to Ansible module and improve task description formatting

Author: devanshjainms

tests/roles/ha_scs/kill_sapstartsrv_test.py

@@ -55,7 +55,8 @@ def test_environment(self, ansible_inventory):
             role_type="ha_scs",
             ansible_inventory=ansible_inventory,
             task_name="kill-sapstartsrv-process",
-            task_description="The SAP startsrv Process Kill test simulates failure of the sapstartsrv process",
+            task_description="The SAP startsrv Process Kill test simulates "
+            + "failure of the sapstartsrv process",
             module_names=[
                 "project/library/get_cluster_status_scs",
                 "project/library/log_parser",
@@ -67,6 +68,16 @@ def test_environment(self, ansible_inventory):
             extra_vars_override={"node_tier": "scs"},
         )
 
+        playbook_content = self.file_operations(
+            operation="read",
+            file_path=f"{temp_dir}/project/roles/ha_scs/tasks/kill-sapstartsrv-process.yml",
+        )
+        self.file_operations(
+            operation="write",
+            file_path=f"{temp_dir}/project/roles/ha_scs/tasks/kill-sapstartsrv-process.yml",
+            content=playbook_content.replace("set -o pipefail &&", ""),
+        )
+
         yield temp_dir
         shutil.rmtree(temp_dir)
 

tests/roles/mock_data/get_pcmk_properties_scs.txt

@@ -10,7 +10,8 @@ def main():
             ansible_os_family=dict(type='str', required=True),
             virtual_machine_name=dict(type='str', required=True),
             pcmk_constants=dict(type='dict', required=True),
-            fencing_mechanism=dict(type='str', required=True)
+            fencing_mechanism=dict(type='str', required=True),
+            nfs_provider=dict(type='str', required=True)
         )
     )