Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated for core cli #1981

Merged
merged 68 commits into from
Jan 30, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
68 commits
Select commit Hold shift + click to select a range
e3e0d7f
Constants.ts
Nov 13, 2023
31afed9
removing ZosFilesCreateExtraOptions.showAttributes
Nov 14, 2023
eb5d4f7
fix in secure.handler.unit.test.ts
Nov 14, 2023
961c33f
remove ZosTso.constants.ts
Nov 14, 2023
58878f3
Merge remote-tracking branch 'origin/master' into removeDeprecatedCore
Nov 16, 2023
e3e9b40
Removed
Nov 16, 2023
e2c7b41
ZosFiles.messages.ts
Nov 16, 2023
2b060a6
excludePatterns
Nov 16, 2023
0513d22
ZosmfSession.ts
Nov 17, 2023
b2ee293
SshSession.ts
Nov 17, 2023
d9d6e8a
zosmfProfile
Nov 17, 2023
a8fc9cf
startCmdFlag
Nov 17, 2023
e55b6c3
more removal for excludePatterns
Nov 17, 2023
c2b5d4f
getSpoolDownloadFile
Nov 17, 2023
1bdc6b0
changelog
Nov 17, 2023
23370c5
Merge branch 'next' of https://github.com/zowe/zowe-cli into removeDe…
zFernand0 Nov 17, 2023
0dca19e
Merge branch 'removeDeprecatedCore' of https://github.com/zowe/zowe-c…
zFernand0 Nov 17, 2023
f6127d2
issues with downloadjobs system test
Nov 20, 2023
b51f5c0
failing system test
Nov 28, 2023
0477754
sorry that took me so long
Nov 29, 2023
958449f
merging from next into branch
Nov 29, 2023
f6a5fbb
Merge branch 'next' into removeDeprecatedCore
ATorrise Nov 30, 2023
6fb233e
removing unused import from ZosFilesBase.handler.ts
ATorrise Nov 30, 2023
18b625d
updating package changelogs
Nov 30, 2023
b763bbe
Merge branch 'removeDeprecatedCore' of https://github.com/zowe/zowe-c…
Nov 30, 2023
ca8461b
adding back in showAttributes
Dec 13, 2023
190fa84
removing dup line and requested imperative dependencies
Dec 13, 2023
16ac001
resolving over-exclusion of excludePatterns
Jan 2, 2024
536a545
reverting additions
Jan 3, 2024
d355872
Merge remote-tracking branch 'origin/next' into removeDeprecatedCore
Jan 3, 2024
aed1a1d
removing deprecated test for excludePatterns
Jan 4, 2024
7fc13d7
Update CHANGELOG.md
ATorrise Jan 4, 2024
6e1ab9e
fix CHANGELOG.md?
ATorrise Jan 4, 2024
6adc65c
Update CHANGELOG.md
ATorrise Jan 4, 2024
9833d15
Update CHANGELOG.md
ATorrise Jan 4, 2024
85c9ea1
Update CHANGELOG.md
ATorrise Jan 5, 2024
45d94c5
Update CHANGELOG.md
ATorrise Jan 5, 2024
a9ce838
Merge branch 'next' into removeDeprecatedCore
ATorrise Jan 5, 2024
53473fd
Update CHANGELOG.md
ATorrise Jan 5, 2024
a3a8852
adding semicolon
ATorrise Jan 5, 2024
79bf5c0
reverting changes to DataSetMatching.handler.unit.test.ts
ATorrise Jan 8, 2024
0b752ff
adding test back to List.unit.test.ts
ATorrise Jan 8, 2024
35902b2
reverting packages/imperative changelog changes
ATorrise Jan 8, 2024
c84b6d9
revert CHANGELOG.md ? imperative
ATorrise Jan 8, 2024
bc83fdb
ZosFilesMessages.noDataSetsInList.message to noDataSetsMatchingPatter…
ATorrise Jan 8, 2024
eac535c
adding back in noDataSetsInList since it might have been mistakenly d…
ATorrise Jan 8, 2024
ca4016d
fixing test by deleting exclude pattern from fakeDownloadOptions
ATorrise Jan 8, 2024
15b7244
adding back show attributes to options
Jan 22, 2024
45cd1d7
Merge branch 'next' into removeDeprecatedCore
ATorrise Jan 22, 2024
800d030
removing flasy showAttributes
Jan 22, 2024
ff36667
tims requests
Jan 23, 2024
9373997
updating snaps
Jan 23, 2024
e60919b
updating snaps
Jan 24, 2024
5c1595e
Merge branch 'next' into removeDeprecatedCore
ATorrise Jan 24, 2024
ec28fe7
update edit uss snaps
ATorrise Jan 24, 2024
ac477a3
updated integration
ATorrise Jan 25, 2024
5b17290
might need to revert this change jest.mock('readline')
ATorrise Jan 25, 2024
d6c3eba
fix?
ATorrise Jan 25, 2024
90c4ee7
addressing andrews comments
ATorrise Jan 25, 2024
f624747
removing jest.mock from cliutils.unit.test
ATorrise Jan 25, 2024
32c0111
testing the revert of a suggested change
ATorrise Jan 25, 2024
874031f
reverting uneccesary changes to DownloadJobs.system.test
ATorrise Jan 26, 2024
f9d2ccf
reverting reverted changes. i think i was making required changes and…
ATorrise Jan 26, 2024
a9f942a
Update Create.unit.test.ts
ATorrise Jan 29, 2024
8fcdfe4
Update Create.unit.test.ts
ATorrise Jan 29, 2024
6ac0bcd
Merge branch 'next' into removeDeprecatedCore
ATorrise Jan 29, 2024
0226009
Update Create.unit.test.ts
ATorrise Jan 30, 2024
5617fe2
Update DownloadJobs.system.test.ts
ATorrise Jan 30, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 0 additions & 9 deletions __tests__/__packages__/cli-test-utils/src/TestUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -62,15 +62,6 @@ export function runCliScript(scriptPath: string, testEnvironment: ITestEnvironme
}
}

/**
* Check if stderr output is empty for profiles command. Ignores any message
* about profiles being deprecated.
* @deprecated Use `stripProfileDeprecationMessages`
*/
export function isStderrEmptyForProfilesCommand(output: Buffer): boolean {
return stripProfileDeprecationMessages(output).length === 0;
}

/**
* Strip v1 profile deprecation messages from stderr output.
*/
Expand Down
14 changes: 0 additions & 14 deletions npm-shrinkwrap.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

19 changes: 19 additions & 0 deletions packages/cli/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,25 @@

All notable changes to the Zowe CLI package will be documented in this file.

## Recent Changes

LTS Breaking: Removed the following previously deprecated items: [#1981](https://github.com/zowe/zowe-cli/pull/1981)
- Moved the many constants from `zowe-cli/packages/cli/src/Constants.ts` to `zowe-cli/packages/core/src/constants/Core.constants.ts`
- Removing `ZosFilesCreateExtraOptions.showAttributes` without replacement
- Moved all constants from `zowe-cli/packages/cli/src/zostso/constants/ZosTso.constants.ts` to `@zowe/zos-tso-for-zowe-sdk`
- Removed `isStderrEmptyForProfilesCommand` use `stripProfileDeprecationMessages` from `zowe-cli/__tests__/__packages__/cli-test-utils/src/TestUtils.ts` instead
- Removed `allDataSetsArchived`, `datasetsDownloadedSuccessfully`, `noDataSetsMatchingPatternRemain` and `onlyEmptyPartitionedDataSets` from ZosFiles.messages.ts
- Removed `getSpoolDownloadFile` use `getSpoolDownloadFilePath` instead
- Removed constants from ZosmfSession
- ZOSMF_OPTION_HOST_PROFILE use ZOSMF_OPTION_HOST instead
- ZOSMF_OPTION_USER_PROFILE use ZOSMF_OPTION_USER instead
- ZOSMF_OPTION_PASSWORD_PROFILE use ZOSMF_OPTION_PASSWORD instead
- Removed constants from SshSession.ts
- SSH_OPTION_USER_PROFILE use SSH_OPTION_USER
- SSH_OPTION_HOST_PROFILE use SSH_OPTION_HOST
- Removed zosmfProfile from `ZosFilesBase.handler.ts`
- Removed statCmdFlag as an export from Shell.ts

## `8.0.0-next.202401262128`

- Enhancement: Adding `--binary` and `--encoding` options to `zosfiles edit`
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
// Jest Snapshot v1, https://goo.gl/fbAQLP

exports[`View USS file should display the help 1`] = `
exports[`Edit USS file should display the help 1`] = `
"================Z/OS FILES EDIT USS FILE HELP===============

COMMAND NAME
Expand Down Expand Up @@ -161,7 +161,7 @@ exports[`View USS file should display the help 1`] = `
"
`;

exports[`View USS file should display the help in json format 1`] = `
exports[`Edit USS file should display the help in json format 1`] = `
"================Z/OS FILES EDIT USS FILE HELP===============
{
\\"success\\": true,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ import { ITestPropertiesSchema } from "../../../../../../../__tests__/__src__/pr

let TEST_ENVIRONMENT: ITestEnvironment<ITestPropertiesSchema>;

describe("View USS file", () => {
describe("Edit USS file", () => {

beforeAll(async () => {
TEST_ENVIRONMENT = await TestEnvironment.setUp({
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
*
*/

import { AbstractSession, IHandlerParameters, IProfile, Session } from "@zowe/imperative";
import { AbstractSession, IHandlerParameters, Session } from "@zowe/imperative";
import { IZosFilesResponse } from "@zowe/zos-files-for-zowe-sdk";
import { ZosFilesBaseHandler } from "../../../src/zosfiles/ZosFilesBase.handler";

Expand All @@ -21,8 +21,7 @@ describe("ZosFilesBaseHandler", () => {

public async processWithSession(
commandParameters: IHandlerParameters,
session: AbstractSession,
zosmfProfile: IProfile
session: AbstractSession
): Promise<IZosFilesResponse> {
return this.returnResponse;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -101,15 +101,6 @@ Array [
"name": "secondary-space",
"type": "number",
},
Object {
"aliases": Array [
"pa",
],
"description": "Show the full allocation attributes",
"hidden": true,
"name": "show-attributes",
"type": "boolean",
},
Object {
"aliases": Array [
"sz",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -101,15 +101,6 @@ Array [
"name": "secondary-space",
"type": "number",
},
Object {
"aliases": Array [
"pa",
],
"description": "Show the full allocation attributes",
"hidden": true,
"name": "show-attributes",
"type": "boolean",
},
Object {
"aliases": Array [
"sz",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -101,15 +101,6 @@ Array [
"name": "secondary-space",
"type": "number",
},
Object {
"aliases": Array [
"pa",
],
"description": "Show the full allocation attributes",
"hidden": true,
"name": "show-attributes",
"type": "boolean",
},
Object {
"aliases": Array [
"sz",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -104,15 +104,6 @@ Array [
"name": "secondary-space",
"type": "number",
},
Object {
"aliases": Array [
"pa",
],
"description": "Show the full allocation attributes",
"hidden": true,
"name": "show-attributes",
"type": "boolean",
},
Object {
"aliases": Array [
"sz",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -110,15 +110,6 @@ Array [
"name": "secondary-space",
"type": "number",
},
Object {
"aliases": Array [
"pa",
],
"description": "Show the full allocation attributes",
"hidden": true,
"name": "show-attributes",
"type": "boolean",
},
Object {
"aliases": Array [
"sz",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -91,15 +91,6 @@ Array [
"name": "secondary-space",
"type": "number",
},
Object {
"aliases": Array [
"pa",
],
"description": "Show the full allocation attributes",
"hidden": true,
"name": "show-attributes",
"type": "boolean",
},
Object {
"aliases": Array [
"sz",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -80,15 +80,6 @@ Array [
"name": "secondary-space",
"type": "number",
},
Object {
"aliases": Array [
"pa",
],
"description": "Show the full allocation attributes",
"hidden": true,
"name": "show-attributes",
"type": "boolean",
},
Object {
"aliases": Array [
"sz",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ const fakeDownloadOptions: IDownloadOptions = {
binary: undefined,
directory: undefined,
encoding: undefined,
excludePatterns: undefined,
extension: undefined,
extensionMap: undefined,
failFast: undefined,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ import { TestEnvironment } from "../../../../../../__tests__/__src__/environment
import { ITestPropertiesSchema } from "../../../../../../__tests__/__src__/properties/ITestPropertiesSchema";
import { ZosFilesConstants } from "@zowe/zos-files-for-zowe-sdk";
import { ZosmfRestClient } from "@zowe/core-for-zowe-sdk";
import { startCmdFlag } from "@zowe/zos-uss-for-zowe-sdk";
import { Shell } from "@zowe/zos-uss-for-zowe-sdk";


// Test environment will be populated in the "beforeAll"
Expand All @@ -35,7 +35,7 @@ let keyPassphrase: string;
function checkResponse(response: any, expectStatus: number) {
expect(response.stderr.toString()).toBe("");
expect(response.status).toBe(expectStatus);
expect(response.stdout.toString()).not.toMatch(startCmdFlag);
expect(response.stdout.toString()).not.toMatch(Shell.startCmdFlag);
}

function generateRandomString(j: number) {
Expand Down
Loading
Loading