[Bugfix] [zos_copy] Copy from file to member when dest is just PDS/E name (#1570)

* Compute member name for implicit copies

* Add test for implicit member copy

* Update module doc

* Add changelog fragment

Author: rexemin

changelogs/fragments/1570-compute-member-name-zos_copy.yml

@@ -0,0 +1,6 @@
+bugfixes:
+  - zos_copy - a regression in version 1.4.0 made the module stop automatically
+    computing member names when copying a single file into a PDS/E. Fix now
+    lets a user copy a single file into a PDS/E without adding a member in the
+    dest option.
+    (https://github.com/ansible-collections/ibm_zos_core/pull/1570).

plugins/modules/zos_copy.py

@@ -109,6 +109,9 @@
         attributes will be computed. If I(executable=true),C(dest) will have an Undefined (U) record
         format with a record length of 0, block size of 32760, and the remaining attributes will be
         computed.
+      - If C(src) is a file and C(dest) a partitioned data set, C(dest) does not need to include
+        a member in its value, the module can automatically compute the resulting member name from
+        C(src).
       - When C(dest) is a data set, precedence rules apply. If C(dest_data_set)
         is set, this will take precedence over an existing data set. If C(dest)
         is an empty data set, the empty data set will be written with the
@@ -3429,6 +3432,13 @@ def run_module(module, arg_def):
                     dest_has_asa_chars = True
 
             if dest_ds_type in data_set.DataSet.MVS_PARTITIONED:
+                # Checking if we need to copy a member when the user requests it implicitly.
+                # src is a file and dest was just the PDS/E dataset name.
+                if not copy_member and src_ds_type == "USS" and os.path.isfile(src):
+                    copy_member = True
+                    dest_member = data_set.DataSet.get_member_name_from_file(os.path.basename(src))
+                    dest = f"{dest_name}({dest_member})"
+
                 # Checking if the members that would be created from the directory files
                 # are already present on the system.
                 if copy_member:

tests/functional/modules/test_zos_copy_func.py

@@ -2595,6 +2595,46 @@ def test_copy_file_to_non_existing_member(ansible_zos_module, src):
         hosts.all.zos_data_set(name=data_set, state="absent")
 
 
+# Test related to issue #774: https://github.com/ansible-collections/ibm_zos_core/issues/774.
+@pytest.mark.uss
+@pytest.mark.pdse
+def test_copy_file_to_non_existing_member_implicit(ansible_zos_module):
+    hosts = ansible_zos_module
+    dest_data_set = get_tmp_ds_name()
+    dest_member = f"{dest_data_set}(PROFILE)"
+
+    try:
+        hosts.all.zos_data_set(
+            name=dest_data_set,
+            type="pdse",
+            space_primary=5,
+            space_type="m",
+            record_format="fba",
+            record_length=80,
+            replace=True
+        )
+
+        copy_result = hosts.all.zos_copy(
+            src="/etc/profile",
+            dest=dest_data_set,
+            remote_src=True
+        )
+
+        verify_copy = hosts.all.shell(
+            cmd="cat \"//'{0}'\" > /dev/null 2>/dev/null".format(dest_member),
+            executable=SHELL_EXECUTABLE,
+        )
+
+        for cp_res in copy_result.contacted.values():
+            assert cp_res.get("msg") is None
+            assert cp_res.get("changed") is True
+            assert cp_res.get("dest") == dest_member
+        for v_cp in verify_copy.contacted.values():
+            assert v_cp.get("rc") == 0
+    finally:
+        hosts.all.zos_data_set(name=dest_data_set, state="absent")
+
+
 @pytest.mark.uss
 @pytest.mark.pdse
 @pytest.mark.parametrize("src", [