Delegating AST Visitor and Linter refactoring.

pkg/analyzer/test/generated/engine_test.dart

 // Copyright (c) 2014, 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.
 
 // This code was auto-generated, is not intended to be edited, and is subject to
 // significant change. Please see the README file for more information.
 
 library engine.engine_test;
 
 import 'dart:async';
@@ skipping 4072 matching lines @@
 library lib;
 ''');
 
     CompilationUnit unit =
         context.resolveCompilationUnit2(librarySource, librarySource);
     List<CompilationUnit> units = <CompilationUnit>[];
     units.add(unit);
 
     RecordingErrorListener errorListener = new RecordingErrorListener();
 
-    LintGeneratorTest_Verifier verifier = new LintGeneratorTest_Verifier();
+    LintGeneratorTest_Linter linter = new LintGeneratorTest_Linter();
 
     LintGenerator lintGenerator =
-        new LintGenerator(units, errorListener, [verifier]);
+        new LintGenerator(units, errorListener, [linter]);
     lintGenerator.generate();
 
-    verifier.testExpectations();
+    linter.testExpectations();
   }
+
+  void test_generate_null_visitor() {
+
+    InternalAnalysisContext context = AnalysisContextFactory.contextWithCore();
+    ChangeSet changeSet = new ChangeSet();
+    Source librarySource =
+        new FileBasedSource.con1(FileUtilities2.createFile("/test.dart"));
+    changeSet.addedSource(librarySource);
+    context.applyChanges(changeSet);
+    context.setContents(librarySource, r'''
+library lib;
+''');
+
+    CompilationUnit unit =
+        context.resolveCompilationUnit2(librarySource, librarySource);
+    List<CompilationUnit> units = <CompilationUnit>[];
+    units.add(unit);
+
+    RecordingErrorListener errorListener = new RecordingErrorListener();
+
+    Linter badLinter = new LintGeneratorTest_Linter_Null_Visitor();
+    LintGeneratorTest_Linter goodLinter = new LintGeneratorTest_Linter();
+
+    LintGenerator lintGenerator =
+        new LintGenerator(units, errorListener, [badLinter, goodLinter]);
+    // Test that generate does not fall down with a null visitor
+    lintGenerator.generate();
+    // Well-formed linter should still get called
+    goodLinter.testExpectations();
+  }
+
 }
 
 
-class LintGeneratorTest_Verifier extends LintVerifier {
+class LintGeneratorTest_Linter extends Linter with SimpleAstVisitor<Object> {

[Brian Wilkerson, 2015/01/15 14:59:54]

Note that the original class name was only chosen because in Java the class
Verifier was an inner class. (That's the only time you'll find underscores in
type names.) Up to you whether to change it.

[pquitslund, 2015/01/15 18:07:30]

Thanks.  Were this Java, it would be an inner class so I'll stick with it for
now.

 
   bool visited;
 
   @override
   Object visitCompilationUnit(CompilationUnit node) {
     visited = true;
     return null;
   }
 
   testExpectations() {
     expect(reporter, isNotNull);
     expect(visited, isTrue);
   }
+
+  @override
+  AstVisitor getVisitor() => this;
+}
+
+class LintGeneratorTest_Linter_Null_Visitor extends Linter {
+  @override
+  AstVisitor getVisitor() => null;
 }
 
 
 @reflectiveTest
 class GetContentTaskTest extends EngineTestCase {
   void test_accept() {
     Source source = new TestSource('/test.dart', '');
     GetContentTask task = new GetContentTask(null, source);
     expect(task.accept(new GetContentTaskTestTV_accept()), isTrue);
   }
@@ skipping 2910 matching lines @@
   bool contains(Source source) => source == libB;
 }
 
 
 class _UniversalCachePartitionTest_test_setMaxCacheSize implements
     CacheRetentionPolicy {
   @override
   RetentionPriority getAstPriority(Source source, SourceEntry sourceEntry) =>
       RetentionPriority.LOW;
 }