Fixed bug in csv reading

Author: pkpio

ado-extension.json

@@ -2,7 +2,7 @@
     "manifestVersion": 1,
     "id": "android-app-size-diff-utils",
     "name": "Android app size changes",
-    "version": "0.0.26",
+    "version": "0.0.27",
     "publisher": "PraveenPendyala",
     "targets": [
         {

dist_gh_action/index.js

@@ -1,6 +1,6 @@
 {
     "name": "android-app-size-diff",
-    "version": "0.0.26",
+    "version": "0.0.27",
     "description": "Azure DevOps task to measure the size in Android app size by looking at 2 given APKs and AABs",
     "main": "index.js",
     "scripts": {
@@ -50,4 +50,4 @@
         "shelljs": "^0.8.3",
         "sync-request": "^6.1.0"
     }
-}
+}

package.json

@@ -65,7 +65,7 @@ export default class CiRunner {
             abortRun = true;
         }
 
-        const metricsList: Array<string> = metrics.split(',');
+        const metricsList: Array<string> = this.readCsv(metrics);
         var thresholdsList: Array<number> = []
         try {
             thresholdsList = this.parseThresholdsFromInput(thresholds, metricsList);
@@ -114,7 +114,7 @@ export default class CiRunner {
         }
 
         // Check if same number of thresholds and metrics are passed
-        const thresholdStrings: Array<string> = thresholdsInput.split(',');
+        const thresholdStrings: Array<string> = this.readCsv(thresholdsInput);
         if (thresholdStrings.length != metricsList.length) {
             throw 'Thresholds must be set corresponding to each metric in metrics or none!';
         }
@@ -127,5 +127,9 @@ export default class CiRunner {
         return thresholdsList;
     }
 
+    private readCsv(inputString: string): Array<string> {
+        const valuesList: Array<string> = inputString.split(',');
+        return valuesList.map((value) => value.trim());
+    }
 
 }

src/apkAnalyzer/CiRunner.ts

@@ -10,7 +10,7 @@
     "version": {
         "Major": 0,
         "Minor": 0,
-        "Patch": 26
+        "Patch": 27
     },
     "instanceNameFormat": "Android App size change - $(baseAppPath) vs $(targetAppPath)",
     "inputs": [