From f6bf8928f208c0363588c09d38c0025a5b3f76d0 Mon Sep 17 00:00:00 2001 From: cexbrayat Date: Thu, 16 May 2019 17:39:07 +0200 Subject: [PATCH] fix(core): temporarily remove @deprecated jsdoc tag for a TextBed.get overload (#30514) PR #29290 introduced a new `TestBed.get` signature and deprecated the existing one. This raises a lot of TSLint deprecation warnings in projects using a strict TS config (see #29905 for context), so we are temporarily removing the `@deprecated` annotation in favor of a plain text warning until we properly fix it. Refs #29905 Fixes FW-1336 PR Close #30514 --- packages/core/testing/src/test_bed.ts | 8 +++++++- tools/public_api_guard/core/testing.d.ts | 2 +- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/packages/core/testing/src/test_bed.ts b/packages/core/testing/src/test_bed.ts index f1653b4b6e..6fe51195c9 100644 --- a/packages/core/testing/src/test_bed.ts +++ b/packages/core/testing/src/test_bed.ts @@ -57,8 +57,14 @@ export interface TestBed { compileComponents(): Promise; get(token: Type|InjectionToken, notFoundValue?: T, flags?: InjectFlags): any; + + // TODO: switch back to official deprecation marker once TSLint issue is resolved + // https://github.com/palantir/tslint/issues/4522 /** - * @deprecated from v8.0.0 use Type or InjectionToken + * deprecated from v8.0.0 use Type or InjectionToken + * This does not use the deprecated jsdoc tag on purpose + * because it renders all overloads as deprecated in TSLint + * due to https://github.com/palantir/tslint/issues/4522. */ get(token: any, notFoundValue?: any): any; diff --git a/tools/public_api_guard/core/testing.d.ts b/tools/public_api_guard/core/testing.d.ts index 5c10def5a9..11e0cb85b5 100644 --- a/tools/public_api_guard/core/testing.d.ts +++ b/tools/public_api_guard/core/testing.d.ts @@ -59,7 +59,7 @@ export interface TestBed { createComponent(component: Type): ComponentFixture; execute(tokens: any[], fn: Function, context?: any): any; get(token: Type | InjectionToken, notFoundValue?: T, flags?: InjectFlags): any; - /** @deprecated */ get(token: any, notFoundValue?: any): any; + get(token: any, notFoundValue?: any): any; initTestEnvironment(ngModule: Type | Type[], platform: PlatformRef, aotSummaries?: () => any[]): void; overrideComponent(component: Type, override: MetadataOverride): void; overrideDirective(directive: Type, override: MetadataOverride): void;