azure-pipelines-task-lib 4.11.0 → 4.12.1

package/internal.d.ts

@@ -16,6 +16,10 @@ export declare enum IssueSource {
     CustomerScript = "CustomerScript",
     TaskInternal = "TaskInternal"
 }
+export declare enum IssueAuditAction {
+    Unknown = 0,
+    ShellTasksValidation = 1
+}
 export declare function _startsWith(str: string, start: string): boolean;
 export declare function _endsWith(str: string, end: string): boolean;
 export declare function _truncateBeforeSensitiveKeyword(str: string, sensitiveKeywordsPattern: RegExp): string;
@@ -56,8 +60,8 @@ export interface _KnownVariableInfo {
     secret: boolean;
 }
 export declare function _command(command: string, properties: any, message: string): void;
-export declare function _warning(message: string, source?: IssueSource): void;
-export declare function _error(message: string, source?: IssueSource): void;
+export declare function _warning(message: string, source?: IssueSource, auditAction?: IssueAuditAction): void;
+export declare function _error(message: string, source?: IssueSource, auditAction?: IssueAuditAction): void;
 export declare function _debug(message: string): void;
 /**
  * Returns whether a path exists.

package/internal.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports._exposeCertSettings = exports._exposeProxySettings = exports._normalizeSeparators = exports._isRooted = exports._getDirectoryName = exports._ensureRooted = exports._isUncPath = exports._loadData = exports._ensurePatternRooted = exports._getFindInfoFromPattern = exports._cloneMatchOptions = exports._legacyFindFiles_convertPatternToRegExp = exports._which = exports._checkPath = exports._exist = exports._debug = exports._error = exports._warning = exports._command = exports._getVariableKey = exports._getVariable = exports._loc = exports._setResourcePath = exports._setErrStream = exports._setStdStream = exports._writeLine = exports._truncateBeforeSensitiveKeyword = exports._endsWith = exports._startsWith = exports.IssueSource = exports._vault = exports._knownVariableMap = void 0;
+exports._exposeCertSettings = exports._exposeProxySettings = exports._normalizeSeparators = exports._isRooted = exports._getDirectoryName = exports._ensureRooted = exports._isUncPath = exports._loadData = exports._ensurePatternRooted = exports._getFindInfoFromPattern = exports._cloneMatchOptions = exports._legacyFindFiles_convertPatternToRegExp = exports._which = exports._checkPath = exports._exist = exports._debug = exports._error = exports._warning = exports._command = exports._getVariableKey = exports._getVariable = exports._loc = exports._setResourcePath = exports._setErrStream = exports._setStdStream = exports._writeLine = exports._truncateBeforeSensitiveKeyword = exports._endsWith = exports._startsWith = exports.IssueAuditAction = exports.IssueSource = exports._vault = exports._knownVariableMap = void 0;
 var fs = require("fs");
 var path = require("path");
 var os = require("os");
@@ -29,6 +29,11 @@ var IssueSource;
     IssueSource["CustomerScript"] = "CustomerScript";
     IssueSource["TaskInternal"] = "TaskInternal";
 })(IssueSource = exports.IssueSource || (exports.IssueSource = {}));
+var IssueAuditAction;
+(function (IssueAuditAction) {
+    IssueAuditAction[IssueAuditAction["Unknown"] = 0] = "Unknown";
+    IssueAuditAction[IssueAuditAction["ShellTasksValidation"] = 1] = "ShellTasksValidation";
+})(IssueAuditAction = exports.IssueAuditAction || (exports.IssueAuditAction = {}));
 //-----------------------------------------------------
 // Validation Checks
 //-----------------------------------------------------
@@ -268,14 +273,24 @@ function _command(command, properties, message) {
     _writeLine(taskCmd.toString());
 }
 exports._command = _command;
-function _warning(message, source) {
+function _warning(message, source, auditAction) {
     if (source === void 0) { source = IssueSource.TaskInternal; }
-    _command('task.issue', { 'type': 'warning', 'source': source, 'correlationId': _commandCorrelationId }, message);
+    _command('task.issue', {
+        'type': 'warning',
+        'source': source,
+        'correlationId': _commandCorrelationId,
+        'auditAction': auditAction
+    }, message);
 }
 exports._warning = _warning;
-function _error(message, source) {
+function _error(message, source, auditAction) {
     if (source === void 0) { source = IssueSource.TaskInternal; }
-    _command('task.issue', { 'type': 'error', 'source': source, 'correlationId': _commandCorrelationId }, message);
+    _command('task.issue', {
+        'type': 'error',
+        'source': source,
+        'correlationId': _commandCorrelationId,
+        'auditAction': auditAction
+    }, message);
 }
 exports._error = _error;
 function _debug(message) {

package/mock-test.d.ts

@@ -10,7 +10,7 @@ export declare class MockTestRunner {
     succeeded: boolean;
     errorIssues: string[];
     warningIssues: string[];
-    LoadAsync(testPath: string, taskJsonPath?: string): Promise<MockTestRunner>;
+    LoadAsync(testPath?: string, taskJsonPath?: string): Promise<MockTestRunner>;
     get failed(): boolean;
     ran(cmdline: string): boolean;
     createdErrorIssue(message: string): boolean;
@@ -18,12 +18,7 @@ export declare class MockTestRunner {
     stdOutContained(message: string): boolean;
     stdErrContained(message: string): boolean;
     runAsync(nodeVersion?: number): Promise<void>;
-    /**
-    * @deprecated This method uses library which is not prefered to use on production
-    */
-    run(nodeVersion?: number): void;
     private getNodePath;
-    private getNodePathSync;
     private getNodeVersion;
     private getTaskJsonPath;
     private downloadNode;

package/mock-test.js

@@ -39,12 +39,10 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.MockTestRunner = void 0;
 var cp = require("child_process");
 var fs = require("fs");
-var ncp = require("child_process");
 var os = require("os");
 var path = require("path");
 var cmdm = require("./taskcommand");
 var shelljs = require("shelljs");
-var deasync = require("deasync");
 var Downloader = require("nodejs-file-downloader");
 var COMMAND_TAG = '[command]';
 var COMMAND_LENGTH = COMMAND_TAG.length;
@@ -65,7 +63,6 @@ var MockTestRunner = /** @class */ (function () {
             return;
         this._taskJsonPath = taskJsonPath || '';
         this._testPath = testPath;
-        this.nodePath = this.getNodePathSync();
     }
     MockTestRunner.prototype.LoadAsync = function (testPath, taskJsonPath) {
         return __awaiter(this, void 0, void 0, function () {
@@ -80,8 +77,12 @@ var MockTestRunner = /** @class */ (function () {
                                         resolve(this);
                                         return [2 /*return*/];
                                     }
-                                    this._taskJsonPath = taskJsonPath || '';
-                                    this._testPath = testPath;
+                                    if (testPath) {
+                                        this._testPath = testPath;
+                                    }
+                                    if (taskJsonPath) {
+                                        this._taskJsonPath = taskJsonPath;
+                                    }
                                     _a = this;
                                     return [4 /*yield*/, this.getNodePath()];
                                 case 1:
@@ -133,8 +134,15 @@ var MockTestRunner = /** @class */ (function () {
                         return [4 /*yield*/, this.getNodePath(nodeVersion)];
                     case 1:
                         nodePath = _a.sent();
-                        _a.label = 2;
+                        return [3 /*break*/, 4];
                     case 2:
+                        if (!!nodePath) return [3 /*break*/, 4];
+                        return [4 /*yield*/, this.getNodePath()];
+                    case 3:
+                        nodePath = _a.sent();
+                        this.nodePath = nodePath;
+                        _a.label = 4;
+                    case 4:
                         spawn = cp.spawnSync(nodePath, [this._testPath]);
                         // Clean environment
                         Object.keys(process.env)
@@ -199,14 +207,6 @@ var MockTestRunner = /** @class */ (function () {
             });
         });
     };
-    /**
-    * @deprecated This method uses library which is not prefered to use on production
-    */
-    MockTestRunner.prototype.run = function (nodeVersion) {
-        var completeExecution = false;
-        this.runAsync(nodeVersion).then(function (t) { return completeExecution = true; });
-        deasync.loopWhile(function () { return !completeExecution; });
-    };
     // Returns a path to node.exe with the correct version for this task (based on if its node10 or node)
     MockTestRunner.prototype.getNodePath = function (nodeVersion) {
         return __awaiter(this, void 0, void 0, function () {
@@ -240,12 +240,6 @@ var MockTestRunner = /** @class */ (function () {
             });
         });
     };
-    MockTestRunner.prototype.getNodePathSync = function (nodeVersion) {
-        var nodePath = '';
-        this.getNodePath(nodeVersion).then(function (t) { return nodePath = t; });
-        deasync.loopWhile(function () { return nodePath == ''; });
-        return nodePath;
-    };
     // Determines the correct version of node to use based on the contents of the task's task.json. Defaults to Node 16.
     MockTestRunner.prototype.getNodeVersion = function () {
         var taskJsonPath = this.getTaskJsonPath();
@@ -389,7 +383,7 @@ var MockTestRunner = /** @class */ (function () {
                         originalCwd = process.cwd();
                         process.chdir(downloadDestination);
                         try {
-                            ncp.execSync("tar -xzf \"" + path.join(downloadDestination, tarGzName) + "\"");
+                            cp.execSync("tar -xzf \"" + path.join(downloadDestination, tarGzName) + "\"");
                         }
                         catch (_b) {
                             throw new Error('Failed to unzip node tar.gz from ' + url);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "azure-pipelines-task-lib",
-  "version": "4.11.0",
+  "version": "4.12.1",
   "description": "Azure Pipelines Task SDK",
   "main": "./task.js",
   "typings": "./task.d.ts",
@@ -28,7 +28,6 @@
   "homepage": "https://github.com/Microsoft/azure-pipelines-task-lib",
   "dependencies": {
     "adm-zip": "^0.5.10",
-    "deasync": "^0.1.28",
     "minimatch": "3.0.5",
     "nodejs-file-downloader": "^4.11.1",
     "q": "^1.5.1",