From 8fe80ee8859ff3b4dfcf220cf0cfb271dd7ea261 Mon Sep 17 00:00:00 2001
From: Patrick Dreyer <patrickdreyer@users.noreply.github.com>
Date: Thu, 13 Aug 2015 19:13:51 +0200
Subject: [PATCH 1/6] Preserve line breaks

StdOut, StdErr and StackTrace line breaks have been normalized and thus concatenated with only a space between.
Now the line breaks are preserved properly and makes the output much more readable
---
 TrxerConsole/Trxer.xslt | 33 +++++++++++++++++++++++++++++----
 1 file changed, 29 insertions(+), 4 deletions(-)

diff --git a/TrxerConsole/Trxer.xslt b/TrxerConsole/Trxer.xslt
index 1ada859..0f865ba 100644
--- a/TrxerConsole/Trxer.xslt
+++ b/TrxerConsole/Trxer.xslt
@@ -490,7 +490,10 @@
             <tbody>
               <tr class="visibleRow">
                 <td class="ex">
-                  <xsl:value-of select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output/t:ErrorInfo/t:StackTrace" />
+                  <xsl:value-of select="text" />
+                  <xsl:call-template name="break">
+                    <xsl:with-param name="text" select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output/t:ErrorInfo/t:StackTrace" />
+                  </xsl:call-template>
                 </td>
               </tr>
             </tbody>
@@ -538,7 +541,10 @@
 
       <xsl:variable name="MessageErrorStacktrace" select="t:ErrorInfo/t:StackTrace"/>
 
-      <xsl:variable name="StdOut" select="t:StdOut"/>
+      <xsl:variable name="StdOut" select="text" />
+      <xsl:call-template name="break">
+        <xsl:with-param name="text" select="t:StdOut" />
+      </xsl:call-template>
       <xsl:if test="$StdOut or $MessageErrorStacktrace">
         <xsl:value-of select="$StdOut"/>
         <xsl:if test="$MessageErrorStacktrace">
@@ -548,8 +554,10 @@
           <br/>
         </xsl:if>
       </xsl:if>
-      <xsl:value-of select="t:StdErr" />
-      <xsl:variable name="StdErr" select="t:StdErr"/>
+      <xsl:variable name="StdErr" select="text" />
+      <xsl:call-template name="break">
+        <xsl:with-param name="text" select="t:StdErr" />
+      </xsl:call-template>
       <xsl:if test="$StdErr">
         <xsl:value-of select="$StdErr"/>
         <br/>
@@ -564,5 +572,22 @@
 
     </xsl:for-each>
   </xsl:template>
+
+  <xsl:template name="break">
+    <xsl:param name="text" />
+    <xsl:param name="replace" select="'&#10;'" />
+    <xsl:choose>
+      <xsl:when test="contains($text,$replace)">
+        <xsl:value-of select="substring-before($text,$replace)"/>
+        <br />
+        <xsl:call-template name="break">
+          <xsl:with-param name="text" select="substring-after($text,$replace)"/>
+        </xsl:call-template>
+      </xsl:when>
+      <xsl:otherwise>
+        <xsl:value-of select="$text" />
+      </xsl:otherwise>
+    </xsl:choose>
+  </xsl:template>
 </xsl:stylesheet>
 

From fd3f71c93bfefde3c8e722ef0dea52f59fa26fe6 Mon Sep 17 00:00:00 2001
From: Patrick Dreyer <patrickdreyer@users.noreply.github.com>
Date: Fri, 14 Aug 2015 13:35:27 +0200
Subject: [PATCH 2/6] Collabse StdOut, StdErr and StackTrace by default

Move StdOut, StdErr and StackTrace into their own rows and have em collapsible/expandable
Use toggle button instead link
---
 TrxerConsole/Trxer.xslt | 90 +++++++++++++++++++++++++----------------
 1 file changed, 56 insertions(+), 34 deletions(-)

diff --git a/TrxerConsole/Trxer.xslt b/TrxerConsole/Trxer.xslt
index 0f865ba..04c57c0 100644
--- a/TrxerConsole/Trxer.xslt
+++ b/TrxerConsole/Trxer.xslt
@@ -481,7 +481,45 @@
           <xsl:value-of select="trxreport:ToExactTimeDefinition(@duration)" />
         </td>
       </tr>
-      <tr id="{generate-id($testId)}Stacktrace" class="hiddenRow">
+      <tr id="{generate-id($testId)}StdOut" class="hiddenRow">
+        <!--Outer-->
+        <td colspan="6">
+          <div id="exceptionArrow">↳</div>
+          <table>
+            <!--Inner-->
+            <tbody>
+              <tr class="visibleRow">
+                <td class="ex">
+                  <xsl:value-of select="text" />
+                  <xsl:call-template name="break">
+                    <xsl:with-param name="text" select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output/t:StdOut" />
+                  </xsl:call-template>
+                </td>
+              </tr>
+            </tbody>
+          </table>
+        </td>
+      </tr>
+      <tr id="{generate-id($testId)}StdErr" class="hiddenRow">
+        <!--Outer-->
+        <td colspan="6">
+          <div id="exceptionArrow">↳</div>
+          <table>
+            <!--Inner-->
+            <tbody>
+              <tr class="visibleRow">
+                <td class="ex">
+                  <xsl:value-of select="text" />
+                  <xsl:call-template name="break">
+                    <xsl:with-param name="text" select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output/t:StdErr" />
+                  </xsl:call-template>
+                </td>
+              </tr>
+            </tbody>
+          </table>
+        </td>
+      </tr>
+      <tr id="{generate-id($testId)}StackTrace" class="hiddenRow">
         <!--Outer-->
         <td colspan="6">
           <div id="exceptionArrow">↳</div>
@@ -525,51 +563,35 @@
           <div class="atachmentImage" onclick="show('floatingImageBackground');updateFloatingImage('{$MessageErrorInfo}');"></div>
         </xsl:when>
       </xsl:choose>
-
     </xsl:for-each>
   </xsl:template>
 
-
-
-
-
-
-
   <xsl:template name="debugInfo">
     <xsl:param name="testId" />
     <xsl:for-each select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output">
-
-      <xsl:variable name="MessageErrorStacktrace" select="t:ErrorInfo/t:StackTrace"/>
-
-      <xsl:variable name="StdOut" select="text" />
+      <xsl:variable name="Message" select="text" />
       <xsl:call-template name="break">
-        <xsl:with-param name="text" select="t:StdOut" />
+        <xsl:with-param name="text" select="t:ErrorInfo/t:Message" />
       </xsl:call-template>
-      <xsl:if test="$StdOut or $MessageErrorStacktrace">
-        <xsl:value-of select="$StdOut"/>
-        <xsl:if test="$MessageErrorStacktrace">
-          <a style="float:right;" id="{generate-id($testId)}StacktraceToggle" href="javascript:ShowHide('{generate-id($testId)}Stacktrace','{generate-id($testId)}StacktraceToggle','Show Stacktrace','Hide Stacktrace');">Show Stacktrace</a>
-        </xsl:if>
-        <xsl:if test="$StdOut">
-          <br/>
-        </xsl:if>
+
+      <xsl:variable name="StdOut" select="t:StdOut" />
+      <xsl:variable name="StdErr" select="t:StdErr" />
+      <xsl:variable name="StackTrace" select="t:ErrorInfo/t:StackTrace"/>
+      <xsl:if test="$StdOut">
+        <div class="OpenMoreButton" style="float:right;" onclick="ShowHide('{generate-id($testId)}StdOut','{generate-id($testId)}StdOutButton','Show StdOut','Hide StdOut');">
+          <div class="MoreButtonText" id="{generate-id($testId)}StdOutButton">Show StdOut</div>
+        </div>
       </xsl:if>
-      <xsl:variable name="StdErr" select="text" />
-      <xsl:call-template name="break">
-        <xsl:with-param name="text" select="t:StdErr" />
-      </xsl:call-template>
       <xsl:if test="$StdErr">
-        <xsl:value-of select="$StdErr"/>
-        <br/>
+        <div class="OpenMoreButton" style="float:right;" onclick="ShowHide('{generate-id($testId)}StdErr','{generate-id($testId)}StdErrButton','Show StdErr','Hide StdErr');">
+          <div class="MoreButtonText" id="{generate-id($testId)}StdErrButton">Show StdErr</div>
+        </div>
       </xsl:if>
-      <xsl:variable name="MessageErrorInfo" select="t:ErrorInfo/t:Message"/>
-      <xsl:if test="$MessageErrorInfo">
-        <xsl:value-of select="$MessageErrorInfo"/>
-        <br/>
+      <xsl:if test="$StackTrace">
+        <div class="OpenMoreButton" style="float:right;" onclick="ShowHide('{generate-id($testId)}StackTrace','{generate-id($testId)}StackTraceButton','Show StackTrace','Hide StackTrace');">
+          <div class="MoreButtonText" id="{generate-id($testId)}StackTraceButton">Show StackTrace</div>
+        </div>
       </xsl:if>
-
-
-
     </xsl:for-each>
   </xsl:template>
 

From 6ac5acfcdefbd02d25ab33834500f247f49648c9 Mon Sep 17 00:00:00 2001
From: Patrick Dreyer <patrickdreyer@users.noreply.github.com>
Date: Mon, 24 Aug 2015 15:55:29 +0200
Subject: [PATCH 3/6] Enlarge tables to maximize usage of horizontal space

---
 TrxerConsole/TrxerTable.css | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/TrxerConsole/TrxerTable.css b/TrxerConsole/TrxerTable.css
index b74b7f4..7895170 100644
--- a/TrxerConsole/TrxerTable.css
+++ b/TrxerConsole/TrxerTable.css
@@ -10,7 +10,7 @@ img {
 }
 
 table {
-    width: 90%;
+    width: 98%;
     margin: 1em auto;
     border-collapse: collapse;
     border: 1px solid #a2caf2;

From cc57e7009d87efa8ba133ed23af93d660d5dae52 Mon Sep 17 00:00:00 2001
From: Patrick Dreyer <patrickdreyer@users.noreply.github.com>
Date: Mon, 24 Aug 2015 15:56:17 +0200
Subject: [PATCH 4/6] Enlarge tables to maximize usage of horizontal space

---
 TrxerConsole/Trxer.css | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/TrxerConsole/Trxer.css b/TrxerConsole/Trxer.css
index dfede24..0b19a25 100644
--- a/TrxerConsole/Trxer.css
+++ b/TrxerConsole/Trxer.css
@@ -218,7 +218,7 @@ td.slowest {
 }
 
 .SummaryDiv {
-    width: 90%;
+    width: 98%;
     border-top: 1px solid #e5eff8;
     border-right: 1px solid #e5eff8;
     margin: 1em auto;

From 7a516850357c63757149c0b74e97df77d01bddfb Mon Sep 17 00:00:00 2001
From: Patrick Dreyer <patrickdreyer@users.noreply.github.com>
Date: Mon, 24 Aug 2015 15:56:47 +0200
Subject: [PATCH 5/6] Wrap long output lines part1/2

---
 TrxerConsole/Trxer.css | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/TrxerConsole/Trxer.css b/TrxerConsole/Trxer.css
index 0b19a25..27be641 100644
--- a/TrxerConsole/Trxer.css
+++ b/TrxerConsole/Trxer.css
@@ -359,3 +359,12 @@ td.slowest {
 .visibleRow {
     visibility: visible;
 }
+
+.wordwrap {
+    white-space: pre-wrap; /* css-3 */
+    white-space: -moz-pre-wrap; /* Mozilla, since 1999 */
+    white-space: -pre-wrap; /* Opera 4-6 */
+    white-space: -o-pre-wrap; /* Opera 7 */
+    word-wrap: break-word; /* Internet Explorer 5.5+ */ 
+    word-break: break-all; /* Firefox */
+}

From f20a2beac510f31789ebd9b40680d7352fa60af3 Mon Sep 17 00:00:00 2001
From: Patrick Dreyer <patrickdreyer@users.noreply.github.com>
Date: Mon, 24 Aug 2015 15:57:27 +0200
Subject: [PATCH 6/6] Wrap long output lines part2/2

---
 TrxerConsole/Trxer.xslt | 142 ++++++++++++++++++++++++----------------
 1 file changed, 87 insertions(+), 55 deletions(-)

diff --git a/TrxerConsole/Trxer.xslt b/TrxerConsole/Trxer.xslt
index 04c57c0..2bd7308 100644
--- a/TrxerConsole/Trxer.xslt
+++ b/TrxerConsole/Trxer.xslt
@@ -9,6 +9,7 @@
   <!--<xsl:namespace-alias stylesheet-prefix="t" result-prefix="#default"/>-->
 
   <msxsl:script language="C#" implements-prefix="trxreport">
+    <msxsl:using namespace="System.IO"/>
     <![CDATA[
     public string RemoveAssemblyName(string asm) 
     {
@@ -66,6 +67,38 @@
 	    }
       return string.Empty;
     }
+
+    public bool IsLongText(string text)
+    {
+      var index = 0;
+      var found = 0;
+      do
+      {
+        index = text.IndexOf("\n", index);
+        if (index != -1)
+        {
+          ++found;
+          ++index;
+        }
+      }
+      while (found < 3 && index != -1);
+      return (found > 2);
+    }
+
+    public string ShortenText(string text)
+    {
+      var ret = new StringBuilder();
+      using (var reader = new StringReader(text))
+      {
+        var lines = 3;
+        for (var line = reader.ReadLine(); lines > 0 & line != null; --lines, line = reader.ReadLine())
+        {
+          ret.Append(line);
+          ret.Append("\n");
+        }
+      }
+      return ret.ToString();
+    }
         
     ]]>
   </msxsl:script>
@@ -481,6 +514,22 @@
           <xsl:value-of select="trxreport:ToExactTimeDefinition(@duration)" />
         </td>
       </tr>
+      <tr id="{generate-id($testId)}Message" class="hiddenRow">
+        <!--Outer-->
+        <td colspan="6">
+          <div id="exceptionArrow">↳</div>
+          <table>
+            <!--Inner-->
+            <tbody>
+              <tr class="visibleRow">
+                <td class="ex wordwrap">
+                  <xsl:value-of select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output/t:ErrorInfo/t:Message"/>
+                </td>
+              </tr>
+            </tbody>
+          </table>
+        </td>
+      </tr>
       <tr id="{generate-id($testId)}StdOut" class="hiddenRow">
         <!--Outer-->
         <td colspan="6">
@@ -489,11 +538,8 @@
             <!--Inner-->
             <tbody>
               <tr class="visibleRow">
-                <td class="ex">
-                  <xsl:value-of select="text" />
-                  <xsl:call-template name="break">
-                    <xsl:with-param name="text" select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output/t:StdOut" />
-                  </xsl:call-template>
+                <td class="ex wordwrap">
+                  <xsl:value-of select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output/t:StdOut"/>
                 </td>
               </tr>
             </tbody>
@@ -508,11 +554,8 @@
             <!--Inner-->
             <tbody>
               <tr class="visibleRow">
-                <td class="ex">
-                  <xsl:value-of select="text" />
-                  <xsl:call-template name="break">
-                    <xsl:with-param name="text" select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output/t:StdErr" />
-                  </xsl:call-template>
+                <td class="ex wordwrap">
+                  <xsl:value-of select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output/t:StdErr"/>
                 </td>
               </tr>
             </tbody>
@@ -527,11 +570,8 @@
             <!--Inner-->
             <tbody>
               <tr class="visibleRow">
-                <td class="ex">
-                  <xsl:value-of select="text" />
-                  <xsl:call-template name="break">
-                    <xsl:with-param name="text" select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output/t:ErrorInfo/t:StackTrace" />
-                  </xsl:call-template>
+                <td class="ex wordwrap">
+                  <xsl:value-of select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output/t:ErrorInfo/t:StackTrace"/>>
                 </td>
               </tr>
             </tbody>
@@ -569,47 +609,39 @@
   <xsl:template name="debugInfo">
     <xsl:param name="testId" />
     <xsl:for-each select="/t:TestRun/t:Results/t:UnitTestResult[@testId=$testId]/t:Output">
-      <xsl:variable name="Message" select="text" />
-      <xsl:call-template name="break">
-        <xsl:with-param name="text" select="t:ErrorInfo/t:Message" />
-      </xsl:call-template>
+      <xsl:variable name="Message" select="t:ErrorInfo/t:Message" />
+      <xsl:choose>
+        <xsl:when test="trxreport:IsLongText($Message)">
+          <xsl:value-of select="trxreport:ShortenText($Message)"/>
+          <div class="OpenMoreButton" style="float:none" onclick="ShowHide('{generate-id($testId)}Message','{generate-id($testId)}MessageButton','Show Full Message','Hide Full Message');">
+            <div class="MoreButtonText" id="{generate-id($testId)}MessageButton">Show Full Message</div>
+          </div>
+        </xsl:when>
+        <xsl:otherwise>
+          <xsl:value-of select="$Message"/>
+        </xsl:otherwise>
+      </xsl:choose>
 
-      <xsl:variable name="StdOut" select="t:StdOut" />
-      <xsl:variable name="StdErr" select="t:StdErr" />
-      <xsl:variable name="StackTrace" select="t:ErrorInfo/t:StackTrace"/>
-      <xsl:if test="$StdOut">
-        <div class="OpenMoreButton" style="float:right;" onclick="ShowHide('{generate-id($testId)}StdOut','{generate-id($testId)}StdOutButton','Show StdOut','Hide StdOut');">
-          <div class="MoreButtonText" id="{generate-id($testId)}StdOutButton">Show StdOut</div>
-        </div>
-      </xsl:if>
-      <xsl:if test="$StdErr">
-        <div class="OpenMoreButton" style="float:right;" onclick="ShowHide('{generate-id($testId)}StdErr','{generate-id($testId)}StdErrButton','Show StdErr','Hide StdErr');">
-          <div class="MoreButtonText" id="{generate-id($testId)}StdErrButton">Show StdErr</div>
-        </div>
-      </xsl:if>
-      <xsl:if test="$StackTrace">
-        <div class="OpenMoreButton" style="float:right;" onclick="ShowHide('{generate-id($testId)}StackTrace','{generate-id($testId)}StackTraceButton','Show StackTrace','Hide StackTrace');">
-          <div class="MoreButtonText" id="{generate-id($testId)}StackTraceButton">Show StackTrace</div>
-        </div>
-      </xsl:if>
+      <div style="float:left;">
+        <xsl:variable name="StdOut" select="t:StdOut" />
+        <xsl:variable name="StdErr" select="t:StdErr" />
+        <xsl:variable name="StackTrace" select="t:ErrorInfo/t:StackTrace"/>
+        <xsl:if test="$StdOut">
+          <div class="OpenMoreButton" onclick="ShowHide('{generate-id($testId)}StdOut','{generate-id($testId)}StdOutButton','Show StdOut','Hide StdOut');">
+            <div class="MoreButtonText" id="{generate-id($testId)}StdOutButton">Show StdOut</div>
+          </div>
+        </xsl:if>
+        <xsl:if test="$StdErr">
+          <div class="OpenMoreButton" onclick="ShowHide('{generate-id($testId)}StdErr','{generate-id($testId)}StdErrButton','Show StdErr','Hide StdErr');">
+            <div class="MoreButtonText" id="{generate-id($testId)}StdErrButton">Show StdErr</div>
+          </div>
+        </xsl:if>
+        <xsl:if test="$StackTrace">
+          <div class="OpenMoreButton" onclick="ShowHide('{generate-id($testId)}StackTrace','{generate-id($testId)}StackTraceButton','Show StackTrace','Hide StackTrace');">
+            <div class="MoreButtonText" id="{generate-id($testId)}StackTraceButton">Show StackTrace</div>
+          </div>
+        </xsl:if>
+      </div>
     </xsl:for-each>
   </xsl:template>
-
-  <xsl:template name="break">
-    <xsl:param name="text" />
-    <xsl:param name="replace" select="'&#10;'" />
-    <xsl:choose>
-      <xsl:when test="contains($text,$replace)">
-        <xsl:value-of select="substring-before($text,$replace)"/>
-        <br />
-        <xsl:call-template name="break">
-          <xsl:with-param name="text" select="substring-after($text,$replace)"/>
-        </xsl:call-template>
-      </xsl:when>
-      <xsl:otherwise>
-        <xsl:value-of select="$text" />
-      </xsl:otherwise>
-    </xsl:choose>
-  </xsl:template>
 </xsl:stylesheet>
-