Bugfix for extent-framework#333

Author: realulim

.gitignore

@@ -27,6 +27,10 @@ hs_err_pid*
 .project
 .settings/
 
+# netbeans
+nb-configuration.xml
+nbproject/
+
 # java folders
 target/
 test-output/

src/main/java/com/aventstack/extentreports/append/RawEntityConverter.java

@@ -78,10 +78,10 @@ else if (log.hasMedia())
     private void addMedia(Log log, ExtentTest extentTest, Throwable ex) {
         Media m = log.getMedia();
         if (m.getPath() != null) {
-            extentTest.log(log.getStatus(), ex,
+            extentTest.log(log.getStatus(), log.getDetails(), ex,
                     MediaEntityBuilder.createScreenCaptureFromPath(m.getPath()).build());
         } else if (((ScreenCapture) m).getBase64() != null) {
-            extentTest.log(log.getStatus(), ex,
+            extentTest.log(log.getStatus(), log.getDetails(), ex,
                     MediaEntityBuilder.createScreenCaptureFromBase64String(((ScreenCapture) m).getBase64())
                             .build());
         }

src/test/java/com/aventstack/extentreports/AppenderTest.java

@@ -1,13 +1,16 @@
 package com.aventstack.extentreports;
 
-import java.io.IOException;
-import java.util.List;
+import com.aventstack.extentreports.model.Log;
+import com.aventstack.extentreports.model.Media;
+import com.aventstack.extentreports.model.ScreenCapture;
+import com.aventstack.extentreports.reporter.ExtentSparkReporter;
+import com.aventstack.extentreports.reporter.JsonFormatter;
 
 import org.testng.Assert;
 import org.testng.annotations.Test;
 
-import com.aventstack.extentreports.model.ScreenCapture;
-import com.aventstack.extentreports.reporter.JsonFormatter;
+import java.io.IOException;
+import java.util.List;
 
 public class AppenderTest {
     private static final String JSON_ARCHIVE = "target/json/jsonArchive.json";
@@ -163,4 +166,38 @@ public void testWithMediaBase64() throws IOException {
         Assert.assertEquals(((ScreenCapture) list.get(0).getLogs().get(0).getMedia()).getBase64(),
                 ((ScreenCapture) test1.getModel().getLogs().get(0).getMedia()).getBase64());
     }
+
+    @Test
+    public void appendWithSparkReporter() throws IOException {
+        ExtentReports extent = new ExtentReports();
+        String testNamePrefix = "Append Test ";
+        String textWithoutScreenshotPrefix = "Just some random text ";
+        String textWithScreenshotPrefix = "Some text explaining the screenshot ";
+        for (int i = 1; i < 3; i++) {
+            String reportPath = "target/append/index.html";
+            String jsonPath = "target/append/index.json";
+            ExtentSparkReporter sparkReporter = new ExtentSparkReporter(reportPath);
+            JsonFormatter jsonReporter = new JsonFormatter(jsonPath);
+            extent.createDomainFromJsonArchive(jsonPath);
+            extent.attachReporter(jsonReporter, sparkReporter);
+            Media screenshot = MediaEntityBuilder.createScreenCaptureFromPath("img.png").build();
+            ExtentTest test = extent.createTest(testNamePrefix + i, "Test Description")
+                    .log(Status.INFO, textWithoutScreenshotPrefix + i, null, null)
+                    .log(Status.INFO, textWithScreenshotPrefix + i, null, screenshot);
+            extent.flush();
+        }
+        for (com.aventstack.extentreports.model.Test test : extent.getReport().getTestList()) {
+            if (test.getName().startsWith(testNamePrefix)) {
+                String runNumber = test.getName().substring(testNamePrefix.length());
+                for (Log logEntry : test.getLogs()) {
+                    Assert.assertEquals(Status.INFO, logEntry.getStatus());
+                    String details = logEntry.getDetails();
+                    if (logEntry.getMedia() != null) Assert.assertEquals(details, textWithScreenshotPrefix + runNumber);
+                    else Assert.assertEquals(details, textWithoutScreenshotPrefix + runNumber);
+                }
+            }
+        }
+        
+    }
+
 }