diff --git a/packages/language-service/src/typescript_symbols.ts b/packages/language-service/src/typescript_symbols.ts index 6635ee5a18..3e9401a1d1 100644 --- a/packages/language-service/src/typescript_symbols.ts +++ b/packages/language-service/src/typescript_symbols.ts @@ -271,7 +271,11 @@ class TypeWrapper implements Symbol { } members(): SymbolTable { - return new SymbolTableWrapper(this.tsType.getProperties(), this.context); + // Should call getApparentProperties() instead of getProperties() because + // the former includes properties on the base class whereas the latter does + // not. This provides properties like .bind(), .call(), .apply(), etc for + // functions. + return new SymbolTableWrapper(this.tsType.getApparentProperties(), this.context); } signatures(): Signature[] { return signaturesOf(this.tsType, this.context); } diff --git a/packages/language-service/test/diagnostics_spec.ts b/packages/language-service/test/diagnostics_spec.ts index edd0ec103a..4523ba1fe5 100644 --- a/packages/language-service/test/diagnostics_spec.ts +++ b/packages/language-service/test/diagnostics_spec.ts @@ -119,6 +119,14 @@ describe('diagnostics', () => { expect(diagnostics).toEqual([]); }); + it('should not produce errors on function.bind()', () => { + mockHost.override(TEST_TEMPLATE, ` + + `); + const diags = ngLS.getDiagnostics(TEST_TEMPLATE); + expect(diags).toEqual([]); + }); + describe('in expression-cases.ts', () => { it('should report access to an unknown field', () => { const diags = ngLS.getDiagnostics(EXPRESSION_CASES).map(d => d.messageText);