refactor(material/legacy-list): rename ts symbols

wagnermaciel · wagnermaciel · commit 640a65ba861a · 2022-09-03T15:35:26.000Z
diff --git a/src/material/legacy-list/public-api.ts b/src/material/legacy-list/public-api.ts
@@ -6,8 +6,24 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-export * from './list-module';
-export * from './list';
-export * from './selection-list';
+export {MatLegacyListModule} from './list-module';
+export {
+  MatLegacyNavList,
+  MatLegacyList,
+  MatLegacyListAvatarCssMatStyler,
+  MatLegacyListIconCssMatStyler,
+  MatLegacyListSubheaderCssMatStyler,
+  MatLegacyListItem,
+} from './list';
+export {
+  MAT_LEGACY_SELECTION_LIST_VALUE_ACCESSOR,
+  MatLegacySelectionListChange,
+  MatLegacyListOptionCheckboxPosition,
+  MatLegacyListOption,
+  MatLegacySelectionList,
+} from './selection-list';
 
-export {MAT_LIST, MAT_NAV_LIST} from '@angular/material/list';
+export {
+  MAT_LIST as MAT_LEGACY_LIST,
+  MAT_NAV_LIST as MAT_LEGACY_NAV_LIST,
+} from '@angular/material/list';
diff --git a/src/material/legacy-list/selection-list.ts b/src/material/legacy-list/selection-list.ts
@@ -47,7 +47,7 @@ const _MatSelectionListBase = mixinDisableRipple(class {});
 const _MatListOptionBase = mixinDisableRipple(class {});
 
 /** @docs-private */
-export const MAT_SELECTION_LIST_VALUE_ACCESSOR: any = {
+export const MAT_LEGACY_SELECTION_LIST_VALUE_ACCESSOR: any = {
   provide: NG_VALUE_ACCESSOR,
   useExisting: forwardRef(() => MatLegacySelectionList),
   multi: true,
@@ -341,7 +341,7 @@ export class MatLegacyListOption
   template: '<ng-content></ng-content>',
   styleUrls: ['list.css'],
   encapsulation: ViewEncapsulation.None,
-  providers: [MAT_SELECTION_LIST_VALUE_ACCESSOR],
+  providers: [MAT_LEGACY_SELECTION_LIST_VALUE_ACCESSOR],
   changeDetection: ChangeDetectionStrategy.OnPush,
 })
 export class MatLegacySelectionList
diff --git a/src/material/legacy-list/testing/action-list-harness.ts b/src/material/legacy-list/testing/action-list-harness.ts
@@ -8,14 +8,17 @@
 
 import {HarnessPredicate} from '@angular/cdk/testing';
 import {MatLegacyListHarnessBase} from './list-harness-base';
-import {ActionListHarnessFilters, ActionListItemHarnessFilters} from './list-harness-filters';
+import {
+  LegacyActionListHarnessFilters,
+  LegacyActionListItemHarnessFilters,
+} from './list-harness-filters';
 import {getListItemPredicate, MatLegacyListItemHarnessBase} from './list-item-harness-base';
 
 /** Harness for interacting with a standard mat-action-list in tests. */
 export class MatLegacyActionListHarness extends MatLegacyListHarnessBase<
   typeof MatLegacyActionListItemHarness,
   MatLegacyActionListItemHarness,
-  ActionListItemHarnessFilters
+  LegacyActionListItemHarnessFilters
 > {
   /** The selector for the host element of a `MatActionList` instance. */
   static hostSelector = 'mat-action-list.mat-list';
@@ -27,7 +30,7 @@ export class MatLegacyActionListHarness extends MatLegacyListHarnessBase<
    * @return a `HarnessPredicate` configured with the given options.
    */
   static with(
-    options: ActionListHarnessFilters = {},
+    options: LegacyActionListHarnessFilters = {},
   ): HarnessPredicate<MatLegacyActionListHarness> {
     return new HarnessPredicate(MatLegacyActionListHarness, options);
   }
@@ -47,7 +50,7 @@ export class MatLegacyActionListItemHarness extends MatLegacyListItemHarnessBase
    * @return a `HarnessPredicate` configured with the given options.
    */
   static with(
-    options: ActionListItemHarnessFilters = {},
+    options: LegacyActionListItemHarnessFilters = {},
   ): HarnessPredicate<MatLegacyActionListItemHarness> {
     return getListItemPredicate(MatLegacyActionListItemHarness, options);
   }
diff --git a/src/material/legacy-list/testing/list-harness-base.ts b/src/material/legacy-list/testing/list-harness-base.ts
@@ -13,7 +13,10 @@ import {
   parallel,
 } from '@angular/cdk/testing';
 import {DividerHarnessFilters, MatDividerHarness} from '@angular/material/divider/testing';
-import {BaseListItemHarnessFilters, SubheaderHarnessFilters} from './list-harness-filters';
+import {
+  LegacyBaseListItemHarnessFilters,
+  LegacySubheaderHarnessFilters,
+} from './list-harness-filters';
 import {MatLegacySubheaderHarness} from './list-item-harness-base';
 
 /** Represents a section of a list falling under a specific header. */
@@ -35,7 +38,7 @@ export interface ListSection<I> {
 export abstract class MatLegacyListHarnessBase<
   T extends ComponentHarnessConstructor<C> & {with: (options?: F) => HarnessPredicate<C>},
   C extends ComponentHarness,
-  F extends BaseListItemHarnessFilters,
+  F extends LegacyBaseListItemHarnessFilters,
 > extends ComponentHarness {
   protected _itemHarness: T;
 
@@ -130,7 +133,7 @@ export abstract class MatLegacyListHarnessBase<
   }): Promise<C[]>;
   getItemsWithSubheadersAndDividers(filters: {
     item: false;
-    subheader?: SubheaderHarnessFilters | false;
+    subheader?: LegacySubheaderHarnessFilters | false;
     divider: false;
   }): Promise<MatLegacySubheaderHarness[]>;
   getItemsWithSubheadersAndDividers(filters: {
@@ -140,7 +143,7 @@ export abstract class MatLegacyListHarnessBase<
   }): Promise<MatDividerHarness[]>;
   getItemsWithSubheadersAndDividers(filters: {
     item?: F | false;
-    subheader?: SubheaderHarnessFilters | false;
+    subheader?: LegacySubheaderHarnessFilters | false;
     divider: false;
   }): Promise<(C | MatLegacySubheaderHarness)[]>;
   getItemsWithSubheadersAndDividers(filters: {
@@ -150,18 +153,18 @@ export abstract class MatLegacyListHarnessBase<
   }): Promise<(C | MatDividerHarness)[]>;
   getItemsWithSubheadersAndDividers(filters: {
     item: false;
-    subheader?: false | SubheaderHarnessFilters;
+    subheader?: false | LegacySubheaderHarnessFilters;
     divider?: false | DividerHarnessFilters;
   }): Promise<(MatLegacySubheaderHarness | MatDividerHarness)[]>;
   getItemsWithSubheadersAndDividers(filters?: {
     item?: F | false;
-    subheader?: SubheaderHarnessFilters | false;
+    subheader?: LegacySubheaderHarnessFilters | false;
     divider?: DividerHarnessFilters | false;
   }): Promise<(C | MatLegacySubheaderHarness | MatDividerHarness)[]>;
   async getItemsWithSubheadersAndDividers(
     filters: {
       item?: F | false;
-      subheader?: SubheaderHarnessFilters | false;
+      subheader?: LegacySubheaderHarnessFilters | false;
       divider?: DividerHarnessFilters | false;
     } = {},
   ): Promise<(C | MatLegacySubheaderHarness | MatDividerHarness)[]> {
diff --git a/src/material/legacy-list/testing/list-harness-filters.ts b/src/material/legacy-list/testing/list-harness-filters.ts
@@ -8,30 +8,30 @@
 
 import {BaseHarnessFilters} from '@angular/cdk/testing';
 
-export interface ListHarnessFilters extends BaseHarnessFilters {}
+export interface LegacyListHarnessFilters extends BaseHarnessFilters {}
 
-export interface ActionListHarnessFilters extends BaseHarnessFilters {}
+export interface LegacyActionListHarnessFilters extends BaseHarnessFilters {}
 
-export interface NavListHarnessFilters extends BaseHarnessFilters {}
+export interface LegacyNavListHarnessFilters extends BaseHarnessFilters {}
 
-export interface SelectionListHarnessFilters extends BaseHarnessFilters {}
+export interface LegacySelectionListHarnessFilters extends BaseHarnessFilters {}
 
-export interface BaseListItemHarnessFilters extends BaseHarnessFilters {
+export interface LegacyBaseListItemHarnessFilters extends BaseHarnessFilters {
   text?: string | RegExp;
 }
 
-export interface ListItemHarnessFilters extends BaseListItemHarnessFilters {}
+export interface LegacyListItemHarnessFilters extends LegacyBaseListItemHarnessFilters {}
 
-export interface ActionListItemHarnessFilters extends BaseListItemHarnessFilters {}
+export interface LegacyActionListItemHarnessFilters extends LegacyBaseListItemHarnessFilters {}
 
-export interface NavListItemHarnessFilters extends BaseListItemHarnessFilters {
+export interface LegacyNavListItemHarnessFilters extends LegacyBaseListItemHarnessFilters {
   href?: string | RegExp | null;
 }
 
-export interface ListOptionHarnessFilters extends BaseListItemHarnessFilters {
+export interface LegacyListOptionHarnessFilters extends LegacyBaseListItemHarnessFilters {
   selected?: boolean;
 }
 
-export interface SubheaderHarnessFilters extends BaseHarnessFilters {
+export interface LegacySubheaderHarnessFilters extends BaseHarnessFilters {
   text?: string | RegExp;
 }
diff --git a/src/material/legacy-list/testing/list-harness.ts b/src/material/legacy-list/testing/list-harness.ts
@@ -8,14 +8,14 @@
 
 import {HarnessPredicate} from '@angular/cdk/testing';
 import {MatLegacyListHarnessBase} from './list-harness-base';
-import {ListHarnessFilters, ListItemHarnessFilters} from './list-harness-filters';
+import {LegacyListHarnessFilters, LegacyListItemHarnessFilters} from './list-harness-filters';
 import {getListItemPredicate, MatLegacyListItemHarnessBase} from './list-item-harness-base';
 
 /** Harness for interacting with a standard mat-list in tests. */
 export class MatLegacyListHarness extends MatLegacyListHarnessBase<
   typeof MatLegacyListItemHarness,
   MatLegacyListItemHarness,
-  ListItemHarnessFilters
+  LegacyListItemHarnessFilters
 > {
   /** The selector for the host element of a `MatList` instance. */
   static hostSelector = '.mat-list:not(mat-action-list)';
@@ -26,7 +26,7 @@ export class MatLegacyListHarness extends MatLegacyListHarnessBase<
    * @param options Options for filtering which list instances are considered a match.
    * @return a `HarnessPredicate` configured with the given options.
    */
-  static with(options: ListHarnessFilters = {}): HarnessPredicate<MatLegacyListHarness> {
+  static with(options: LegacyListHarnessFilters = {}): HarnessPredicate<MatLegacyListHarness> {
     return new HarnessPredicate(MatLegacyListHarness, options);
   }
 
@@ -44,7 +44,9 @@ export class MatLegacyListItemHarness extends MatLegacyListItemHarnessBase {
    * @param options Options for filtering which list item instances are considered a match.
    * @return a `HarnessPredicate` configured with the given options.
    */
-  static with(options: ListItemHarnessFilters = {}): HarnessPredicate<MatLegacyListItemHarness> {
+  static with(
+    options: LegacyListItemHarnessFilters = {},
+  ): HarnessPredicate<MatLegacyListItemHarness> {
     return getListItemPredicate(MatLegacyListItemHarness, options);
   }
 }
diff --git a/src/material/legacy-list/testing/list-item-harness-base.ts b/src/material/legacy-list/testing/list-item-harness-base.ts
@@ -13,7 +13,10 @@ import {
   ContentContainerComponentHarness,
   parallel,
 } from '@angular/cdk/testing';
-import {BaseListItemHarnessFilters, SubheaderHarnessFilters} from './list-harness-filters';
+import {
+  LegacyBaseListItemHarnessFilters,
+  LegacySubheaderHarnessFilters,
+} from './list-harness-filters';
 
 const iconSelector = '.mat-list-icon';
 const avatarSelector = '.mat-list-avatar';
@@ -28,7 +31,7 @@ const avatarSelector = '.mat-list-avatar';
  */
 export function getListItemPredicate<H extends MatLegacyListItemHarnessBase>(
   harnessType: ComponentHarnessConstructor<H>,
-  options: BaseListItemHarnessFilters,
+  options: LegacyBaseListItemHarnessFilters,
 ): HarnessPredicate<H> {
   return new HarnessPredicate(harnessType, options).addOption(
     'text',
@@ -41,7 +44,9 @@ export function getListItemPredicate<H extends MatLegacyListItemHarnessBase>(
 export class MatLegacySubheaderHarness extends ComponentHarness {
   static hostSelector = '.mat-subheader';
 
-  static with(options: SubheaderHarnessFilters = {}): HarnessPredicate<MatLegacySubheaderHarness> {
+  static with(
+    options: LegacySubheaderHarnessFilters = {},
+  ): HarnessPredicate<MatLegacySubheaderHarness> {
     return new HarnessPredicate(MatLegacySubheaderHarness, options).addOption(
       'text',
       options.text,
diff --git a/src/material/legacy-list/testing/nav-list-harness.ts b/src/material/legacy-list/testing/nav-list-harness.ts
@@ -8,14 +8,14 @@
 
 import {HarnessPredicate} from '@angular/cdk/testing';
 import {MatLegacyListHarnessBase} from './list-harness-base';
-import {NavListHarnessFilters, NavListItemHarnessFilters} from './list-harness-filters';
+import {LegacyNavListHarnessFilters, LegacyNavListItemHarnessFilters} from './list-harness-filters';
 import {getListItemPredicate, MatLegacyListItemHarnessBase} from './list-item-harness-base';
 
 /** Harness for interacting with a standard mat-nav-list in tests. */
 export class MatLegacyNavListHarness extends MatLegacyListHarnessBase<
   typeof MatLegacyNavListItemHarness,
   MatLegacyNavListItemHarness,
-  NavListItemHarnessFilters
+  LegacyNavListItemHarnessFilters
 > {
   /** The selector for the host element of a `MatNavList` instance. */
   static hostSelector = '.mat-nav-list';
@@ -26,7 +26,9 @@ export class MatLegacyNavListHarness extends MatLegacyListHarnessBase<
    * @param options Options for filtering which nav list instances are considered a match.
    * @return a `HarnessPredicate` configured with the given options.
    */
-  static with(options: NavListHarnessFilters = {}): HarnessPredicate<MatLegacyNavListHarness> {
+  static with(
+    options: LegacyNavListHarnessFilters = {},
+  ): HarnessPredicate<MatLegacyNavListHarness> {
     return new HarnessPredicate(MatLegacyNavListHarness, options);
   }
 
@@ -45,7 +47,7 @@ export class MatLegacyNavListItemHarness extends MatLegacyListItemHarnessBase {
    * @return a `HarnessPredicate` configured with the given options.
    */
   static with(
-    options: NavListItemHarnessFilters = {},
+    options: LegacyNavListItemHarnessFilters = {},
   ): HarnessPredicate<MatLegacyNavListItemHarness> {
     return getListItemPredicate(MatLegacyNavListItemHarness, options).addOption(
       'href',
diff --git a/src/material/legacy-list/testing/public-api.ts b/src/material/legacy-list/testing/public-api.ts
@@ -6,9 +6,20 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-export * from './action-list-harness';
-export * from './list-harness';
-export * from './list-harness-filters';
-export * from './nav-list-harness';
-export * from './selection-list-harness';
+export {MatLegacyActionListHarness, MatLegacyActionListItemHarness} from './action-list-harness';
+export {MatLegacyListHarness, MatLegacyListItemHarness} from './list-harness';
+export {
+  LegacyListHarnessFilters,
+  LegacyActionListHarnessFilters,
+  LegacyNavListHarnessFilters,
+  LegacySelectionListHarnessFilters,
+  LegacyBaseListItemHarnessFilters,
+  LegacyListItemHarnessFilters,
+  LegacyActionListItemHarnessFilters,
+  LegacyNavListItemHarnessFilters,
+  LegacyListOptionHarnessFilters,
+  LegacySubheaderHarnessFilters,
+} from './list-harness-filters';
+export {MatLegacyNavListHarness, MatLegacyNavListItemHarness} from './nav-list-harness';
+export {MatLegacySelectionListHarness, MatLegacyListOptionHarness} from './selection-list-harness';
 export {MatLegacyListItemSection} from './list-item-harness-base';
diff --git a/src/material/legacy-list/testing/selection-list-harness.ts b/src/material/legacy-list/testing/selection-list-harness.ts
@@ -10,17 +10,17 @@ import {HarnessPredicate, parallel} from '@angular/cdk/testing';
 import {MatLegacyListOptionCheckboxPosition} from '@angular/material/legacy-list';
 import {MatLegacyListHarnessBase} from './list-harness-base';
 import {
-  ListItemHarnessFilters,
-  ListOptionHarnessFilters,
-  SelectionListHarnessFilters,
+  LegacyListItemHarnessFilters,
+  LegacyListOptionHarnessFilters,
+  LegacySelectionListHarnessFilters,
 } from './list-harness-filters';
 import {getListItemPredicate, MatLegacyListItemHarnessBase} from './list-item-harness-base';
 
 /** Harness for interacting with a standard mat-selection-list in tests. */
 export class MatLegacySelectionListHarness extends MatLegacyListHarnessBase<
   typeof MatLegacyListOptionHarness,
   MatLegacyListOptionHarness,
-  ListOptionHarnessFilters
+  LegacyListOptionHarnessFilters
 > {
   /** The selector for the host element of a `MatSelectionList` instance. */
   static hostSelector = '.mat-selection-list';
@@ -32,7 +32,7 @@ export class MatLegacySelectionListHarness extends MatLegacyListHarnessBase<
    * @return a `HarnessPredicate` configured with the given options.
    */
   static with(
-    options: SelectionListHarnessFilters = {},
+    options: LegacySelectionListHarnessFilters = {},
   ): HarnessPredicate<MatLegacySelectionListHarness> {
     return new HarnessPredicate(MatLegacySelectionListHarness, options);
   }
@@ -48,7 +48,7 @@ export class MatLegacySelectionListHarness extends MatLegacyListHarnessBase<
    * Selects all items matching any of the given filters.
    * @param filters Filters that specify which items should be selected.
    */
-  async selectItems(...filters: ListOptionHarnessFilters[]): Promise<void> {
+  async selectItems(...filters: LegacyListOptionHarnessFilters[]): Promise<void> {
     const items = await this._getItems(filters);
     await parallel(() => items.map(item => item.select()));
   }
@@ -57,14 +57,14 @@ export class MatLegacySelectionListHarness extends MatLegacyListHarnessBase<
    * Deselects all items matching any of the given filters.
    * @param filters Filters that specify which items should be deselected.
    */
-  async deselectItems(...filters: ListItemHarnessFilters[]): Promise<void> {
+  async deselectItems(...filters: LegacyListItemHarnessFilters[]): Promise<void> {
     const items = await this._getItems(filters);
     await parallel(() => items.map(item => item.deselect()));
   }
 
   /** Gets all items matching the given list of filters. */
   private async _getItems(
-    filters: ListOptionHarnessFilters[],
+    filters: LegacyListOptionHarnessFilters[],
   ): Promise<MatLegacyListOptionHarness[]> {
     if (!filters.length) {
       return this.getItems();
@@ -88,7 +88,7 @@ export class MatLegacyListOptionHarness extends MatLegacyListItemHarnessBase {
    * @return a `HarnessPredicate` configured with the given options.
    */
   static with(
-    options: ListOptionHarnessFilters = {},
+    options: LegacyListOptionHarnessFilters = {},
   ): HarnessPredicate<MatLegacyListOptionHarness> {
     return getListItemPredicate(MatLegacyListOptionHarness, options).addOption(
       'is selected',
diff --git a/src/material/legacy-list/testing/shared.spec.ts b/src/material/legacy-list/testing/shared.spec.ts
diff --git a/tools/public_api_guard/material/legacy-list-testing.md b/tools/public_api_guard/material/legacy-list-testing.md
diff --git a/tools/public_api_guard/material/legacy-list.md b/tools/public_api_guard/material/legacy-list.md
