Implement AstBuilder support for library, part, and part-of declarations.

pkg/analyzer/test/generated/parser_fasta_test.dart

Index: pkg/analyzer/test/generated/parser_fasta_test.dart
diff --git a/pkg/analyzer/test/generated/parser_fasta_test.dart b/pkg/analyzer/test/generated/parser_fasta_test.dart
index edb0a67d863547972e0a9ae167264327a1050d32..988fafa756b05688419e0637e73696f284d6ac98 100644
--- a/pkg/analyzer/test/generated/parser_fasta_test.dart
+++ b/pkg/analyzer/test/generated/parser_fasta_test.dart
@@ -583,20 +583,6 @@ class TopLevelParserTest_Fasta extends FastaParserTestCase
 
   @override
   @failingTest
-  void test_parseCompilationUnit_directives_multiple() {
-    // TODO(paulberry): Unhandled event: LibraryName
-    super.test_parseCompilationUnit_directives_multiple();
-  }
-
-  @override
-  @failingTest
-  void test_parseCompilationUnit_directives_single() {
-    // TODO(paulberry): Unhandled event: LibraryName
-    super.test_parseCompilationUnit_directives_single();
-  }
-
-  @override
-  @failingTest
   void test_parseCompilationUnit_empty() {
     // TODO(paulberry): No objects placed on stack
     super.test_parseCompilationUnit_empty();
@@ -810,29 +796,8 @@ class TopLevelParserTest_Fasta extends FastaParserTestCase
 
   @override
   @failingTest
-  void test_parseDirective_library() {
-    // TODO(paulberry): Unhandled event: LibraryName
-    super.test_parseDirective_library();
-  }
-
-  @override
-  @failingTest
-  void test_parseDirective_part() {
-    // TODO(paulberry): Unhandled event: Part
-    super.test_parseDirective_part();
-  }
-
-  @override
-  @failingTest
-  void test_parseDirective_partOf() {
-    // TODO(paulberry): Unhandled event: PartOf
-    super.test_parseDirective_partOf();
-  }
-
-  @override
-  @failingTest
   void test_parseDirectives_complete() {
-    // TODO(paulberry): Unhandled event: LibraryName
+    // TODO(paulberry,ahe): Fasta doesn't support script tags yet.

[ahe, 2017/02/20 09:59:39]

Can I assume this is low priority?

[Paul Berry, 2017/02/20 14:48:31]

Agreed, It's pretty low priority IMO.

     super.test_parseDirectives_complete();
   }
 
@@ -846,19 +811,15 @@ class TopLevelParserTest_Fasta extends FastaParserTestCase
   @override
   @failingTest
   void test_parseDirectives_mixed() {
-    // TODO(paulberry): Unhandled event: LibraryName
+    // TODO(paulberry,ahe): This test verifies the analyzer parser's ability to
+    // stop parsing as soon as the first non-directive is encountered; this is
+    // useful for quickly traversing an import graph.  Consider adding a similar
+    // ability to Fasta's parser.

[ahe, 2017/02/20 09:59:39]

I think that should be easy. Let me know how soon you need this.

[Paul Berry, 2017/02/20 14:48:31]

It would be nice to get this within the next couple of weeks.  I have some code
in pkg/front_end/lib/src/dependency_grapher_impl.dart that I'd like to switch
over to using Fasta, and it would likely take a performance hit without this
feature.

     super.test_parseDirectives_mixed();
   }
 
   @override
   @failingTest
-  void test_parseDirectives_multiple() {
-    // TODO(paulberry): Unhandled event: LibraryName
-    super.test_parseDirectives_multiple();
-  }
-
-  @override
-  @failingTest
   void test_parseDirectives_script() {
     // TODO(paulberry): No objects placed on stack
     super.test_parseDirectives_script();
@@ -866,13 +827,6 @@ class TopLevelParserTest_Fasta extends FastaParserTestCase
 
   @override
   @failingTest
-  void test_parseDirectives_single() {
-    // TODO(paulberry): Unhandled event: LibraryName
-    super.test_parseDirectives_single();
-  }
-
-  @override
-  @failingTest
   void test_parseEnumDeclaration_one() {
     // TODO(paulberry): Unhandled event: Enum
     super.test_parseEnumDeclaration_one();
@@ -958,22 +912,11 @@ class TopLevelParserTest_Fasta extends FastaParserTestCase
 
   @override
   @failingTest
-  void test_parseLibraryDirective() {
-    // TODO(paulberry): Unhandled event: LibraryName
-    super.test_parseLibraryDirective();
-  }
-
-  @override
-  @failingTest
-  void test_parsePartDirective() {
-    // TODO(paulberry): Unhandled event: Part
-    super.test_parsePartDirective();
-  }
-
-  @override
-  @failingTest
   void test_parsePartOfDirective_name() {
-    // TODO(paulberry): Unhandled event: PartOf
+    // TODO(paulberry,ahe): Thes test verifies that even if URIs in "part of"
+    // declarations are enabled, a construct of the form "part of identifier;"
+    // is still properly handled.  URIs in "part of" declarations are not
+    // supported by Fasta yet.

[ahe, 2017/02/20 09:59:39]

Yeah. I think this is legacy syntax, and I generally try to keep support for old
syntax for improved error recovery.

     super.test_parsePartOfDirective_name();
   }