From 7e0ba5800e73161c66af9cbba03172dbc111a268 Mon Sep 17 00:00:00 2001
From: Sweta Kumari <swetakumari@microsoft.com>
Date: Tue, 4 Jan 2022 17:42:11 +0530
Subject: [PATCH] removing testRun desc

---
 lib/main.js    | 4 ++--
 lib/mappers.js | 3 ++-
 src/main.ts    | 4 ++--
 src/mappers.ts | 3 ++-
 4 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/lib/main.js b/lib/main.js
index 80fa0ddb..8e08dc72 100644
--- 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
index 750f57b9..d648f23c 100644
--- 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
index 25a224b3..0e5477d7 100644
--- 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
index 41a7cbd2..ea603200 100644
--- 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)))