@sasjs/cli 4.4.2 → 4.5.0

package/build/commands/job/jobCommand.js

@@ -82,7 +82,7 @@ var executeParseOptions = {
         type: 'boolean',
         alias: 'i',
         default: false,
-        description: 'If present and return status only is provided, CLI will return status 0 when the job state is warning.'
+        description: 'If present, CLI will return status 0 when the job state is warning.'
     },
     log: {
         type: 'string',
@@ -94,6 +94,7 @@ var executeParseOptions = {
         alias: 'o',
         description: 'path where output of the finished job execution will be saved.'
     },
+    // returnStatusOnly flag is deprecated and is left to display warning if used
     returnStatusOnly: {
         type: 'boolean',
         default: false,
@@ -118,6 +119,12 @@ var executeParseOptions = {
     streamLog: {
         type: 'boolean',
         description: 'Flag indicating whether the logs should be streamed to a local file as the job executes.'
+    },
+    verbose: {
+        type: 'boolean',
+        alias: 'v',
+        default: false,
+        description: 'If present, CLI will return status 0, 1 or 2 together with HTTP response summaries.'
     }
 };
 var JobCommand = /** @class */ (function (_super) {
@@ -139,15 +146,25 @@ var JobCommand = /** @class */ (function (_super) {
         _this.jobSubCommands = jobSubCommands;
         return _this;
     }
+    /**
+     * Method responsible for command execution.
+     * @returns - promise that resolves into return code.
+     */
     JobCommand.prototype.execute = function () {
         var _a;
         return __awaiter(this, void 0, void 0, function () {
-            var target, _b, _c, _d, _e;
+            var target, returnStatusOnly, _b, _c, _d, _e;
             return __generator(this, function (_f) {
                 switch (_f.label) {
-                    case 0: return [4 /*yield*/, this.getTargetInfo()];
+                    case 0: return [4 /*yield*/, this.getTargetInfo()
+                        // returnStatusOnly flag is deprecated and is left to display warning if used
+                    ];
                     case 1:
                         target = (_f.sent()).target;
+                        returnStatusOnly = !!this.parsed.returnStatusOnly;
+                        if (returnStatusOnly) {
+                            process.logger.warn('--returnStatusOnly (-r) flag is deprecated.');
+                        }
                         _b = target.serverType;
                         switch (_b) {
                             case utils_1.ServerType.SasViya: return [3 /*break*/, 2];
@@ -194,6 +211,11 @@ var JobCommand = /** @class */ (function (_super) {
             });
         });
     };
+    /**
+     * Executes job on SASJS server.
+     * @param target - SASJS server configuration.
+     * @returns - promise that resolves into return code.
+     */
     JobCommand.prototype.executeJobSasjs = function (target) {
         var _a;
         return __awaiter(this, void 0, void 0, function () {
@@ -210,6 +232,7 @@ var JobCommand = /** @class */ (function (_super) {
                                 .then(function () { return command_1.ReturnCode.Success; })
                                 .catch(function (err) {
                                 var _a;
+                                // handle job execution failure
                                 (_a = process.logger) === null || _a === void 0 ? void 0 : _a.error('Error executing job: ', err);
                                 return command_1.ReturnCode.InternalError;
                             })];
@@ -220,36 +243,41 @@ var JobCommand = /** @class */ (function (_super) {
             });
         });
     };
+    /**
+     * Executes job on SAS9 server.
+     * @param target - SAS9 server configuration.
+     * @returns - promise that resolves into return code.
+     */
     JobCommand.prototype.executeJobSas9 = function (target) {
         var _a;
         return __awaiter(this, void 0, void 0, function () {
-            var jobPath, log, output, source, _b, sasjs, authConfigSas9, userName, password, returnCode;
+            var jobPath, log, source, output, _b, sasjs, authConfigSas9, userName, password, returnCode;
             return __generator(this, function (_c) {
                 switch (_c.label) {
                     case 0:
                         jobPath = (0, prefixAppLoc_1.prefixAppLoc)(target.appLoc, this.parsed.jobPath);
                         log = (0, getLogFilePath_1.getLogFilePath)(this.parsed.log, jobPath);
+                        source = this.parsed.source;
                         output = ((_a = this.parsed.output) === null || _a === void 0 ? void 0 : _a.length)
                             ? this.parsed.output
                             : undefined;
-                        source = this.parsed.source;
                         return [4 /*yield*/, (0, utils_2.getSASjsAndAuthConfig)(target).catch(function (err) {
                                 var _a;
+                                // handle getting instance of @sasjs/adapter and auth config failure
                                 (_a = process.logger) === null || _a === void 0 ? void 0 : _a.error('Unable to execute job. Error fetching auth config: ', err);
-                                return { sasjs: (0, utils_2.getSASjs)(target), authConfigSas9: undefined };
+                                return { sasjs: undefined, authConfigSas9: undefined };
                             })];
                     case 1:
                         _b = _c.sent(), sasjs = _b.sasjs, authConfigSas9 = _b.authConfigSas9;
-                        if (!authConfigSas9)
+                        if (!authConfigSas9 || !sasjs)
                             return [2 /*return*/, command_1.ReturnCode.InternalError];
                         userName = authConfigSas9.userName;
                         password = (0, utils_1.decodeFromBase64)(authConfigSas9.password);
                         return [4 /*yield*/, (0, execute_1.executeJobSas9)(sasjs, { userName: userName, password: password }, jobPath, log, output, source)
-                                .then(function () {
-                                return command_1.ReturnCode.Success;
-                            })
+                                .then(function () { return command_1.ReturnCode.Success; })
                                 .catch(function (err) {
                                 var _a;
+                                // handle job execution failure
                                 (_a = process.logger) === null || _a === void 0 ? void 0 : _a.error('Error executing job: ', err);
                                 return command_1.ReturnCode.InternalError;
                             })];
@@ -260,51 +288,53 @@ var JobCommand = /** @class */ (function (_super) {
             });
         });
     };
+    /**
+     * Executes job on Viya server.
+     * @param target - Viya server configuration.
+     * @returns - promise that resolves into return code.
+     */
     JobCommand.prototype.executeJobViya = function (target) {
-        var _a, _b, _c;
+        var _a, _b;
         return __awaiter(this, void 0, void 0, function () {
-            var jobPath, log, wait, output, statusFile, returnStatusOnly, ignoreWarnings, source, streamLog, _d, sasjs, authConfig, returnCode;
-            return __generator(this, function (_e) {
-                switch (_e.label) {
+            var jobPath, statusFile, log, ignoreWarnings, streamLog, verbose, source, wait, output, _c, sasjs, authConfig, returnCode;
+            return __generator(this, function (_d) {
+                switch (_d.label) {
                     case 0:
                         jobPath = (0, prefixAppLoc_1.prefixAppLoc)(target.appLoc, this.parsed.jobPath);
+                        statusFile = getStatusFilePath(this.parsed.statusFile);
                         log = (0, getLogFilePath_1.getLogFilePath)(this.parsed.log, jobPath);
+                        ignoreWarnings = !!this.parsed.ignoreWarnings;
+                        streamLog = !!this.parsed.streamLog;
+                        verbose = !!this.parsed.verbose;
+                        source = this.parsed.source;
                         wait = this.parsed.wait || !!log;
                         output = ((_a = this.parsed.output) === null || _a === void 0 ? void 0 : _a.length)
                             ? this.parsed.output
                             : ((_b = this.parsed.output) === null || _b === void 0 ? void 0 : _b.length) === 0
                                 ? true
                                 : false;
-                        statusFile = getStatusFilePath(this.parsed.statusFile);
-                        returnStatusOnly = !!this.parsed.returnStatusOnly;
-                        ignoreWarnings = !!this.parsed.ignoreWarnings;
-                        source = this.parsed.source;
-                        streamLog = !!this.parsed.streamLog;
-                        if (returnStatusOnly && !wait)
+                        if (verbose && !wait)
                             wait = true;
-                        if (ignoreWarnings && !returnStatusOnly) {
-                            (_c = process.logger) === null || _c === void 0 ? void 0 : _c.warn("Using the 'ignoreWarnings' flag without 'returnStatusOnly' flag will not affect the sasjs job execute command.");
-                        }
                         return [4 /*yield*/, (0, utils_2.getSASjsAndAuthConfig)(target).catch(function (err) {
                                 var _a;
+                                // handle getting instance of @sasjs/adapter and auth config failure
                                 (_a = process.logger) === null || _a === void 0 ? void 0 : _a.error('Unable to execute job. Error fetching auth config: ', err);
-                                return { sasjs: (0, utils_2.getSASjs)(target), authConfig: undefined };
+                                return { sasjs: undefined, authConfig: undefined };
                             })];
                     case 1:
-                        _d = _e.sent(), sasjs = _d.sasjs, authConfig = _d.authConfig;
-                        if (!authConfig)
+                        _c = _d.sent(), sasjs = _c.sasjs, authConfig = _c.authConfig;
+                        if (!authConfig || !sasjs)
                             return [2 /*return*/, command_1.ReturnCode.InternalError];
-                        return [4 /*yield*/, (0, execute_1.executeJobViya)(sasjs, authConfig, jobPath, target, wait, output, log, statusFile, returnStatusOnly, ignoreWarnings, source, streamLog)
-                                .then(function () {
-                                return command_1.ReturnCode.Success;
-                            })
+                        return [4 /*yield*/, (0, execute_1.executeJobViya)(sasjs, authConfig, jobPath, target, wait, output, log, statusFile, ignoreWarnings, source, streamLog, verbose)
+                                .then(function () { return command_1.ReturnCode.Success; })
                                 .catch(function (err) {
                                 var _a;
+                                // handle job execution failure
                                 (_a = process.logger) === null || _a === void 0 ? void 0 : _a.error('Error executing job: ', err);
                                 return command_1.ReturnCode.InternalError;
                             })];
                     case 2:
-                        returnCode = _e.sent();
+                        returnCode = _d.sent();
                         return [2 /*return*/, returnCode];
                 }
             });
@@ -313,6 +343,11 @@ var JobCommand = /** @class */ (function (_super) {
     return JobCommand;
 }(targetCommand_1.TargetCommand));
 exports.JobCommand = JobCommand;
+/**
+ * Gets status file path.
+ * @param statusFileArg - file path provided as command attribute.
+ * @returns - absolute status file path or undefined if command attribute wasn't provided.
+ */
 var getStatusFilePath = function (statusFileArg) {
     if (statusFileArg) {
         var currentDirPath = path_1.default.isAbsolute(statusFileArg)

package/build/commands/job/spec/execute.spec.js

@@ -78,7 +78,6 @@ var target = new types_1.Target({
     appLoc: '/test',
     contextName: 'Mock Context'
 });
-var statusFile;
 describe('executeJobViya', function () {
     beforeEach(function () { return __awaiter(void 0, void 0, void 0, function () {
         return __generator(this, function (_a) {
@@ -95,7 +94,7 @@ describe('executeJobViya', function () {
     it('should set streamLog to true when the flag is passed in', function () { return __awaiter(void 0, void 0, void 0, function () {
         return __generator(this, function (_a) {
             switch (_a.label) {
-                case 0: return [4 /*yield*/, (0, execute_1.executeJobViya)(sasjs, mocks_1.mockAuthConfig, 'test/job', target, false, false, testLogsPath, testFilePath, false, false, undefined, true)];
+                case 0: return [4 /*yield*/, (0, execute_1.executeJobViya)(sasjs, mocks_1.mockAuthConfig, 'test/job', target, false, false, testLogsPath, testFilePath, false, undefined, true, false)];
                 case 1:
                     _a.sent();
                     expect(sasjs.startComputeJob).toHaveBeenCalledWith('test/job', null, {
@@ -105,7 +104,7 @@ describe('executeJobViya', function () {
                         pollInterval: 1000,
                         streamLog: true,
                         logFolderPath: testLogsPath
-                    }, true, undefined);
+                    }, true, undefined, undefined);
                     return [4 /*yield*/, (0, utils_1.deleteFile)(testFilePath)];
                 case 2:
                     _a.sent();
@@ -116,7 +115,7 @@ describe('executeJobViya', function () {
     it('should set streamLog to false when the flag is false', function () { return __awaiter(void 0, void 0, void 0, function () {
         return __generator(this, function (_a) {
             switch (_a.label) {
-                case 0: return [4 /*yield*/, (0, execute_1.executeJobViya)(sasjs, mocks_1.mockAuthConfig, 'test/job', target, false, false, testLogsPath, testFilePath, false, false, undefined, false)];
+                case 0: return [4 /*yield*/, (0, execute_1.executeJobViya)(sasjs, mocks_1.mockAuthConfig, 'test/job', target, false, false, testLogsPath, testFilePath, false, undefined, false, false)];
                 case 1:
                     _a.sent();
                     expect(sasjs.startComputeJob).toHaveBeenCalledWith('test/job', null, {
@@ -126,7 +125,52 @@ describe('executeJobViya', function () {
                         pollInterval: 1000,
                         streamLog: false,
                         logFolderPath: testLogsPath
-                    }, true, undefined);
+                    }, true, undefined, undefined);
+                    return [4 /*yield*/, (0, utils_1.deleteFile)(testFilePath)];
+                case 2:
+                    _a.sent();
+                    return [2 /*return*/];
+            }
+        });
+    }); });
+    it('should pass verbose as undefined when the flag is false', function () { return __awaiter(void 0, void 0, void 0, function () {
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0: return [4 /*yield*/, (0, execute_1.executeJobViya)(sasjs, mocks_1.mockAuthConfig, 'test/job', target, false, false, testLogsPath, testFilePath, false, undefined, false, false)];
+                case 1:
+                    _a.sent();
+                    expect(sasjs.startComputeJob).toHaveBeenCalledWith('test/job', null, {
+                        contextName: 'Mock Context'
+                    }, mocks_1.mockAuthConfig, true, {
+                        maxPollCount: 24 * 60 * 60,
+                        pollInterval: 1000,
+                        streamLog: false,
+                        logFolderPath: testLogsPath
+                    }, true, undefined, undefined);
+                    return [4 /*yield*/, (0, utils_1.deleteFile)(testFilePath)];
+                case 2:
+                    _a.sent();
+                    return [2 /*return*/];
+            }
+        });
+    }); });
+    it('should pass verbose as true when the flag is true', function () { return __awaiter(void 0, void 0, void 0, function () {
+        var verbose;
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0:
+                    verbose = true;
+                    return [4 /*yield*/, (0, execute_1.executeJobViya)(sasjs, mocks_1.mockAuthConfig, 'test/job', target, false, false, testLogsPath, testFilePath, false, undefined, false, verbose)];
+                case 1:
+                    _a.sent();
+                    expect(sasjs.startComputeJob).toHaveBeenCalledWith('test/job', null, {
+                        contextName: 'Mock Context'
+                    }, mocks_1.mockAuthConfig, true, {
+                        maxPollCount: 24 * 60 * 60,
+                        pollInterval: 1000,
+                        streamLog: false,
+                        logFolderPath: testLogsPath
+                    }, true, undefined, verbose);
                     return [4 /*yield*/, (0, utils_1.deleteFile)(testFilePath)];
                 case 2:
                     _a.sent();

package/build/commands/job/spec/job.spec.server.viya.js

@@ -470,8 +470,7 @@ describe('sasjs job execute with Viya', function () {
                     mockExit = (0, test_1.mockProcessExit)();
                     return [4 /*yield*/, expect(executeWrapper({
                             jobPath: 'jobs/testJob/job',
-                            waitForJob: true,
-                            returnStatusOnly: true
+                            waitForJob: true
                         })).toResolve()];
                 case 1:
                     _a.sent();
@@ -488,8 +487,7 @@ describe('sasjs job execute with Viya', function () {
                     mockExit = (0, test_1.mockProcessExit)();
                     return [4 /*yield*/, expect(executeWrapper({
                             jobPath: 'jobs/testJob/jobWithWarning',
-                            waitForJob: true,
-                            returnStatusOnly: true
+                            waitForJob: true
                         })).toResolve()];
                 case 1:
                     _a.sent();
@@ -507,7 +505,6 @@ describe('sasjs job execute with Viya', function () {
                     return [4 /*yield*/, expect(executeWrapper({
                             jobPath: 'jobs/testJob/jobWithWarning',
                             waitForJob: true,
-                            returnStatusOnly: true,
                             ignoreWarnings: true
                         })).toResolve()];
                 case 1:
@@ -526,8 +523,7 @@ describe('sasjs job execute with Viya', function () {
                     return [4 /*yield*/, expect(executeWrapper({
                             jobPath: 'jobs/testJob/failingJob',
                             waitForJob: true,
-                            logFile: path_1.default.join(process.projectDir, 'failingJob.log'),
-                            returnStatusOnly: true
+                            logFile: path_1.default.join(process.projectDir, 'failingJob.log')
                         })).toResolve()];
                 case 1:
                     _a.sent();
@@ -556,8 +552,7 @@ describe('sasjs job execute with Viya', function () {
                     mockExit = (0, test_1.mockProcessExit)();
                     return [4 /*yield*/, expect(executeWrapper({
                             jobPath: 'jobs/testJob/failingJob_DOES_NOT_EXIST',
-                            waitForJob: true,
-                            returnStatusOnly: true
+                            waitForJob: true
                         })).toResolve()];
                 case 1:
                     _a.sent();
@@ -601,10 +596,10 @@ describe('sasjs job execute with Viya', function () {
     }); });
 });
 var executeWrapper = function (_a) {
-    var _b = _a.adapter, adapter = _b === void 0 ? sasjs : _b, jobPath = _a.jobPath, _c = _a.waitForJob, waitForJob = _c === void 0 ? false : _c, _d = _a.output, output = _d === void 0 ? false : _d, _e = _a.logFile, logFile = _e === void 0 ? undefined : _e, _f = _a.statusFile, statusFile = _f === void 0 ? undefined : _f, _g = _a.returnStatusOnly, returnStatusOnly = _g === void 0 ? false : _g, _h = _a.ignoreWarnings, ignoreWarnings = _h === void 0 ? false : _h, _j = _a.source, source = _j === void 0 ? undefined : _j, _k = _a.streamLog, streamLog = _k === void 0 ? false : _k;
+    var _b = _a.adapter, adapter = _b === void 0 ? sasjs : _b, jobPath = _a.jobPath, _c = _a.waitForJob, waitForJob = _c === void 0 ? false : _c, _d = _a.output, output = _d === void 0 ? false : _d, _e = _a.logFile, logFile = _e === void 0 ? undefined : _e, _f = _a.statusFile, statusFile = _f === void 0 ? undefined : _f, _g = _a.ignoreWarnings, ignoreWarnings = _g === void 0 ? false : _g, _h = _a.source, source = _h === void 0 ? undefined : _h, _j = _a.streamLog, streamLog = _j === void 0 ? false : _j;
     return __awaiter(void 0, void 0, void 0, function () {
-        return __generator(this, function (_l) {
-            return [2 /*return*/, (0, execute_1.executeJobViya)(adapter, authConfig, jobPath, target, waitForJob, output, logFile, statusFile, returnStatusOnly, ignoreWarnings, source, streamLog)];
+        return __generator(this, function (_k) {
+            return [2 /*return*/, (0, execute_1.executeJobViya)(adapter, authConfig, jobPath, target, waitForJob, output, logFile, statusFile, ignoreWarnings, source, streamLog, false)];
         });
     });
 };

package/build/commands/job/spec/jobCommand.spec.js

@@ -183,17 +183,17 @@ describe('JobCommand', function () {
                 }
             });
         }); });
-        it('should pass wait as true if returnStatusOnly flag is present', function () { return __awaiter(void 0, void 0, void 0, function () {
+        it('should pass wait as true if verbose flag is present', function () { return __awaiter(void 0, void 0, void 0, function () {
             var _a;
             return __generator(this, function (_b) {
                 switch (_b.label) {
-                    case 0: return [4 /*yield*/, executeCommandWrapper([jobPath, '--returnStatusOnly'])];
+                    case 0: return [4 /*yield*/, executeCommandWrapper([jobPath, '--verbose'])];
                     case 1:
                         _b.sent();
                         (_a = expect(viyaExecuteModule.executeJobViya)).toHaveBeenCalledWith.apply(_a, __spreadArray([], __read(executeCalledWith({
                             jobPath: jobPath,
                             waitForJob: true,
-                            returnStatusOnly: true
+                            verbose: true
                         })), false));
                         return [2 /*return*/];
                 }
@@ -215,10 +215,10 @@ describe('JobCommand', function () {
                             statusFile,
                             '--source',
                             source,
-                            '--returnStatusOnly',
                             '--ignoreWarnings',
                             '--wait',
-                            '--streamLog'
+                            '--streamLog',
+                            '--verbose'
                         ])];
                     case 1:
                         _b.sent();
@@ -228,10 +228,10 @@ describe('JobCommand', function () {
                             output: output,
                             logFile: path_1.default.join(projectFolder, log),
                             statusFile: path_1.default.join(projectFolder, statusFile),
-                            returnStatusOnly: true,
                             ignoreWarnings: true,
                             source: source,
-                            streamLog: true
+                            streamLog: true,
+                            verbose: true
                         })), false));
                         return [2 /*return*/];
                 }
@@ -253,10 +253,10 @@ describe('JobCommand', function () {
                             statusFile,
                             '-s',
                             source,
-                            '-r',
                             '-i',
                             '-w',
-                            '--streamLog'
+                            '--streamLog',
+                            '-v'
                         ])];
                     case 1:
                         _b.sent();
@@ -266,10 +266,10 @@ describe('JobCommand', function () {
                             output: output,
                             logFile: path_1.default.join(projectFolder, log),
                             statusFile: path_1.default.join(projectFolder, statusFile),
-                            returnStatusOnly: true,
                             ignoreWarnings: true,
                             source: source,
-                            streamLog: true
+                            streamLog: true,
+                            verbose: true
                         })), false));
                         return [2 /*return*/];
                 }
@@ -478,6 +478,19 @@ describe('JobCommand', function () {
             });
         }); });
     });
+    it('should log deprecation warning if returnStatusOnly flag is present', function () { return __awaiter(void 0, void 0, void 0, function () {
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0:
+                    jest.spyOn(process.logger, 'warn');
+                    return [4 /*yield*/, executeCommandWrapper([jobPath, '--returnStatusOnly'])];
+                case 1:
+                    _a.sent();
+                    expect(process.logger.warn).toHaveBeenCalledWith('--returnStatusOnly (-r) flag is deprecated.');
+                    return [2 /*return*/];
+            }
+        });
+    }); });
 });
 var setupMocksForViya = function () {
     jest.resetAllMocks();
@@ -527,7 +540,7 @@ var setupMocksForSAS9 = function () {
     jest.spyOn(process.logger, 'error');
 };
 var executeCalledWith = function (_a) {
-    var jobPath = _a.jobPath, _b = _a.waitForJob, waitForJob = _b === void 0 ? false : _b, _c = _a.output, output = _c === void 0 ? false : _c, _d = _a.logFile, logFile = _d === void 0 ? undefined : _d, _e = _a.statusFile, statusFile = _e === void 0 ? undefined : _e, _f = _a.returnStatusOnly, returnStatusOnly = _f === void 0 ? false : _f, _g = _a.ignoreWarnings, ignoreWarnings = _g === void 0 ? false : _g, _h = _a.source, source = _h === void 0 ? undefined : _h, _j = _a.streamLog, streamLog = _j === void 0 ? false : _j;
+    var jobPath = _a.jobPath, _b = _a.waitForJob, waitForJob = _b === void 0 ? false : _b, _c = _a.output, output = _c === void 0 ? false : _c, _d = _a.logFile, logFile = _d === void 0 ? undefined : _d, _e = _a.statusFile, statusFile = _e === void 0 ? undefined : _e, _f = _a.ignoreWarnings, ignoreWarnings = _f === void 0 ? false : _f, _g = _a.source, source = _g === void 0 ? undefined : _g, _h = _a.streamLog, streamLog = _h === void 0 ? false : _h, _j = _a.verbose, verbose = _j === void 0 ? false : _j;
     return [
         expect.anything(),
         authConfig,
@@ -537,10 +550,10 @@ var executeCalledWith = function (_a) {
         output,
         logFile,
         statusFile,
-        returnStatusOnly,
         ignoreWarnings,
         source,
-        streamLog
+        streamLog,
+        verbose
     ];
 };
 var executeCommandWrapper = function (additionalParams) { return __awaiter(void 0, void 0, void 0, function () {

package/build/commands/run/run.js

@@ -298,7 +298,7 @@ function createOutputFile(log, logFilePath, silent) {
                     if (!logFilePath) {
                         logFilePath = path_1.default.join(buildDestinationResultsFolder, "sasjs-run-".concat(timestamp, ".log"));
                     }
-                    return [4 /*yield*/, (0, saveLog_1.saveLog)(log || '', logFilePath, '', false, silent)];
+                    return [4 /*yield*/, (0, saveLog_1.saveLog)(log || '', logFilePath, '', silent)];
                 case 1:
                     _a.sent();
                     return [2 /*return*/, logFilePath];

package/build/commands/run/spec/run.spec.js

@@ -104,7 +104,7 @@ describe('sasjs run', function () {
                         return [4 /*yield*/, (0, run_1.runSasCode)(target, filePath, false, '')];
                     case 1:
                         _a.sent();
-                        expect(utils_2.saveLog).toHaveBeenCalledWith(logData, expect.stringMatching(/sasjs-run-\d{14}\.log$/), '', false, undefined);
+                        expect(utils_2.saveLog).toHaveBeenCalledWith(logData, expect.stringMatching(/sasjs-run-\d{14}\.log$/), '', undefined);
                         return [2 /*return*/];
                 }
             });

package/build/utils/config.d.ts

@@ -97,8 +97,9 @@ export declare const overrideEnvVariables: (targetName: string) => Promise<void>
 export declare const getTestSetUp: (target: Target) => Promise<string | undefined>;
 export declare const getTestTearDown: (target: Target) => Promise<string | undefined>;
 /**
- * Returns configuration object of SAS Adapter and authentication configuration
- * @param {Target} target - the target to get auth configuration from.
+ * Provides instance of @sasjs/adapter.
+ * @param target - server server.
+ * @returns - instance of @sasjs/adapter.
  */
 export declare function getSASjs(target: Target): SASjs;
 /**

package/build/utils/config.js

@@ -1143,8 +1143,9 @@ var getTestTearDown = function (target) { return __awaiter(void 0, void 0, void
 }); };
 exports.getTestTearDown = getTestTearDown;
 /**
- * Returns configuration object of SAS Adapter and authentication configuration
- * @param {Target} target - the target to get auth configuration from.
+ * Provides instance of @sasjs/adapter.
+ * @param target - server server.
+ * @returns - instance of @sasjs/adapter.
  */
 function getSASjs(target) {
     return new node_1.default({
@@ -1154,7 +1155,8 @@ function getSASjs(target) {
         contextName: target.contextName,
         httpsAgentOptions: target.httpsAgentOptions,
         debug: true,
-        useComputeApi: target.serverType === utils_1.ServerType.SasViya
+        useComputeApi: target.serverType === utils_1.ServerType.SasViya,
+        verbose: !!process.env.VERBOSE // any not empty string should be considered as true
     });
 }
 exports.getSASjs = getSASjs;

package/build/utils/saveLog.d.ts

@@ -1,2 +1,9 @@
 import { LogJson } from '../types';
-export declare const saveLog: (logData: LogJson | string, logFile: string | undefined, jobPath: string, returnStatusOnly: boolean, silent?: boolean) => Promise<void>;
+/**
+ * Saves log to the log file.
+ * @param logData - log content.
+ * @param logFile - file path to log file.
+ * @param jobPath - file path to job, job name will be used as a fallback name for log file.
+ * @param silent - boolean indicating if additional info should be logged.
+ */
+export declare const saveLog: (logData: LogJson | string, logFile: string | undefined, jobPath: string, silent?: boolean) => Promise<void>;

package/build/utils/saveLog.js

@@ -43,7 +43,14 @@ exports.saveLog = void 0;
 var utils_1 = require("@sasjs/utils");
 var path_1 = __importDefault(require("path"));
 var _1 = require(".");
-var saveLog = function (logData, logFile, jobPath, returnStatusOnly, silent) {
+/**
+ * Saves log to the log file.
+ * @param logData - log content.
+ * @param logFile - file path to log file.
+ * @param jobPath - file path to job, job name will be used as a fallback name for log file.
+ * @param silent - boolean indicating if additional info should be logged.
+ */
+var saveLog = function (logData, logFile, jobPath, silent) {
     if (silent === void 0) { silent = false; }
     return __awaiter(void 0, void 0, void 0, function () {
         var error, logPath, folderPath, parentFolderPath, logLines;
@@ -51,6 +58,7 @@ var saveLog = function (logData, logFile, jobPath, returnStatusOnly, silent) {
         return __generator(this, function (_b) {
             switch (_b.label) {
                 case 0:
+                    // throw an error if log is an object containing error property
                     if (typeof logData !== 'string') {
                         error = logData.error;
                         if (error)
@@ -75,7 +83,7 @@ var saveLog = function (logData, logFile, jobPath, returnStatusOnly, silent) {
                     return [4 /*yield*/, (0, utils_1.createFile)(logPath, logLines)];
                 case 4:
                     _b.sent();
-                    if (!returnStatusOnly && !silent)
+                    if (!silent)
                         (0, _1.displaySuccess)("Log saved to ".concat(logPath));
                     return [2 /*return*/];
             }

package/build/utils/spec/config.spec.js

@@ -70,7 +70,6 @@ var path_1 = __importDefault(require("path"));
 var utils_1 = require("@sasjs/utils");
 var test_1 = require("../test");
 var setConstants_1 = require("../setConstants");
-jest.mock('@sasjs/adapter/node');
 describe('getAccessToken', function () {
     beforeEach(function () { return __awaiter(void 0, void 0, void 0, function () {
         return __generator(this, function (_a) {
@@ -726,3 +725,63 @@ describe('removeFromLocalConfig', function () {
         });
     }); });
 });
+describe('getSASjs', function () {
+    it('should set sasjsConfig according to target', function () {
+        var target = {
+            serverUrl: 'test_serverUrl',
+            appLoc: 'test_appLoc',
+            serverType: utils_1.ServerType.SasViya,
+            contextName: 'test_contextName',
+            httpsAgentOptions: {
+                caPath: 'test_caPath',
+                keyPath: 'test_keyPath',
+                certPath: 'test_certPath',
+                allowInsecureRequests: false
+            }
+        };
+        var sasjs = (0, config_1.getSASjs)(target);
+        var sasjsConfig = sasjs.getSasjsConfig();
+        var serverUrl = sasjsConfig.serverUrl, appLoc = sasjsConfig.appLoc, serverType = sasjsConfig.serverType, contextName = sasjsConfig.contextName, httpsAgentOptions = sasjsConfig.httpsAgentOptions;
+        expect(serverUrl).toEqual(target.serverUrl);
+        expect(appLoc).toEqual(target.appLoc);
+        expect(serverType).toEqual(target.serverType);
+        expect(contextName).toEqual(target.contextName);
+        expect(httpsAgentOptions).toEqual(target.httpsAgentOptions);
+    });
+    it('should set debug to true by default', function () {
+        var sasjs = (0, config_1.getSASjs)({});
+        var sasjsConfig = sasjs.getSasjsConfig();
+        var debug = sasjsConfig.debug;
+        expect(debug).toEqual(true);
+    });
+    it('should set useComputeApi to true if server type is Viya', function () {
+        var sasjs = (0, config_1.getSASjs)({ serverType: utils_1.ServerType.SasViya });
+        var sasjsConfig = sasjs.getSasjsConfig();
+        var useComputeApi = sasjsConfig.useComputeApi;
+        expect(useComputeApi).toEqual(true);
+    });
+    it('should set useComputeApi to false if server type is not Viya', function () {
+        var sasjs = (0, config_1.getSASjs)({ serverType: utils_1.ServerType.Sas9 });
+        var sasjsConfig = sasjs.getSasjsConfig();
+        var useComputeApi = sasjsConfig.useComputeApi;
+        expect(useComputeApi).toEqual(false);
+        sasjs = (0, config_1.getSASjs)({ serverType: utils_1.ServerType.Sasjs });
+        sasjsConfig = sasjs.getSasjsConfig();
+        useComputeApi = sasjsConfig.useComputeApi;
+        expect(useComputeApi).toEqual(false);
+    });
+    it('should enable verbose mode if VERBOSE env is present', function () {
+        process.env.VERBOSE = 'on';
+        var sasjs = (0, config_1.getSASjs)({});
+        var sasjsConfig = sasjs.getSasjsConfig();
+        var verbose = sasjsConfig.verbose;
+        expect(verbose).toEqual(true);
+    });
+    it('should disable verbose mode if VERBOSE env is not present', function () {
+        process.env.VERBOSE = '';
+        var sasjs = (0, config_1.getSASjs)({});
+        var sasjsConfig = sasjs.getSasjsConfig();
+        var verbose = sasjsConfig.verbose;
+        expect(verbose).toEqual(false);
+    });
+});