fix(profInfo): Fix secure property names being returned for wrong profile

[t1m0thyj]

What It Does
Fixes a small bug in ConfigSecure.securePropsForProfile where listing secure property names for "abcdef" profile also included property names for "abc" profile

Also moves top-level methods in ConfigUtils.ts inside a class to conform to best practices (thanks @zFernand0 for the suggestion).

How to Test

• Create a project config like the one in Log out from API ML is not available anymore with Manage Profile menu zowe-explorer-vscode#2633 (comment)
• Run a test script like the following and verify that "zosmf-apiml" profile has no secure properties

const { Config } = require("@zowe/imperative");
(async () => {
    const config = await Config.load("zowe", { projectDir: <projectDir> });
    for (const profName of ["zosmf", "zosmf-apiml", "base"]) {
        console.log(profName, config.api.secure.securePropsForProfile(profName));
    }
})();

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: 2 lines in your changes are missing coverage. Please review.

Comparison is base (461c540) 91.03% compared to head (66929ba) 91.02%.
Report is 1 commits behind head on master.

Files	Patch %	Lines
...imperative/src/config/cmd/secure/secure.handler.ts	66.66%	1 Missing ⚠️
...e/src/imperative/src/config/cmd/set/set.handler.ts	66.66%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2016      +/-   ##
==========================================
- Coverage   91.03%   91.02%   -0.01%     
==========================================
  Files         636      636              
  Lines       18731    18737       +6     
  Branches     3907     3908       +1     
==========================================
+ Hits        17051    17055       +4     
- Misses       1679     1681       +2     
  Partials        1        1              

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

[zFernand0]

I learned that using .startsWith may not always be a good practice 😋
LGTM! 😋

[zFernand0]

I think this is the way in which we will have to eventually deal with the Imperative package.
Little by little, and as non-breaking changes. 😋
Thanks for moving the functions into a class 😋

[zFernand0]

Thanks for addressing these edge cases for the ProfileInfo APIs

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
94.1% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[traeok]

Tested and LGTM! Thanks @t1m0thyj