Make test.dart not barf if the path to your repo has 'src/' in it.

tools/testing/dart/test_suite.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.
 
 #library("test_suite");
 
 #import("status_file_parser.dart");
 #import("test_runner.dart");
 #import("multitest.dart");
 
@@ skipping 210 matching lines @@
   }
 
   void enqueueTestCaseFromTestInformation(TestInformation info) {
     var filename = info.filename;
     var optionsFromFile = info.optionsFromFile;
     var isNegative = info.isNegative;
 
     // Look up expectations in status files using a modified file path.
     String testName;
     filename = filename.replaceAll('\\', '/');
+
+    // See if there's a 'src' directory inside the 'tests' one.
+    int testsStart = filename.lastIndexOf('tests/');
     int start = filename.lastIndexOf('src/');
-    if (start != -1) {
+    if (start > testsStart) {
       testName = filename.substring(start + 4, filename.length - 5);
     } else if (optionsFromFile['isMultitest']) {
       start = filename.lastIndexOf('/');
       int middle = filename.lastIndexOf('_');
       testName = filename.substring(start + 1, middle) + '/' +
           filename.substring(middle + 1, filename.length - 5);
     } else {
       // This case is hit by the dartc client compilation
       // tests. These tests are pretty broken compared to the
       // rest. They use the .dart suffix in the status files. They
@@ skipping 85 matching lines @@
       testGeneratorStarted();
       DoMultitest(filename,
                   TestUtils.outputDir(configuration),
                   directoryPath,
                   createTestCase,
                   testGeneratorDone);
     } else {
       createTestCase(filename, optionsFromFile['isNegative']);
     }
   }
-        
+
   void enqueueBrowserTest(String filename,
                           String testName,
                           Map optionsFromFile,
                           Set<String> expectations,
                           bool isNegative) {
     if (optionsFromFile['isMultitest']) return;
     bool isWebTest = optionsFromFile['containsDomImport'];
     bool isLibraryDefinition = optionsFromFile['isLibraryDefinition'];
     if (!isLibraryDefinition && optionsFromFile['containsSourceOrImport']) {
       print('Warning for $filename: Browser tests require #library ' +
             'in any file that uses #import or #source');
-    }      
+    }
 
     final String component = configuration['component'];
     final String testPath = new File(filename).fullPathSync();
     String dartDir = new File('.').fullPathSync();
     if (!testPath.startsWith(dartDir) ||
         dartDir.endsWith('/frog')) {
       dartDir = new File('..').fullPathSync();
       if (!testPath.startsWith(dartDir)) {
         print('Run test.dart from the dart directory or' +
               ' an immediate subdirectory only.');
@@ skipping 28 matching lines @@
         // or indirectly through test_as_library.dart, if it is not.
         String dartLibraryFilename;
         if (isLibraryDefinition) {
           dartLibraryFilename = testPath;
         } else {
           dartLibraryFilename = 'test_as_library.dart';
           File file = new File('${tempDir.path}/$dartLibraryFilename');
           RandomAccessFile dartLibrary = file.openSync(FileMode.WRITE);
           dartLibrary.writeStringSync(WrapDartTestInLibrary(testPath));
           dartLibrary.closeSync();
-        }  
-      
+        }
+
         File file = new File(dartWrapperFilename);
         RandomAccessFile dartWrapper = file.openSync(FileMode.WRITE);
         dartWrapper.writeStringSync(DartTestWrapper(
             domLibraryImport,
             '$dartDir/tests/isolate/src/TestFramework.dart',
             dartLibraryFilename));
         dartWrapper.closeSync();
       } else {
         dartWrapperFilename = testPath;
         // TODO(whesse): Once test.py is retired, adjust the relative path in
@@ skipping 62 matching lines @@
               case 'internet-explorer':
                 browserFlag = 'ie';
                 break;
               case 'safari':
                 browserFlag = 'safari';
                 break;
             }
           }
         }
         args = ['--out', htmlPath, '--browser', browserFlag];
-        
+
       } else {
         args = ['--no-timeout'];
         if (component == 'dartium') {
           var dartFlags = ['--enable_asserts',
                            '--enable_type_checks',
                            '--ignore-unrecognized-flags'];
           dartFlags.addAll(vmOptions);
           args.add('--dart-flags=${Strings.join(dartFlags, " ")}');
         }
         args.add(htmlPath);
       }
       // Create BrowserTestCase and queue it.
       var testCase = new BrowserTestCase(
           testName,
           compilerExecutable,
           compilerArgs,
           executable,
           args,
           configuration,
           completeHandler,
           expectations,
           optionsFromFile['isNegative']);
       doTest(testCase);
-    }        
+    }
   }
 
   /***
    * Create a directory for the generated test.  Drop the path to the
    * dart checkout and the final ".dart" from the test path, and replace
    * all path separators with underscores.
    * All variables are block local, except tempDir.
    */
   Directory createTemporaryDirectory(String testPath,
                                      String dartDir,
@@ skipping 20 matching lines @@
         throw new Exception(
             'Non-relative path to build directory in test_suite.dart');
       }
       buildPath.removeRange(0, 1);
       if (buildPath.last() == '') buildPath.removeLast();
       buildPath.addAll(generatedTestPath);
       generatedTestPath = buildPath;
       tempDir = new Directory(tempDirPath);
       if (!tempDir.existsSync()) {
         tempDir.createSync();
-      }      
+      }
     }
     tempDirPath = new File(tempDirPath).fullPathSync();
 
     for (String subdirectory in generatedTestPath) {
       tempDirPath = '$tempDirPath/$subdirectory';
       tempDir = new Directory(tempDirPath);
       if (!tempDir.existsSync()) {
         tempDir.createSync();
       }
     }
@@ skipping 15 matching lines @@
   }
 
   String getHtmlName(String filename) {
     return filename.replaceAll('/', '_') + configuration['component'] + '.html';
   }
 
   String get dumpRenderTreeFilename() {
     if (new Platform().operatingSystem() == 'macos') {
       return 'client/tests/drt/DumpRenderTree.app/Contents/' +
           'MacOS/DumpRenderTree';
-    } 
+    }
     return 'client/tests/drt/DumpRenderTree';
   }
 
   void testGeneratorStarted() {
     ++activeTestGenerators;
   }
 
   void testGeneratorDone() {
     --activeTestGenerators;
     if (activeTestGenerators == 0 && listingDone) {
@@ skipping 55 matching lines @@
     RegExp multiTestRegExp = const RegExp(@"/// [0-9][0-9]:(.*)");
     RegExp leadingHashRegExp = const RegExp(@"^#", multiLine: true);
     RegExp isolateStubsRegExp = const RegExp(@"// IsolateStubs=(.*)");
     RegExp domImportRegExp =
         const RegExp(@"^#import.*(dart:(dom|html)|html\.dart).*\)",
                      multiLine: true);
     RegExp libraryDefinitionRegExp =
         const RegExp(@"^#library\(", multiLine: true);
     RegExp sourceOrImportRegExp =
         const RegExp(@"^#(source|import)\(", multiLine: true);
-    
+
     // Read the entire file into a byte buffer and transform it to a
     // String. This will treat the file as ascii but the only parts
     // we are interested in will be ascii in any case.
     RandomAccessFile file = new File(filename).openSync();
     List chars = new List(file.lengthSync());
     var offset = 0;
     while (offset != chars.length) {
       offset += file.readListSync(chars, offset, chars.length - offset);
     }
     file.closeSync();
@@ skipping 125 matching lines @@
 
   bool isTestFile(String filename) => filename.endsWith("Tests.java") &&
       !filename.contains('com/google/dart/compiler/vm') &&
       !filename.contains('com/google/dart/corelib/SharedTests.java');
 
   void forEachTest(Function onTest,
                    Map testCacheIgnored,
                    [Function onDone = null]) {
     doTest = onTest;
     doDone = (onDone != null) ? onDone : (() => null);
-    
+
     if (configuration['component'] != 'dartc') {
       // Do nothing.  Asynchronously report that the suite is enqueued.
       new Timer((timerUnused){ doDone(); }, 0);
       return;
     }
     RegExp pattern = configuration['selectors']['dartc'];
     if (!pattern.hasMatch('junit_tests')) {
       new Timer((timerUnused){ doDone(); }, 0);
       return;
     }
@@ skipping 24 matching lines @@
 
     int index = filename.indexOf('compiler/javatests/com/google/dart');
     if (index != -1) {
       String testRelativePath =
           filename.substring(index + 'compiler/javatests/'.length,
                              filename.length - '.java'.length);
       String testClass = testRelativePath.replaceAll('/', '.');
       testClasses.add(testClass);
     }
   }
-      
+
   void createTest(successIgnored) {
     String d8 = '$dartDir/$buildDir/d8${TestUtils.executableSuffix}';
     List<String> args = <String>[
         '-ea',
         '-classpath', classPath,
         '-Dcom.google.dart.runner.d8=$d8',
         '-Dcom.google.dart.corelib.SharedTests.test_py=$dartDir/tools/test.py',
         'org.junit.runner.JUnitCore'];
     args.addAll(testClasses);
-          
+
     doTest(new TestCase(suiteName,
                         'java',
                         args,
                         configuration,
                         completeHandler,
                         new Set<String>.from([PASS])));
     doDone();
   }
 
   void completeHandler(TestCase testCase) {
@@ skipping 15 matching lines @@
          'third_party/hamcrest/v1_3/hamcrest-integration-1.3.0RC2.jar',
          'third_party/hamcrest/v1_3/hamcrest-library-1.3.0RC2.jar',
          'third_party/junit/v4_8_2/junit.jar'],
         ':');  // Path separator.
   }
 }
 
 
 class TestUtils {
   static String get executableSuffix() =>
-      (new Platform().operatingSystem() == 'windows') ? '.exe' : '';      
+      (new Platform().operatingSystem() == 'windows') ? '.exe' : '';
 
   static String executableName(Map configuration) {
     String postfix = executableSuffix;
     switch (configuration['component']) {
       case 'vm':
         return 'dart$postfix';
       case 'dartc':
         return 'compiler/bin/dartc_test$postfix';
       case 'frog':
       case 'leg':
@@ skipping 121 matching lines @@
  * $noCrash tests are expected to be flaky but not crash
  * $pass tests are expected to pass
  * $failOk tests are expected to fail that we won't fix
  * $fail tests are expected to fail that we should fix
  * $crash tests are expected to crash that we should fix
  * $timeout tests are allowed to timeout\
 """;
     print(report);
   }
 }