Remove the 'zowe profiles' command command handlers & APIs

gejohnston commented 4 months ago

What It Does

This PR removes the command handlers for the zowe profiles command group.

Also:

Numerous APIs, interfaces, and constants that supported V1 profiles were also removed.
Three classes (AbstractProfileManager which was extended by BasicProfileManager, which was extended by CliProfileManager) were consolidated into the one CliProfileManager class.
The --show-inputs-only options and the config report-env commands no longer report V1 profile information.
Replaced the term "Team configuration" with "Zowe client configuration" in the zowe config report-env command.
Changed a reference in the config convert-profiles command from "V2 profiles" to "TeamConfig".
Removed the display of the NVM version number in the config report-env command. Newer versions of NVM display a GUI popup message when launched as a subprocess, which complains that NVM should be run from a terminal (version 1.1.12 displays this behavior). The popup blocks further output until the user responds to the popup. NVM is NOT used by customers frequently enough to warrant an exhaustive effort to keep that piece of information in the report. For awareness, the NVM dialog is shown below.

How to Test

Commands should continue to run as before when no configuration exists or a team configuration exists. Commands should ignore V1 profiles, even when they are the only available configuration. In particular, if only V1 profiles exist, running a Zowe command now will prompt the user for connection information.
```
> zowe files list data-set "SYS1.PARMLIB*"
Enter the host name of your service: YourHostName
Enter the user name for your service (will be hidden):
Enter the password for your service (will be hidden):
```
Use --show-inputs-only, config report-env, & config convert-profiles and observe that no references to V1 or V2 profiles are displayed.

Follow-ups

The "ProfileIO" class must be renamed to "V1ProfileConversion", and all write functions must be removed.
Any additional V1 logic that is no longer called should be removed.

Review Checklist I certify that I have:

[x] tested my changes
[x] added/updated automated tests
[x] updated the changelog
[x] followed the contribution guidelines

Additional Comments

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 92.70833% with 14 lines in your changes are missing coverage. Please review.

Project coverage is 90.93%. Comparing base (230203e) to head (a38c22d).

Files	Patch %	Lines
...perative/src/cmd/src/profiles/CliProfileManager.ts	81.57%	7 Missing :warning:
...es/imperative/src/config/src/ProfileCredentials.ts	20.00%	4 Missing :warning:
packages/imperative/src/config/src/ConfigUtils.ts	90.90%	1 Missing :warning:
packages/imperative/src/config/src/ProfileInfo.ts	98.61%	1 Missing :warning:
...c/imperative/src/config/cmd/report-env/EnvQuery.ts	83.33%	1 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## next #2073 +/- ## ========================================== - Coverage 91.09% 90.93% -0.16% ========================================== Files 638 616 -22 Lines 18840 17358 -1482 Branches 3814 3613 -201 ========================================== - Hits 17162 15785 -1377 + Misses 1677 1572 -105 Partials 1 1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.