Issue 22381. Fix for navigating to constructor from redirecting factory constructor.

pkg/analysis_server/test/analysis/notification_navigation_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.
 
 library test.analysis.notification.navigation;
 
 import 'dart:async';
 
 import 'package:analysis_server/src/constants.dart';
 import 'package:analysis_server/src/protocol.dart';
@@ skipping 237 matching lines @@
 ''');
     return prepareNavigation().then((_) {
       // has region for complete "A.named"
       assertHasRegion("A(BBB");
       assertHasTarget("A(BBB", 0);
       // validate that we don't forget to resolve parameters
       assertHasRegionTarget('BBB p', 'BBB {}');
     });
   }
 
+  test_factoryRedirectingConstructor_implicit() {
+    addTestFile('''
+class A {
+  factory A() = B;
+}
+class B {
+}
+''');
+    return prepareNavigation().then((_) {
+      assertHasRegion('B;');
+      assertHasTarget('B {');
+    });
+  }
+
+  test_factoryRedirectingConstructor_implicit_withTypeArgument() {
+    addTestFile('''
+class A {}
+class B {
+  factory B() = C<A>;
+}
+class C<T> {}
+''');
+    return prepareNavigation().then((_) {
+      {
+        assertHasRegion('C<A>');
+        assertHasTarget('C<T> {');
+      }
+      {
+        assertHasRegion('A>;');
+        assertHasTarget('A {');
+      }
+    });
+  }
+
+  test_factoryRedirectingConstructor_named() {
+    addTestFile('''
+class A {
+  factory A() = B.named;
+}
+class B {
+  B.named();
+}
+''');
+    return prepareNavigation().then((_) {
+      assertHasRegionString('B.named');
+      assertHasTarget('named();');
+    });
+  }
+
+  test_factoryRedirectingConstructor_named_withTypeArgument() {
+    addTestFile('''
+class A {}
+class B {
+  factory B.named() = C<A>.named;
+}
+class C<T> {
+  C.named() {}
+}
+''');
+    return prepareNavigation().then((_) {
+      {
+        assertHasRegion('C<A>');
+        assertHasTarget('named() {}');
+      }
+      {
+        assertHasRegion('A>.named');
+        assertHasTarget('A {');
+      }
+      {
+        assertHasRegion('.named;', '.named'.length);
+        assertHasTarget('named() {}');
+      }
+    });
+  }
+
+  test_factoryRedirectingConstructor_unnamed() {
+    addTestFile('''
+class A {
+  factory A() = B;
+}
+class B {
+  B() {}
+}
+''');
+    return prepareNavigation().then((_) {
+      assertHasRegion('B;');
+      assertHasTarget('B() {}', 0);
+    });
+  }
+
+  test_factoryRedirectingConstructor_unnamed_withTypeArgument() {
+    addTestFile('''
+class A {}
+class B {
+  factory B() = C<A>;
+}
+class C<T> {
+  C() {}
+}
+''');
+    return prepareNavigation().then((_) {
+      {
+        assertHasRegion('C<A>');
+        assertHasTarget('C() {}', 0);
+      }
+      {
+        assertHasRegion('A>;');
+        assertHasTarget('A {');
+      }
+    });
+  }
+
+  test_factoryRedirectingConstructor_unresolved() {
+    addTestFile('''
+class A {
+  factory A() = B;
+}
+''');
+    return prepareNavigation().then((_) {
+      // don't check regions, but there should be no exceptions
+    });
+  }
+
   test_fieldFormalParameter() {
     addTestFile('''
 class AAA {
   int fff = 123;
   AAA(this.fff);
 }
 ''');
     return prepareNavigation().then((_) {
       assertHasRegionTarget('fff);', 'fff = 123');
     });
@@ skipping 95 matching lines @@
     return prepareNavigation().then((_) {
       assertHasRegionString('new A.named');
       assertHasTarget('named() {}');
     });
   }
 
   test_instanceCreation_named_withTypeArgument() {
     addTestFile('''
 class A {}
 class B<T> {
-  A.named() {}
+  B.named() {}
 }
 main() {
   new B<A>.named();
 }
 ''');
     return prepareNavigation().then((_) {
       {
         assertHasRegionString('new B');
         assertHasTarget('named() {}');
       }
@@ skipping 263 matching lines @@
   test_type_void() {
     addTestFile('''
 void main() {
 }
 ''');
     return prepareNavigation().then((_) {
       assertNoRegionAt('void');
     });
   }
 }