fix lint and api check

Author: mmalerba

src/cdk/testing/tests/protractor.e2e.spec.ts

@@ -12,7 +12,8 @@ import {SubComponentHarness, SubComponentSpecialHarness} from './harnesses/sub-c
 // Kagekiri is available globally in the browser. We declare it here so we can use it in the
 // browser-side script passed to `by.js`.
 // TODO(mmalerba): Replace with type-only import once TS 3.8 is available
-// https://devblogs.microsoft.com/typescript/announcing-typescript-3-8-beta/#type-only-imports-exports
+// tslint:disable-next-line:max-line-length
+// See: https://devblogs.microsoft.com/typescript/announcing-typescript-3-8-beta/#type-only-imports-exports
 declare const kagekiri: {
   querySelectorAll: (selector: string, root: Element) => NodeListOf<Element>;
 };

tools/public_api_guard/cdk/testing/protractor.d.ts

@@ -19,12 +19,16 @@ export declare class ProtractorElement implements TestElement {
 }
 
 export declare class ProtractorHarnessEnvironment extends HarnessEnvironment<ElementFinder> {
-    protected constructor(rawRootElement: ElementFinder, _queryFn?: (selector: string, root: ElementFinder) => ElementArrayFinder);
+    protected constructor(rawRootElement: ElementFinder, options?: ProtractorHarnessEnvironmentOptions);
     protected createEnvironment(element: ElementFinder): HarnessEnvironment<ElementFinder>;
     protected createTestElement(element: ElementFinder): TestElement;
     forceStabilize(): Promise<void>;
     protected getAllRawElements(selector: string): Promise<ElementFinder[]>;
     protected getDocumentRoot(): ElementFinder;
     waitForTasksOutsideAngular(): Promise<void>;
-    static loader(queryFn?: (selector: string, root: ElementFinder) => ElementArrayFinder): HarnessLoader;
+    static loader(options?: ProtractorHarnessEnvironmentOptions): HarnessLoader;
+}
+
+export interface ProtractorHarnessEnvironmentOptions {
+    queryFn: (selector: string, root: ElementFinder) => ElementArrayFinder;
 }

tools/public_api_guard/cdk/testing/testbed.d.ts

@@ -1,14 +1,18 @@
 export declare class TestbedHarnessEnvironment extends HarnessEnvironment<Element> {
-    protected constructor(rawRootElement: Element, _fixture: ComponentFixture<unknown>, _queryFn?: (selector: string, root: Element) => ArrayLike<Element>);
+    protected constructor(rawRootElement: Element, _fixture: ComponentFixture<unknown>, options?: TestbedHarnessEnvironmentOptions);
     protected createEnvironment(element: Element): HarnessEnvironment<Element>;
     protected createTestElement(element: Element): TestElement;
     forceStabilize(): Promise<void>;
     protected getAllRawElements(selector: string): Promise<Element[]>;
     protected getDocumentRoot(): Element;
     waitForTasksOutsideAngular(): Promise<void>;
-    static documentRootLoader(fixture: ComponentFixture<unknown>, queryFn?: (selector: string, root: Element) => ArrayLike<Element>): HarnessLoader;
-    static harnessForFixture<T extends ComponentHarness>(fixture: ComponentFixture<unknown>, harnessType: ComponentHarnessConstructor<T>, queryFn?: (selector: string, root: Element) => ArrayLike<Element>): Promise<T>;
-    static loader(fixture: ComponentFixture<unknown>, queryFn?: (selector: string, root: Element) => ArrayLike<Element>): HarnessLoader;
+    static documentRootLoader(fixture: ComponentFixture<unknown>, options?: TestbedHarnessEnvironmentOptions): HarnessLoader;
+    static harnessForFixture<T extends ComponentHarness>(fixture: ComponentFixture<unknown>, harnessType: ComponentHarnessConstructor<T>, options?: TestbedHarnessEnvironmentOptions): Promise<T>;
+    static loader(fixture: ComponentFixture<unknown>, options?: TestbedHarnessEnvironmentOptions): HarnessLoader;
+}
+
+export interface TestbedHarnessEnvironmentOptions {
+    queryFn: (selector: string, root: Element) => ArrayLike<Element>;
 }
 
 export declare class UnitTestElement implements TestElement {