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');
+#library('dartdoc_test');
 
-#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

[Jennifer Messerly, 2012/01/11 19:15:10]

Up to you, but one thought I had here: instead of commenting it out, you could
change the test to catch the error and print a diagnostic that it was "run from
the wrong path". Ensures the code still compiles & will run correctly if it is
started from the right path. (I've been admonished in the past by test folks
that disabling tests via commenting out is bad, because the code bit rots, and
then never get resurrected because it's too broken.)

The "console" and "process" node.js tests have an interesting "solution" ...
they look for the file in a few different relative paths.

also we should have a bug filed about the lack of a way to get the script's path
on the VM.

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

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

+  // dummy.dart is sensitive to the location that dart was invoked from and
+  // 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>');
     });
   });
+  */
 }