removing testRun desc

Azure · Jan 4, 2022 · 7e0ba58 · 7e0ba58
1 parent 4c04063
commit 7e0ba58
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 6 deletions.
diff --git a/lib/main.js b/lib/main.js
@@ -187,9 +187,9 @@ function getTestRunAPI(testRunId, testStatus, startTime) {
             testStatus = testRunObj.status;
             if (testStatus == "DONE") {
                 util.printTestDuration(testRunObj.vusers, startTime);
-                if (testRunObj.passFailCriteria != null && testRunObj.passFailCriteria.passFailMetrics != null)
+                if (testRunObj.passFailCriteria.passFailMetrics != null && testRunObj.passFailCriteria.passFailMetrics != undefined)
                     util.printCriteria(testRunObj.passFailCriteria.passFailMetrics);
-                if (testRunObj.testRunStatistics != null)
+                if (testRunObj.testRunStatistics != null && testRunObj.testRunStatistics != undefined)
                     util.printClientMetrics(testRunObj.testRunStatistics);
                 var testResultUrl = util.getResultFolder(testRunObj.testArtifacts);
                 if (testResultUrl != null) {

diff --git a/lib/mappers.js b/lib/mappers.js
@@ -135,7 +135,6 @@ function startTestData(testRunName) {
         displayName: getDefaultTestRunName(),
         testId: testName,
         resourceId: resourceId,
-        description: "Sample testRun",
         secrets: secretsRun,
         environmentVariables: envRun
     };
@@ -188,6 +187,8 @@ function getInputParams() {
     return __awaiter(this, void 0, void 0, function* () {
         yield getAccessToken("https://management.core.windows.net");
         YamlPath = core.getInput('loadTestConfigFile');
+        if (!(YamlPath.includes(".yaml") || YamlPath.includes(".yml")))
+            throw "Valid Yaml filePath is required in loadTestConfigFile";
         const config = yaml.load(fs.readFileSync(YamlPath, 'utf8'));
         testName = (config.testName).toLowerCase();
         if (!validateName(getFileName(testName)))

diff --git a/src/main.ts b/src/main.ts
@@ -154,9 +154,9 @@ async function getTestRunAPI(testRunId:string, testStatus:string, startTime:Date
         testStatus = testRunObj.status;
         if(testStatus == "DONE") {
             util.printTestDuration(testRunObj.vusers, startTime);
-            if(testRunObj.passFailCriteria != null && testRunObj.passFailCriteria.passFailMetrics != null)
+            if(testRunObj.passFailCriteria.passFailMetrics != null && testRunObj.passFailCriteria.passFailMetrics != undefined)
                 util.printCriteria(testRunObj.passFailCriteria.passFailMetrics)
-            if(testRunObj.testRunStatistics != null)
+            if(testRunObj.testRunStatistics != null && testRunObj.testRunStatistics != undefined)
                 util.printClientMetrics(testRunObj.testRunStatistics);
             var testResultUrl = util.getResultFolder(testRunObj.testArtifacts);
             if(testResultUrl != null) {

diff --git a/src/mappers.ts b/src/mappers.ts
@@ -113,7 +113,6 @@ export function startTestData(testRunName:string) {
         displayName: getDefaultTestRunName(),
         testId: testName,
         resourceId: resourceId,
-        description: "Sample testRun",
         secrets: secretsRun,
         environmentVariables: envRun
     };
@@ -159,6 +158,8 @@ function validateName(value:string)
 export async function getInputParams() {
     await getAccessToken("https://management.core.windows.net");
     YamlPath = core.getInput('loadTestConfigFile');
+    if(!(YamlPath.includes(".yaml") || YamlPath.includes(".yml")))
+        throw "Valid Yaml filePath is required in loadTestConfigFile";
     const config = yaml.load(fs.readFileSync(YamlPath, 'utf8'));
     testName = (config.testName).toLowerCase();
     if(!validateName(getFileName(testName)))