Merge pull request #5979 from DanCorder/5058

Fix for #5058 - Exclude implemented interface functions from autocompletion suggestions.

Author: DanielRosenwasser

src/services/services.ts

@@ -3751,7 +3751,8 @@ namespace ts {
                     // Ignore omitted expressions for missing members
                     if (m.kind !== SyntaxKind.PropertyAssignment &&
                         m.kind !== SyntaxKind.ShorthandPropertyAssignment &&
-                        m.kind !== SyntaxKind.BindingElement) {
+                        m.kind !== SyntaxKind.BindingElement &&
+                        m.kind !== SyntaxKind.MethodDeclaration) {
                         continue;
                     }
 

tests/cases/fourslash/completionListImplementingInterfaceFunctions.ts

@@ -0,0 +1,21 @@
+/// <reference path="fourslash.ts" />
+
+////interface I1 {
+////    a(): void;
+////    b(): void;
+////}
+////
+////var imp1: I1 {
+////    a() {},
+////    /*0*/
+////}
+////
+////var imp2: I1 {
+////    a: () => {},
+////    /*1*/
+////}
+
+goTo.marker("0");
+verify.not.completionListContains("a");
+goTo.marker("1");
+verify.not.completionListContains("a");