improve recovery of invalid top level abstract modifier usage

pkg/analyzer/test/generated/parser_fasta_test.dart

 // Copyright (c) 2017, 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.
 
 import 'package:analyzer/dart/ast/ast.dart';
 import 'package:analyzer/dart/ast/token.dart' as analyzer;
 import 'package:analyzer/dart/ast/token.dart' show TokenType;
 import 'package:analyzer/error/error.dart';
 import 'package:analyzer/error/listener.dart' show ErrorReporter;
 import 'package:analyzer/src/dart/scanner/scanner.dart';
@@ skipping 123 matching lines @@
 }
 
 /**
  * Tests of the fasta parser based on [ErrorParserTest].
  */
 @reflectiveTest
 class ErrorParserTest_Fasta extends FastaParserTestCase
     with ErrorParserTestMixin {
   @override
   @failingTest
-  void test_abstractEnum() {
-    super.test_abstractEnum();
-  }
-
-  @override
-  @failingTest
-  void test_abstractTypeDef() {
-    super.test_abstractTypeDef();
-  }
-
-  @override
-  @failingTest
   void test_annotationOnEnumConstant_first() {
     super.test_annotationOnEnumConstant_first();
   }
 
   @override
   @failingTest
   void test_annotationOnEnumConstant_middle() {
     super.test_annotationOnEnumConstant_middle();
   }
 
@@ skipping 2400 matching lines @@
     createParser(source);
     Object result = _parserProxy._run(getParseFunction);
     assertErrorsWithCodes(errorCodes);
     return result;
   }
 
   List<ErrorCode> _toFastaGeneratedAnalyzerErrorCodes(
           List<ErrorCode> expectedErrorCodes) =>
       expectedErrorCodes.map((code) {
         if (code == ParserErrorCode.ABSTRACT_CLASS_MEMBER ||
+            code == ParserErrorCode.ABSTRACT_ENUM ||
             code == ParserErrorCode.ABSTRACT_TOP_LEVEL_FUNCTION ||
-            code == ParserErrorCode.ABSTRACT_TOP_LEVEL_VARIABLE)
+            code == ParserErrorCode.ABSTRACT_TOP_LEVEL_VARIABLE ||
+            code == ParserErrorCode.ABSTRACT_TYPEDEF)
           return ParserErrorCode.EXTRANEOUS_MODIFIER;
         return code;
       }).toList();
 }
 
 /**
  * Tests of the fasta parser based on [FormalParameterParserTestMixin].
  */
 @reflectiveTest
 class FormalParameterParserTest_Fasta extends FastaParserTestCase
@@ skipping 2018 matching lines @@
     test_parseClassDeclaration_native_missing_literal();
   }
 
   void test_parseClassDeclaration_native_not_allowed() {
     allowNativeClause = false;
     test_parseClassDeclaration_native();
   }
 
   @override
   @failingTest
+  void test_parseCompilationUnit_abstractAsPrefix_parameterized() {
+    // TODO(danrubel): built-in "abstract" cannot be used as a type
+    super.test_parseCompilationUnit_abstractAsPrefix_parameterized();
+  }
+
+  @override
+  @failingTest
   void test_parseCompilationUnit_builtIn_asFunctionName() {
     // TODO(paulberry,ahe): Fasta's parser is confused when one of the built-in
     // identifiers `export`, `import`, `library`, `part`, or `typedef` appears
     // as the name of a top level function with an implicit return type.
     super.test_parseCompilationUnit_builtIn_asFunctionName();
   }
 
   @override
   @failingTest
   void test_parseCompilationUnit_exportAsPrefix() {
@@ skipping 15 matching lines @@
     assertNoErrors();
   }
 
   @failingTest
   void test_parseCompilationUnit_typedefAsPrefix2() {
     // TODO(danrubel): should not be generating an error
     super.test_parseCompilationUnit_typedefAsPrefix();
     assertNoErrors();
   }
 
+  @override
+  @failingTest
+  void test_parseCompilationUnitMember_abstractAsPrefix() {
+    // TODO(danrubel): built-in "abstract" cannot be used as a prefix
+    super.test_parseCompilationUnitMember_abstractAsPrefix();
+  }
+
   @failingTest
   void test_parseCompilationUnitMember_abstractAsPrefix2() {
     // TODO(danrubel): should not be generating an error
     super.test_parseCompilationUnitMember_abstractAsPrefix();
     assertNoErrors();
   }
 
   @override
   @failingTest
   void test_parseDirectives_mixed() {
     // 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.
     super.test_parseDirectives_mixed();
   }
 }