pkg/docgen: updated min dependencies of args and unittest

pkg/docgen/test/multi_library_test.dart

 library single_library_test;
 
 import 'dart:io';
 
 import 'package:path/path.dart' as path;
 import 'package:unittest/unittest.dart';
 
 import '../lib/docgen.dart';
 import '../../../sdk/lib/_internal/compiler/implementation/mirrors/mirrors_util.dart'
     as dart2js_util;
@@ skipping 73 matching lines @@
   library testLib.bar;
   import 'temp.dart';
 
   /*
    * Normal comment for class C.
    */
   class C {
   }
 ''';
 
-Directory TEMP_DIRNAME;
+Directory _tempDir;

[Alan Knight, 2014/02/19 19:00:46]

What's the point of making things private in a test?

 
-List writeLibFiles() {
-  TEMP_DIRNAME = Directory.systemTemp.createTempSync('single_library_');
-  var fileName = path.join(TEMP_DIRNAME.path, 'temp.dart');
+List<Uri> _writeLibFiles() {
+  _tempDir = Directory.systemTemp.createTempSync('single_library_');
+  var fileName = path.join(_tempDir.path, 'temp.dart');
   var file = new File(fileName);
   file.writeAsStringSync(DART_LIBRARY_1);
 
-  var fileName2 = path.join(TEMP_DIRNAME.path, 'temp2.dart');
+  var fileName2 = path.join(_tempDir.path, 'temp2.dart');
   file = new File(fileName2);
   file.writeAsStringSync(DART_LIBRARY_2);
 
-  var fileName3 = path.join(TEMP_DIRNAME.path, 'temp3.dart');
+  var fileName3 = path.join(_tempDir.path, 'temp3.dart');
   file = new File(fileName3);
   file.writeAsStringSync(DART_LIBRARY_3);
   return [new Uri.file(fileName, windows: Platform.isWindows),
           new Uri.file(fileName2, windows: Platform.isWindows),
           new Uri.file(fileName3, windows: Platform.isWindows)];
 }
 
-main() {
+void main() {
   group('Generate docs for', () {
     test('multiple libraries.', () {
-      var files = writeLibFiles();
-      getMirrorSystem(files)
-        .then(expectAsync1((mirrorSystem) {
+      var files = _writeLibFiles();
+      return getMirrorSystem(files)

[Alan Knight, 2014/02/19 19:00:46]

Is the return value actually used? An explicit return from the middle of
something that has four other functions inside it gets very confusing. Maybe
this needs to be broken up.

+        .then((mirrorSystem) {
           var testLibraryUri = files[0];
           var library = new Library(mirrorSystem.libraries[testLibraryUri]);
 
           /// Testing fixReference
           // Testing Doc comment for class [B].
           var libraryMirror = mirrorSystem.libraries[testLibraryUri];
           var classDocComment = library.fixReference('B').children.first.text;
           expect(classDocComment, 'testLib.B');
 
           // Test for linking to parameter [c]
@@ skipping 42 matching lines @@
           expect(methodParameterDocComment, 'testLib.B');
 
           methodParameterDocComment = exportedClass.fixReference(
               'testFunc').children.first.text;
           expect(methodParameterDocComment, 'testLib.testFunc');
 
         })).whenComplete(() => TEMP_DIRNAME.deleteSync(recursive: true));
     });
   });
 }