Merge pull request #312 from brettz9/master

For reporting errors, provide options to: 1) display errors by defaul…

Author: mreinstein

examples/browser/nodeunit.js

@@ -1941,6 +1941,8 @@ exports.info = "Browser-based test reporter";
 
 exports.run = function (modules, options) {
     var start = new Date().getTime();
+    var textareas = options.textareas;
+    var displayErrorsByDefault = options.displayErrorsByDefault;
 
     function setText(el, txt) {
         if ('innerText' in el) {
@@ -1989,8 +1991,8 @@ exports.run = function (modules, options) {
             test.appendChild(strong);
 
             var aList = document.createElement('ol');
-            aList.style.display = 'none';
-            test.onclick = function () {
+            aList.style.display = displayErrorsByDefault ? 'block' : 'none';
+            (displayErrorsByDefault ? strong : test).onclick = function () {
                 var d = aList.style.display;
                 aList.style.display = (d == 'none') ? 'block': 'none';
             };
@@ -1999,7 +2001,9 @@ exports.run = function (modules, options) {
                 var a = assertions[i];
                 if (a.failed()) {
                     li.innerHTML = (a.message || a.method || 'no message') +
-                        '<pre>' + (a.error.stack || a.error) + '</pre>';
+                        (textareas ?
+                          '<textarea rows="20" cols="100">' + (a.error.stack || a.error) + '</textarea>' :
+                          '<pre>' + (a.error.stack || a.error) + '</pre>');
                     li.className = 'fail';
                 }
                 else {

lib/reporters/browser.js

@@ -31,9 +31,11 @@ exports.info = "Browser-based test reporter";
  */
 
 exports.run = function (modules, options, callback) {
-    var start = new Date().getTime(), div;
-	options = options || {};
-	div = options.div || document.body;
+    var start = new Date().getTime(), div, textareas, displayErrorsByDefault;
+    options = options || {};
+    div = options.div || document.body;
+    textareas = options.textareas;
+    displayErrorsByDefault = options.displayErrorsByDefault;
 
     function setText(el, txt) {
         if ('innerText' in el) {
@@ -82,8 +84,8 @@ exports.run = function (modules, options, callback) {
             test.appendChild(strong);
 
             var aList = document.createElement('ol');
-            aList.style.display = 'none';
-            test.onclick = function () {
+            aList.style.display = displayErrorsByDefault ? 'block' : 'none';
+            (displayErrorsByDefault ? strong : test).onclick = function () {
                 var d = aList.style.display;
                 aList.style.display = (d == 'none') ? 'block': 'none';
             };
@@ -92,7 +94,9 @@ exports.run = function (modules, options, callback) {
                 var a = assertions[i];
                 if (a.failed()) {
                     li.innerHTML = (a.message || a.method || 'no message') +
-                        '<pre>' + (a.error.stack || a.error) + '</pre>';
+                        (textareas ?
+                          '<textarea rows="20" cols="100">' + (a.error.stack || a.error) + '</textarea>' :
+                          '<pre>' + (a.error.stack || a.error) + '</pre>');
                     li.className = 'fail';
                 }
                 else {