Merge pull request #2373 from zowe/v2-sys-test-fix-port

V2 sys test fix port

__tests__/__resources__/properties/default_properties.yaml

@@ -104,6 +104,8 @@ zosjobs:
   modifiedJobclass: B
   # System affinity
   sysaff: zosjobs-sysaff
+  # CIM Support
+  skipCIM: false
 #-----------------------------------------------------------------------------#
 #  Set of properties for testing provisioning                                 #
 #-----------------------------------------------------------------------------#

__tests__/__src__/properties/ITestZosJobsSchema.ts

@@ -16,4 +16,5 @@ export interface ITestZosJobsSchema {
     jobclass: string;
     modifiedJobclass: string;
     sysaff: string;
+    skipCIM?: boolean;
 }

jest.config.js

@@ -133,14 +133,11 @@ module.exports = {
     ...projectConfig,
     "reporters": [
         "default",
+        "jest-stare",
         ["jest-junit", {
             "outputDirectory": "__tests__/__results__",
             "reportTestSuiteErrors": true
         }],
-        ["jest-stare", {
-            "coverageLink": "../unit/coverage/lcov-report/index.html",
-            "resultDir": "__tests__/__results__/jest-stare"
-        }],
         ["github-actions", { "silent": false } ]
     ],
     "testResultsProcessor": "jest-sonar-reporter",

package.json

@@ -41,7 +41,6 @@
     "prepare": "husky install && npm run bundle:webHelp",
     "package": "lerna run prepublishOnly && node scripts/bundleCliTgz.js"
   },
-  "dependencies": {},
   "devDependencies": {
     "@lerna-lite/changed": "^3.3.0",
     "@lerna-lite/cli": "^3.3.0",
@@ -69,7 +68,7 @@
     "jest-html-reporter": "^3.6.0",
     "jest-junit": "^12.0.0",
     "jest-sonar-reporter": "^2.0.0",
-    "jest-stare": "^2.2.0",
+    "jest-stare": "^2.5.2",
     "js-yaml": "^4.1.0",
     "jsonfile": "^4.0.0",
     "madge": "^4.0.1",
@@ -85,5 +84,13 @@
   },
   "jestSonar": {
     "reportPath": "__tests__/__results__/jest-sonar"
+  },
+  "jest-stare": {
+    "additionalResultsProcessors": [
+      "jest-junit",
+      "jest-sonar-reporter"
+    ],
+    "coverageLink": "../unit/coverage/lcov-report/index.html",
+    "resultDir": "__tests__/__results__/jest-stare"
   }
 }

packages/cli/__tests__/auth/__system__/__scripts__/auth_login_apiml_show_token.sh

@@ -1,3 +1,4 @@
+#!/bin/bash
 set -e
 
 HOST=$1

packages/cli/__tests__/auth/__system__/__scripts__/auth_login_apiml_show_token_rfj.sh

@@ -1,3 +1,4 @@
+#!/bin/bash
 set -e
 
 HOST=$1

packages/cli/__tests__/auth/__system__/__scripts__/auth_logout_apiml_show_token.sh

@@ -1,3 +1,4 @@
+#!/bin/bash
 set -e
 
 HOST=$1

packages/cli/__tests__/config/auto-init/__system__/__scripts__/config_auto_init.sh

@@ -1,3 +1,4 @@
+#!/bin/bash
 set -e
 
 HOST=$1

packages/cli/__tests__/config/auto-init/__system__/__scripts__/config_auto_init_cert.sh

@@ -1,3 +1,4 @@
+#!/bin/bash
 set -e
 
 HOST=$1

packages/cli/__tests__/config/auto-init/__system__/__scripts__/config_auto_init_profile.sh

@@ -1,3 +1,4 @@
+#!/bin/bash
 set -e
 
 zowe config auto-init

packages/cli/__tests__/config/auto-init/__system__/cli.config.auto-init.system.test.ts

@@ -45,6 +45,7 @@ describe("config auto-init without profile", () => {
                 base.rejectUnauthorized
             ]
         );
+        expect(response.status).toEqual(0);
 
         const config = fs.readFileSync(path.join(TEST_ENVIRONMENT.workingDir, "zowe.config.json")).toString();
         const profiles = JSONC.parse(config).profiles;
@@ -105,6 +106,7 @@ describe("config auto-init without profile", () => {
                 base.rejectUnauthorized
             ]
         );
+        expect(response.status).toEqual(0);
 
         config = fs.readFileSync(path.join(TEST_ENVIRONMENT.workingDir, "zowe.config.json")).toString();
         const profiles = JSONC.parse(config).profiles;
@@ -183,6 +185,7 @@ describe("config auto-init without profile and with certificates", () => {
                     base.rejectUnauthorized
                 ]
             );
+            expect(response.status).toEqual(0);
 
             const config = fs.readFileSync(path.join(TEST_ENVIRONMENT.workingDir, "zowe.config.json")).toString();
             const profiles = JSONC.parse(config).profiles;
@@ -249,6 +252,7 @@ describe("config auto-init without profile and with certificates", () => {
                     base.rejectUnauthorized
                 ]
             );
+            expect(response.status).toEqual(0);
 
             config = fs.readFileSync(path.join(TEST_ENVIRONMENT.workingDir, "zowe.config.json")).toString();
             const profiles = JSONC.parse(config).profiles;
@@ -313,7 +317,7 @@ describe("config auto-init with profile", () => {
 
     it("should successfully issue the auto-init command", () => {
         const response = runCliScript(__dirname + "/__scripts__/config_auto_init_profile.sh", TEST_ENVIRONMENT);
-
+        expect(response.status).toEqual(0);
         const config = fs.readFileSync(path.join(TEST_ENVIRONMENT.workingDir, "zowe.config.json")).toString();
         const profiles = JSONC.parse(config).profiles;
         let zosmfExists = false;
@@ -372,7 +376,7 @@ describe("config auto-init with profile and certificates", () => {
 
     it("should successfully issue the auto-init command", () => {
         const response = runCliScript(__dirname + "/__scripts__/config_auto_init_profile.sh", TEST_ENVIRONMENT);
-
+        expect(response.status).toEqual(0);
         const config = fs.readFileSync(path.join(TEST_ENVIRONMENT.workingDir, "zowe.config.json")).toString();
         const profiles = JSONC.parse(config).profiles;
         let zosmfExists = false;

packages/cli/__tests__/zosjobs/__system__/cancel/__scripts__/job/cancel_job.sh

@@ -16,6 +16,8 @@ then
     exit $RC
 fi
 
+sleep 1
+
 # Cancel the job
 zowe jobs cancel job $JOBID
 RC=$?

packages/cli/__tests__/zosjobs/__system__/cancel/__scripts__/job/cancel_job_fully_qualified.sh

@@ -22,6 +22,8 @@ then
     exit $RC
 fi
 
+sleep 1
+
 # Cancel the job
 zowe jobs cancel job $JOBID --host $HOST --port $PORT --user $USER --password $PASS --ru=false
 RC=$?

packages/cli/__tests__/zosjobs/__system__/cancel/__scripts__/job/cancel_job_v1.sh

@@ -16,6 +16,8 @@ then
     exit $RC
 fi
 
+sleep 1
+
 # Cancel the job 1
 zowe jobs cancel job $JOBID --modify-version 1.0
 RC=$?

packages/cli/__tests__/zosjobs/__system__/cancel/__scripts__/job/cancel_job_v2.sh

@@ -16,6 +16,8 @@ then
     exit $RC
 fi
 
+sleep 1
+
 # Cancel the job 1
 zowe jobs cancel job $JOBID --modify-version 2.0
 RC=$?

packages/cli/__tests__/zosjobs/__system__/cancel/__scripts__/job/cancel_job_v2_bad.sh

@@ -16,6 +16,8 @@ then
     exit $RC
 fi
 
+sleep 1
+
 # Cancel the job 1
 zowe jobs cancel job $JOBID --modify-version 2.0
 RC=$?
@@ -26,6 +28,8 @@ then
     exit $RC
 fi
 
+sleep 1
+
 # Cancel the job 2
 zowe jobs cancel job $JOBID --modify-version 2.0
 RC=$?

packages/cli/__tests__/zosjobs/__system__/cancel/__scripts__/job/cancel_job_v2_fully_qualified.sh

@@ -22,6 +22,8 @@ then
     exit $RC
 fi
 
+sleep 1
+
 # Cancel the job
 zowe jobs cancel job $JOBID --host $HOST --port $PORT --user $USER --password $PASS --ru=false --modify-version 2.0
 RC=$?

packages/cli/__tests__/zosjobs/__system__/cancel/cli.zos-jobs.cancel.job.system.test.ts

@@ -14,10 +14,16 @@ import { TestEnvironment } from "../../../../../../__tests__/__src__/environment
 import { ITestPropertiesSchema } from "../../../../../../__tests__/__src__/properties/ITestPropertiesSchema";
 import { JobTestsUtils } from "../../../../../zosjobs/__tests__/__system__/JobTestsUtils";
 import { IO } from "@zowe/imperative";
+import { DeleteJobs, IJob } from "@zowe/zos-jobs-for-zowe-sdk";
+import { Session } from "@zowe/imperative";
 
 // Test Environment populated in the beforeAll();
+let REAL_SESSION: Session;
 let TEST_ENVIRONMENT: ITestEnvironment<ITestPropertiesSchema>;
 const LOCAL_JCL_FILE: string = __dirname + "/" + "testFileOfLocalJCL.txt";
+const jobsToDelete = new Map<string, string>();
+const jobDataRegexV1 = /Successfully submitted request to cancel job (\w+) \((JOB\d+)\)/;
+const jobNameRegexV1 = /job (\w+)/i;
 
 describe("zos-jobs cancel job command", () => {
     // Create the unique test environment
@@ -28,6 +34,8 @@ describe("zos-jobs cancel job command", () => {
         });
         const systemProps = TEST_ENVIRONMENT.systemTestProperties;
 
+        REAL_SESSION = TestEnvironment.createZosmfSession(TEST_ENVIRONMENT);
+
         const jcl = JobTestsUtils.getSleepJCL(systemProps.zosmf.user, systemProps.tso.account, systemProps.zosjobs.jobclass);
         const bufferJCL: Buffer = Buffer.from(jcl);
         IO.createFileSync(LOCAL_JCL_FILE);
@@ -36,6 +44,9 @@ describe("zos-jobs cancel job command", () => {
 
     afterAll(async () => {
         IO.deleteFile(LOCAL_JCL_FILE);
+        for (const [jobname, jobid] of jobsToDelete) {
+            await DeleteJobs.deleteJob(REAL_SESSION, jobname, jobid);
+        }
     });
 
     describe("error handling", () => {
@@ -67,10 +78,20 @@ describe("zos-jobs cancel job command", () => {
 
     describe("successful scenario", () => {
         it("should cancel a job v1", () => {
-            const response = runCliScript(__dirname + "/__scripts__/job/cancel_job_v1.sh", TEST_ENVIRONMENT, [LOCAL_JCL_FILE]);
-            expect(response.stderr.toString()).toBe("");
-            expect(response.status).toBe(0);
-            expect(response.stdout.toString()).toContain("Successfully submitted request to cancel job");
+            if (TEST_ENVIRONMENT.systemTestProperties.zosjobs.skipCIM) {
+                process.stdout.write("Skipping test because skipCIM is set.");
+            } else {
+                const response = runCliScript(__dirname + "/__scripts__/job/cancel_job_v1.sh", TEST_ENVIRONMENT, [LOCAL_JCL_FILE]);
+
+                expect(response.stderr.toString()).toBe("");
+                expect(response.status).toBe(0);
+                expect(response.stdout.toString()).toContain("Successfully submitted request to cancel job");
+
+                const jobname = response.stdout.toString().match(jobNameRegexV1).pop();
+                const jobid = response.stdout.toString().match(jobDataRegexV1).pop();
+
+                jobsToDelete.set(jobname, jobid);
+            }
         });
 
         it("should cancel a job v2", () => {
@@ -108,18 +129,27 @@ describe("zos-jobs cancel job command", () => {
             });
 
             it("cancel a job without a profile 1.0", async () => {
-                const response = runCliScript(__dirname + "/__scripts__/job/cancel_job_v1_fully_qualified.sh",
-                    TEST_ENVIRONMENT_NO_PROF,
-                    [
-                        LOCAL_JCL_FILE,
-                        DEFAULT_SYSTEM_PROPS.zosmf.host,
-                        DEFAULT_SYSTEM_PROPS.zosmf.port,
-                        DEFAULT_SYSTEM_PROPS.zosmf.user,
-                        DEFAULT_SYSTEM_PROPS.zosmf.password,
-                    ]);
-                expect(response.stderr.toString()).toBe("");
-                expect(response.status).toBe(0);
-                expect(response.stdout.toString()).toContain("Successfully submitted request to cancel job");
+                if (TEST_ENVIRONMENT.systemTestProperties.zosjobs.skipCIM) {
+                    process.stdout.write("Skipping test because skipCIM is set.");
+                } else {
+                    const response = runCliScript(__dirname + "/__scripts__/job/cancel_job_v1_fully_qualified.sh",
+                        TEST_ENVIRONMENT_NO_PROF,
+                        [
+                            LOCAL_JCL_FILE,
+                            DEFAULT_SYSTEM_PROPS.zosmf.host,
+                            DEFAULT_SYSTEM_PROPS.zosmf.port,
+                            DEFAULT_SYSTEM_PROPS.zosmf.user,
+                            DEFAULT_SYSTEM_PROPS.zosmf.password,
+                        ]);
+                    expect(response.stderr.toString()).toBe("");
+                    expect(response.status).toBe(0);
+                    expect(response.stdout.toString()).toContain("Successfully submitted request to cancel job");
+
+                    const jobname = response.stdout.toString().match(jobNameRegexV1).pop();
+                    const jobid = response.stdout.toString().match(jobDataRegexV1).pop();
+
+                    jobsToDelete.set(jobname, jobid);
+                }
             });
 
             it("cancel a job without a profile 2.0", async () => {

packages/cli/__tests__/zosjobs/__system__/delete/__scripts__/job/delete_job.sh

@@ -17,7 +17,7 @@ then
 fi
 
 # Loop until the job goes to the output queue
-until [ $ATTEMPTS -gt 0 ]
+until [ $ATTEMPTS -lt 1 ]
 do
     STATUS=`zowe jobs view job-status-by-jobid $JOBID --rff status --rft string`
     RC=$?

packages/cli/__tests__/zosjobs/__system__/delete/__scripts__/job/delete_job_fully_qualified.sh

@@ -23,7 +23,7 @@ then
 fi
 
 # Loop until the job goes to the output queue
-until [ $ATTEMPTS -gt 0 ]
+until [ $ATTEMPTS -lt 1 ]
 do
     STATUS=`zowe jobs view job-status-by-jobid $JOBID --host $HOST --port $PORT --user $USER --password $PASS --ru=false --rff status --rft string`
     RC=$?

packages/cli/__tests__/zosjobs/__system__/delete/__scripts__/job/delete_job_v1.sh

@@ -17,7 +17,7 @@ then
 fi
 
 # Loop until the job goes to the output queue
-until [ $ATTEMPTS -gt 0 ]
+until [ $ATTEMPTS -lt 1 ]
 do
     STATUS=`zowe jobs view job-status-by-jobid $JOBID --rff status --rft string`
     RC=$?

packages/cli/__tests__/zosjobs/__system__/delete/__scripts__/job/delete_job_v1_fully_qualified.sh

@@ -23,7 +23,7 @@ then
 fi
 
 # Loop until the job goes to the output queue
-until [ $ATTEMPTS -gt 0 ]
+until [ $ATTEMPTS -lt 1 ]
 do
     STATUS=`zowe jobs view job-status-by-jobid $JOBID --host $HOST --port $PORT --user $USER --password $PASS --ru=false --rff status --rft string`
     RC=$?

packages/cli/__tests__/zosjobs/__system__/delete/__scripts__/job/delete_job_v2.sh

@@ -17,7 +17,7 @@ then
 fi
 
 # Loop until the job goes to the output queue
-until [ $ATTEMPTS -gt 0 ]
+until [ $ATTEMPTS -lt 1 ]
 do
     STATUS=`zowe jobs view job-status-by-jobid $JOBID --rff status --rft string`
     RC=$?

packages/cli/__tests__/zosjobs/__system__/delete/__scripts__/job/delete_job_v2_fully_qualified.sh

@@ -23,7 +23,7 @@ then
 fi
 
 # Loop until the job goes to the output queue
-until [ $ATTEMPTS -gt 0 ]
+until [ $ATTEMPTS -lt 1 ]
 do
     STATUS=`zowe jobs view job-status-by-jobid $JOBID --host $HOST --port $PORT --user $USER --password $PASS --ru=false --rff status --rft string`
     RC=$?

packages/cli/__tests__/zosjobs/__system__/delete/__scripts__/old-jobs/delete_old_jobs.sh

@@ -18,7 +18,7 @@ then
 fi
 
 # Loop until the job goes to the output queue
-until [ $ATTEMPTS -gt 0 ]
+until [ $ATTEMPTS -lt 1 ]
 do
     STATUS=`zowe jobs view job-status-by-jobid $JOBID --rff status --rft string`
     RC=$?