Merge pull request #2661 from Fork-World/2660-fail_annotation_in_xml_report

#2660 fail annotation in xml report

Author: ptrthomas

karate-core/src/main/java/com/intuit/karate/report/ReportUtils.java

@@ -216,7 +216,7 @@ public static File saveJunitXml(String targetDir, FeatureResult result, String f
             testCase.setAttribute("name", name);
             testCase.setAttribute("time", formatter.format(sr.getDurationMillis() / 1000));
             Element stepsHolder;
-            if (error != null) {
+            if (error != null && sr.isFailed()) {
                 stepsHolder = doc.createElement("failure");
                 stepsHolder.setAttribute("message", error.getMessage());
             } else {

karate-core/src/test/java/com/intuit/karate/report/ReportUtilsTest.java

@@ -81,5 +81,25 @@ void testCustomTags() {
         File file = ReportUtils.saveJunitXml("target", fr.result, null);
         assertTrue(FileUtils.toString(file).contains(expectedCustomTags));
     }
+    
+    @Test
+    void testFailAnnotationXML() {
+    	String expectedSystemout = "<system-out>"; 
+        Feature feature = Feature.read("classpath:com/intuit/karate/core/fail-tag.feature");
+        FeatureRuntime fr = FeatureRuntime.of(new Suite(), new FeatureCall(feature));
+        fr.run();
+        File file = ReportUtils.saveJunitXml("target", fr.result, null);
+        assertTrue(FileUtils.toString(file).contains(expectedSystemout));
+    }
+    
+    @Test
+    void testFailAnnotationXMLFailure() {
+    	String expectedSystemout = "<failure"; 
+        Feature feature = Feature.read("classpath:com/intuit/karate/core/fail-tag-failure.feature");
+        FeatureRuntime fr = FeatureRuntime.of(new Suite(), new FeatureCall(feature));
+        fr.run();
+        File file = ReportUtils.saveJunitXml("target", fr.result, null);
+        assertTrue(FileUtils.toString(file).contains(expectedSystemout));
+    }
 
 }