pkg/(unit)test: Enable group names containing spaces in HtmlIndividualConfiguration

lib/html_individual_config.dart

 // Copyright (c) 2013, 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 unit test library for running groups of tests in a browser, instead of the
 /// entire test file. This is especially used for large tests files that have
 /// many subtests, so we can mark groups as failing at a finer granularity than
 /// the entire test file.
 ///
 /// To use, import this file, and call [useHtmlIndividualConfiguration] at the
@@ skipping 16 matching lines @@
           .split('&')
           .where((p) => p.startsWith('group='))
           .toList();
 
       if (!groups.isEmpty) {
         if (groups.length > 1) {
           throw new ArgumentError('More than one "group" parameter provided.');
         }
 
         var testGroupName = groups.single.split('=')[1];
+        testGroupName =
+            testGroupName.replaceAll('+', ' ').replaceAll('%20', ' ');

[nweiz, 2015/05/07 01:41:53]

We should use the Uri class to parse this rather than doing it by hand.

[kevmoo, 2015/05/07 22:39:45]

Done.

         var startsWith = "$testGroupName${unittest.groupSep}";
         unittest.filterTests(
             (unittest.TestCase tc) => tc.description.startsWith(startsWith));
       }
     }
     super.onStart();
   }
 }
 
 void useHtmlIndividualConfiguration([bool isLayoutTest = false]) {
   unittest.unittestConfiguration =
       isLayoutTest ? _singletonLayout : _singletonNotLayout;
 }
 
 final _singletonLayout = new HtmlIndividualConfiguration(true);
 final _singletonNotLayout = new HtmlIndividualConfiguration(false);