Fix for JUnit result parsing (#33)

* npm audit fix
* upgrade to test parser 0.1.19
* additional tests for metadata
* mstest support enabled

PublishTestPlanResultsV1/framework/TestFrameworkFormat.ts

@@ -4,6 +4,6 @@ export enum TestFrameworkFormat {
   cucumber = "cucumber",
   mocha = "mocha",
   nunit = "nunit",
-  testng = "testng"
-  //mstest = "mstest"
+  testng = "testng",
+  mstest = "mstest"
 }

PublishTestPlanResultsV1/framework/TestFrameworkResultReader.ts

@@ -23,7 +23,8 @@ export async function readResults(parameters: TestFrameworkParameters): Promise<
       let result = new TestFrameworkResult(test.name, test.status);
       result.failure = test.failure;
       result.stacktrace = test.stack_trace;
-      result.properties = test.meta_data;
+      // 0.1.19 separated tags from metadata
+      result.properties = new Map<string,string>(Object.entries(test.metadata));
 
       results.push(result);
     })

PublishTestPlanResultsV1/package.json

@@ -13,7 +13,7 @@
   "dependencies": {
     "azure-devops-node-api": "^12.1.0",
     "azure-pipelines-task-lib": "^4.6.1",
-    "test-results-parser": "^0.1.7"
+    "test-results-parser": "0.1.19"
   },
   "devDependencies": {
     "@types/chai": "^4.3.11",

PublishTestPlanResultsV1/task.json

@@ -97,7 +97,8 @@
                 "cucumber": "cucumber",
                 "nUnit": "nUnit",
                 "testng": "testng",
-                "mocha": "mocha"
+                "mocha": "mocha",
+                "mstest": "mstest"
             },
             "helpMarkDown": "Represents which compatible framework the test results are formatted in. (JUnit, XUnit, TestNG, Mocha, Cucumber, etc)"
         },

PublishTestPlanResultsV1/test/TestFrameworkResultReader.specs.ts

@@ -22,6 +22,9 @@ describe("TestFramework Results Reader", () => {
 
     // assert
     expect(results.length).to.eq(1);
+    expect(String(results[0].properties.get("TestID"))).to.equal('1234');
+    expect(results[0].properties.get("TestLevel")).to.equal('Regression');
+    expect(results[0].properties.get("TestProduct")).to.equal('TestProductExample');
   });
 
 
@@ -90,20 +93,18 @@ describe("TestFramework Results Reader", () => {
     expect(results.length).to.be.greaterThan(1);
   });
 
-
-  // waiting for 0.1.7 that includes this
-  // it("Can read MStests results", async () => {
-  //   // arrange
-  //   var files = [];
-  //   files.push(path.join(baseDir, "mstest", "testresults.trx"));
-  //   var parameters = new TestFrameworkParameters(files, "mstest");
+  it("Can read MStests results", async () => {
+    // arrange
+    var files = [];
+    files.push(path.join(baseDir, "mstest", "testresults.trx"));
+    var parameters = new TestFrameworkParameters(files, "mstest");
 
-  //   // act
-  //   var results = await TestFrameworkResultReader.readResults(parameters);
+    // act
+    var results = await TestFrameworkResultReader.readResults(parameters);
 
-  //   // assert
-  //   expect(results.length).to.eq(1);
-  // });
+    // assert
+    expect(results.length).to.eq(10);
+  });
 
 
 })