adjust location of URI helper files for spawn_uri tests in the browser.

tools/testing/dart/test_suite.dart

Index: tools/testing/dart/test_suite.dart
diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart
index 6381a12bb2d8e253e0b8085b3d622f889204636b..5f7e23b408f02df831c4dee2b4947e0ff7abd22c 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -920,19 +920,6 @@ class StandardTestSuite extends TestSuite {
         _createWrapperFile(dartWrapperFilename, filePath);
       } else {
         dartWrapperFilename = filename;
-        // TODO(whesse): Once test.py is retired, adjust the relative path in
-        // the client/samples/dartcombat test to its css file, remove the
-        // "../../" from this path, and move this out of the isWebTest guard.
-        // Also remove getHtmlName, and just use test.html.
-        // TODO(efortuna): this shortening of htmlFilename is a band-aid until
-        // the above TODO gets fixed. Windows cannot have paths that are longer
-        // than 260 characters, and without this hack, we were running past the
-        // the limit.
-        String htmlFilename = getHtmlName(filename);

[kustermann, 2013/04/15 08:02:42]

the 'getHtmlName' function is no longer used.

[Emily Fortuna, 2013/04/16 18:40:16]

Done.

-        while ('$tempDir/../$htmlFilename'.length >= 260) {
-          htmlFilename = htmlFilename.substring(htmlFilename.length~/2);
-        }
-        htmlPath = '$tempDir/../$htmlFilename';
       }
       String scriptPath = (compiler == 'none') ?
           dartWrapperFilename : compiledDartWrapperFilename;
@@ -966,18 +953,27 @@ class StandardTestSuite extends TestSuite {
         commands.add(_compileCommand(
             dartWrapperFilename, compiledDartWrapperFilename,
             compiler, tempDir, vmOptions, optionsFromFile));
+      }
 
-        // some tests require compiling multiple input scripts.
-        List<String> otherScripts = optionsFromFile['otherScripts'];
-        for (String name in otherScripts) {
-          Path namePath = new Path(name);
+      // some tests require compiling multiple input scripts.
+      List<String> otherScripts = optionsFromFile['otherScripts'];

[ahe, 2013/04/15 18:17:45]

I don't understand why this is specific to browser tests.

[Emily Fortuna, 2013/04/16 18:40:16]

This is specific to browser tests because they are not run from the test
directory. The browser tests create an  html file in the output directory
(generated_tests/...) and copy any relevant files (js or dart) into the output
directory. In comparison, the vm tests just run from the same location that they
were originally written.

[ahe, 2013/07/24 09:31:40]

Sorry for the late reply.

I still don't understand why this feature is specific to browser tests.  There
are other tests that get copied, in particular, multi-tests.

I think we need to make general features and avoid special casing browser tests.
 The only thing special about a browser test should be that it is running in a
browser and that need to generate an HTML file.  Everything else should be
generalized.

+      for (String name in otherScripts) {
+        Path namePath = new Path(name);
+        String baseName = namePath.filenameWithoutExtension;
+        Path fromPath = filePath.directoryPath.join(namePath);
+        if (compiler != 'none') {
           assert(namePath.extension == 'dart');
-          String baseName = namePath.filenameWithoutExtension;
-          Path fromPath = filePath.directoryPath.join(namePath);
           commands.add(_compileCommand(
               fromPath.toNativePath(), '$tempDir/$baseName.js',
               compiler, tempDir, vmOptions, optionsFromFile));
         }
+        if (compiler == 'none') {
+          // For the tests that require multiple input scripts but are not
+          // compiled, move the input scripts over with the script so they can
+          // be accessed.
+          String result = new File.fromPath(fromPath).readAsStringSync();
+          new File('$tempDir/$baseName.dart').writeAsStringSync(result);
+        }
       }
 
       // Variables for browser multi-tests.