copy pds into new target data set enhancement

[pujal0909]

What It Does

The enhancement creates a new PDS and copies the source members when a target PDS is inputted that does not exist. The response informs the user a new PDS has been created.
#2349
How to Test

Use the copy command with a target PDS that does not currently exist
zowe zos-files copy ds "existing-pds" "new-pds"

Review Checklist
I certify that I have:

• tested my changes
• added/updated automated tests
• updated the changelog
• followed the contribution guidelines

Additional Comments

[codecov]

Codecov Report

Attention: Patch coverage is 91.30435% with 2 lines in your changes missing coverage. Please review.

Project coverage is 91.32%. Comparing base (5f86044) to head (be0c2e3).
Report is 16 commits behind head on master.

Files with missing lines	Patch %	Lines
packages/zosfiles/src/methods/copy/Copy.ts	91.30%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2393      +/-   ##
==========================================
- Coverage   91.32%   91.32%   -0.01%     
==========================================
  Files         638      638              
  Lines       18210    18231      +21     
  Branches     3931     3942      +11     
==========================================
+ Hits        16630    16649      +19     
- Misses       1579     1580       +1     
- Partials        1        2       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

📅 Suggested merge-by date: 1/20/2025

[t1m0thyj]

Looks pretty good, thanks @pujal0909! Left a few comments

[t1m0thyj]

Tested the functionality and it LGTM 🙂

@pujal0909 Please add a system test for the case where target data set does not exist.

Question for @adam-wolfe and @anaxceron - should we update the help text for the zowe copy ds command to explain that if the target data set does not exist, it will be created?

Requested changes addressed

[adam-wolfe]

Tested the functionality and it LGTM 🙂

@pujal0909 Please add a system test for the case where target data set does not exist.

Question for @adam-wolfe and @anaxceron - should we update the help text for the zowe copy ds command to explain that if the target data set does not exist, it will be created?

I think so. I would also change this line:

[anaxceron]

Left a minor comment re: a changelog.

[traeok]

Functionality LGTM, thanks Pujal! I had couple small requests for changes and a follow-up question...

---

In the event where a user tries to copy to the same data set, can we return early from the command? Currently, if I try copying a PDS from source A.B.C to the same destination (A.B.C), the command copies each member, even though the source and destination are identical:

[t1m0thyj]

LGTM, thanks @pujal0909!

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
96.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud