Hook up the dartdoc tests to test.py, test.dart, and frog/presubmit.py.

utils/tests/dartdoc/src/dartdoc_tests.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.
 
 /// Unit tests for dartdoc.
 #library('dartdoc_tests');
 
-#import('../dartdoc.dart');
-#import('../markdown.dart', prefix: 'md');
+#import('../../../dartdoc/dartdoc.dart');
+#import('../../../dartdoc/markdown.dart', prefix: 'md');
 
 // TODO(rnystrom): Better path to unittest.
-#import('../../../client/testing/unittest/unittest_node.dart');
-#import('../../../frog/lang.dart');
-#import('../../../frog/file_system_node.dart');
+#import('../../../../client/testing/unittest/unittest_node.dart');
+#import('../../../../frog/lang.dart');
+#import('../../../../frog/file_system_node.dart');
 
 main() {
   var files = new NodeFileSystem();
   parseOptions('../../frog', [], files);
   initializeWorld(files);
 
   group('countOccurrences', () {
     test('empty text returns 0', () {
       expect(countOccurrences('', 'needle')).equals(0);
     });
@@ skipping 85 matching lines @@
           '../../other/sub/file.html');
     });
 
     test('from nested to directory with different path', () {
       startFile('dir/sub/file.html');
       expect(relativePath('other/file.html')).equals(
           '../../other/file.html');
     });
   });
 
+  // TODO(rnystrom): Disabling these for now. The problem is that loading
+  // dummy.dart is sensitive to the location that dart was invoked from and

[Bill Hesse, 2012/01/11 21:01:13]

Can we add a bug reference here, to a bug that says that we will get the client
directory and the current directory in test.py, when we implement getting the
script path in the script arguments, and to fix it then.  I'll file the bug now.

[Bob Nystrom, 2012/01/11 21:28:47]

Done and filed already: https://code.google.com/p/dart/issues/detail?id=1145

:)

+  // not relative to *this* file like we'd like. That means these tests only
+  // run correctly from one place. Unfortunately, test.py/test.dart runs this
+  // from one directory and frog/presubmit.py runs it from another.
+  /*
   group('name reference', () {
     var doc = new Dartdoc();
-    doc.document('test/dummy.dart');
-    var dummy = world.libraries['test/dummy.dart'];
+    // TODO(rnystrom): The paths here are a bit strange. They're relative to
+    // where test.dart happens to be invoked from.
+    world.processDartScript('utils/tests/dartdoc/src/dummy.dart');
+    world.resolveAll();
+    var dummy = world.libraries['utils/tests/dartdoc/src/dummy.dart'];
     var klass = dummy.findTypeByName('Class');
     var method = klass.getMember('method');
 
     String render(md.Node node) => md.renderToHtml([node]);
 
     test('to a parameter of the current method', () {
       expect(render(doc.resolveNameReference('param', member: method))).
         equals('<span class="param">param</span>');
     });
 
     test('to a member of the current type', () {
       expect(render(doc.resolveNameReference('method', type: klass))).
-        equals('<a href="../../dummy/Class.html#method" class="crossref">' +
+        equals('<a class="crossref" href="../../dummy/Class.html#method">' +
             'method</a>');
     });
 
     test('to a property with only a getter links to the getter', () {
       expect(render(doc.resolveNameReference('getterOnly', type: klass))).
-        equals('<a href="../../dummy/Class.html#get:getterOnly" ' +
-            'class="crossref">getterOnly</a>');
+        equals('<a class="crossref" ' +
+            'href="../../dummy/Class.html#get:getterOnly">getterOnly</a>');
     });
 
     test('to a property with only a setter links to the setter', () {
       expect(render(doc.resolveNameReference('setterOnly', type: klass))).
-        equals('<a href="../../dummy/Class.html#set:setterOnly" ' +
-            'class="crossref">setterOnly</a>');
+        equals('<a class="crossref" ' +
+            'href="../../dummy/Class.html#set:setterOnly">setterOnly</a>');
     });
 
     test('to a property with a getter and setter links to the getter', () {
       expect(render(doc.resolveNameReference('getterAndSetter', type: klass))).
-        equals('<a href="../../dummy/Class.html#get:getterAndSetter" ' +
-            'class="crossref">getterAndSetter</a>');
+        equals('<a class="crossref" ' +
+            'href="../../dummy/Class.html#get:getterAndSetter">' +
+            'getterAndSetter</a>');
     });
 
     test('to a type in the current library', () {
       expect(render(doc.resolveNameReference('Class', library: dummy))).
-        equals('<a href="../../dummy/Class.html" class="crossref">Class</a>');
+        equals('<a class="crossref" href="../../dummy/Class.html">Class</a>');
     });
 
     test('to a top-level member in the current library', () {
       expect(render(doc.resolveNameReference('topLevelMethod',
                   library: dummy))).
-        equals('<a href="../../dummy.html#topLevelMethod" class="crossref">' +
+        equals('<a class="crossref" href="../../dummy.html#topLevelMethod">' +
             'topLevelMethod</a>');
     });
 
     test('to an unknown name', () {
       expect(render(doc.resolveNameReference('unknownName', library: dummy,
                   type: klass, member: method))).
         equals('<code>unknownName</code>');
     });
   });
+  */
 }