Support for drag/drop of MVS files across LPAR'S

[likhithanimma1]

Proposed changes

Implemented drag and drop functionality(move action) for the MVS files. Now user will be able to drag seq, pds and members from one LPAR and drop it on to another.

Release Notes

Milestone:

Changelog: Implemented drag and drop functionality of data sets within and across LPARs. [#3413]

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• Enhancement (non-breaking change which adds or improves functionality)
• Breaking change (a change that would cause existing functionality to not work as expected)
• Documentation (Markdown, README updates)
• Other (please specify above in "Proposed changes" section)

Checklist

General

• I have read the CONTRIBUTOR GUIDANCE wiki
• All PR dependencies have been merged and published (if applicable)
• A GIF or screenshot is included in the PR for visual changes
• The pre-publish command has been executed:
  • v2 and below: yarn workspace vscode-extension-for-zowe vscode:prepublish
  • v3: pnpm --filter vscode-extension-for-zowe vscode:prepublish

Code coverage

• There is coverage for the code that I have added
• I have added new test cases and they are passing
• I have manually tested the changes

Deployment

• I have added developer documentation (if applicable)
• Documentation should be added to Zowe Docs
  • If you're an outside contributor, please post in the #zowe-doc Slack channel to coordinate documentation.
  • Otherwise, please check with the rest of the squad about any needed documentation before merging.
• These changes may need ported to the appropriate branches (list here):

Further comments

[github-actions]

📅 Suggested merge-by date: 2/10/2025

[codecov]

Codecov Report

Attention: Patch coverage is 94.25287% with 5 lines in your changes missing coverage. Please review.

Project coverage is 93.23%. Comparing base (74f7815) to head (e042b97).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...ages/zowe-explorer/src/trees/shared/SharedUtils.ts	78.57%	3 Missing ⚠️
...ges/zowe-explorer/src/trees/dataset/DatasetTree.ts	97.18%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3413      +/-   ##
==========================================
+ Coverage   93.20%   93.23%   +0.03%     
==========================================
  Files         120      120              
  Lines       12583    12660      +77     
  Branches     2904     2875      -29     
==========================================
+ Hits        11728    11804      +76     
- Misses        854      855       +1     
  Partials        1        1              

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

[traeok]

Thanks for working on this Likhitha! This looks great so far, but I have a request to minimize code duplication.

[JillieBeanSim]

LGTM! thanks for working on this new feature @likhithanimma1 this is really cool!

I believe the failing windows checks have a fix in the port 3.1.0 PR

[zFernand0]

LGTM! 😋

Not that this is a common scenario, but when dragging from LPAR1 to LAPR2, there could be some data-loss if they share the same DASD 😢

Can't really test via the CLI since there is no "Move" equivalent 😢

Changes addressed, will review

[traeok]

Thanks for addressing my feedback Likhitha - sorry I didn't catch this earlier, but I found a misspelled query parameter that needs adjusted. Aside from that, the code LGTM - the drag/drop works well and its very convenient!

[zFernand0]

LGTM! 😋

Changes addressed, will review

[JillieBeanSim]

LGTM! thanks @likhithanimma1 for this feature!