Fix JUnit message / type fields. (#48)

The message and type attributes never worked because they were expected to be nodes rather than attributes.

Note: in some cases it might have previously looked like it worked because when 'message' is not provided the annotation defaults to the first line of 'details' which in some cases happens to be the error.

__tests__/__snapshots__/java-junit.test.ts.snap

@@ -41,7 +41,7 @@ at java.lang.Thread.run(Thread.java:748)
 
     ",
                 "line": 29,
-                "message": undefined,
+                "message": "java.lang.AssertionError: expected [1.2.1] but found [1.2.0]",
                 "path": "pulsar-common/src/test/java/org/apache/pulsar/AddMissingPatchVersionTest.java",
               },
               "name": "testVersionStrings",
@@ -100,7 +100,7 @@ at java.lang.Thread.run(Thread.java:748)
 
     ",
                 "line": 29,
-                "message": undefined,
+                "message": "java.lang.AssertionError: expected [1.2.1] but found [1.2.0]",
                 "path": "pulsar-common/src/test/java/org/apache/pulsar/AddMissingPatchVersionTest.java",
               },
               "name": "testVersionStrings",

src/parsers/java-junit/java-junit-parser.ts

@@ -133,11 +133,18 @@ export class JavaJunitParser implements TestParser {
       line = src.line
     }
 
+    let message
+    if(typeof failure === 'object') {
+      message = failure.$.message
+      if(failure.$?.type) {
+        message = failure.$.type + ": "+ message
+      }
+    }
     return {
       path: filePath,
       line,
       details,
-      message: typeof failure === 'object' ? failure.message : undefined
+      message
     }
   }
 

src/parsers/java-junit/java-junit-types.ts

@@ -40,6 +40,8 @@ export interface TestCase {
 
 export interface Failure {
   _?: string
-  type: string
-  message: string
+  $: {
+    type?: string
+    message: string
+  }
 }