Fixed an error with the screenshots that always displayed the screen …

…source link, even for successful tests.
serenity-bdd · Nov 24, 2015 · 81c87bb · 81c87bb
1 parent fc8442b
commit 81c87bb
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/serenity-report-resources/src/main/resources/freemarker/screenshots.ftl b/serenity-report-resources/src/main/resources/freemarker/screenshots.ftl
@@ -194,7 +194,7 @@
 
     <div id="beforetable"></div>
     <div id="contenttilttle">
-        <#if testOutcome.failureDetails.conciseErrorMessage??>
+        <#if (testOutcome.result == "FAILURE" || testOutcome.result == "ERROR")>
             <div class="screenshotFailure panel panel-danger">
                 <div class="panel-heading"><span class='error-caption ellipsis'>${testOutcome.result}: ${testOutcome.failureDetails.conciseErrorMessage}</span></div>
                 <div class="panel-body">
@@ -209,7 +209,7 @@
                     <#assign caption = "${screenshot.html.description}">
                 <#else>
                     <#if testOutcome.conciseErrorMessage??>
-                        <#assign caption = "${screenshot.html.description}: <span class='error-caption'>${testOutcome.result}</span>">
+                        <#assign caption = "${screenshot.html.description}: <span class='${outcome_text}'>${testOutcome.result}</span>">
                     <#else>
                         <#assign caption = "${screenshot.html.description}">
                     </#if>

diff --git a/serenity-report-resources/src/main/resources/report-resources/css/core.css b/serenity-report-resources/src/main/resources/report-resources/css/core.css
@@ -774,6 +774,19 @@ font-family:euphemia, verdana, sans-serif; font-size:16px; color: #88a717; font-
     background-color: #30cb23; height: 1.1em;
 }
 
+.failing-color {
+    color: #f8001f;
+}
+.error-color {
+    color: darkorange;
+}
+.compromised-color {
+    color: fuchsia;
+}
+.passing-color {
+    color: #30cb23;
+}
+
 #accordion {
     list-style: none;
     padding: 0 0 0 0;