Improve the appearance of the test suite in IE6-7.

Author: savetheclocktower

test.new/static/js/test_helpers.js

@@ -1,11 +1,17 @@
 
 (function () {
+  // Needed because Mocha's HTML test runner assumes the presence of
+  // String#trim.
+  if (!String.prototype.trim) {
+    String.prototype.trim = String.prototype.strip;
+  }
 
   var CONSOLE_LOG_SUPPORTED = ('console' in window) && console.log;
   var CONSOLE_GROUP_SUPPORTED = ('console' in window) && console.group &&
    console.groupEnd;
   var CONSOLE_LOG_APPLY = true;
 
+  // TODO: Find a different way to log info in IE6.
   function info() {
     if (CONSOLE_LOG_APPLY) {
       console.log.apply(console, arguments);
@@ -274,6 +280,8 @@
         return;
       }
 
+      $('mocha').addClassName('done');
+
       if (!resultsUrl) {
         return;
       }

test.new/views/layout.erb

@@ -17,6 +17,11 @@
       border: 2px solid #eee;
     }
 
+    /* Older IEs don't support CANVAS, so we need something other than the progress indicator to show that we're done. */
+    #mocha.done {
+      border-color: #090;
+    }
+
     #mocha-stats {
       background: #fff;
     }
@@ -36,6 +41,20 @@
     }
   </style>
 
+  <!--[if lt IE 8]>
+  <style type="text/css" media="screen">
+
+    #mocha-stats {
+      position: static;
+    }
+
+    #mocha-stats li {
+      float: left;
+    }
+
+  </style>
+  <![endif]-->
+
   <script src="/prototype.js?<%= @unique_asset_string %>" type="text/javascript"></script>
   <script src="/js/mocha.js?<%= @unique_asset_string %>" type="text/javascript"></script>
   <script src="/js/proclaim.js?<%= @unique_asset_string %>" type="text/javascript"></script>