Fix the normalization of test group names before use as DOM class or id names.

lib/unittest/html_enhanced_config.dart

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * A simple unit test library for running tests in a browser.
  *
  * Provides enhanced HTML output with collapsible group headers
  * and other at-a-glance information about the test results.
  */
@@ skipping 115 matching lines @@
       List<TestCase> flattened = new List<TestCase>();
 
       groupedBy
         .getValues()
         .forEach((tList){
           tList.sort((tcA, tcB) => tcA.id - tcB.id);
           flattened.addAll(tList);
           }
         );
 
+      var nonAlphanumeric = new RegExp('[^a-z0-9A-Z]');
+
       // output group headers and test rows
       for (final test_ in flattened) {
 
         // replace everything but numbers and letters from the group name with
         // '_' so we can use in id and class properties.
-        var safeGroup = test_.currentGroup
-                              .replaceAll("(?:[^a-z0-9 ]|(?<=['\"])s)",'_')
-                              .replaceAll(' ','_');
+        var safeGroup = test_.currentGroup.replaceAll(nonAlphanumeric,'_');

[Siggi Cherem (dart-lang), 2012/06/25 21:37:23]

add space after ','

 
         if (test_.currentGroup != previousGroup){
 
           previousGroup = test_.currentGroup;
 
           var testsInGroup = results.filter(
               (TestCase t) => t.currentGroup == previousGroup);
           var groupTotalTestCount = testsInGroup.length;
           var groupTestPassedCount = testsInGroup.filter(
               (TestCase t) => t.result == 'pass').length;
@@ skipping 209 matching lines @@
   .unittest-row-description
   {
   }
 
   ''';
 }
 
 void useHtmlEnhancedConfiguration([bool isLayoutTest = false]) {
   configure(new HtmlEnhancedConfiguration(isLayoutTest));
 }