test(dialog): Add test coverage for browser platform (#2119)

Author: trimox

demos/src/app/app-routing.module.ts

@@ -28,7 +28,7 @@ const routes: Routes = [
     path: 'data-table-demo', loadChildren: () =>
       import('./components/data-table-demo/data-table.module').then(m => m.DataTableModule)
   },
-  {path: 'dialog-demo', loadChildren: () => import('./components/dialog-demo/dialog.module').then(m => m.DialogModule)},
+  {path: 'dialog-demo', loadChildren: () => import('./components/dialog/module').then(m => m.DialogModule)},
   {path: 'drawer-demo', loadChildren: () => import('./components/drawer-demo/drawer.module').then(m => m.DrawerModule)},
   {
     path: 'elevation-demo', loadChildren: () =>

demos/src/app/components/dialog-demo/dialog.module.ts

@@ -79,15 +79,18 @@ <h4 mdcSubtitle2>MdcDialogConfig</h4>
       </tr>
       <tr>
         <td>autoFocus: boolean</td>
-        <td>Whether the dialog should focus the first focusable element on open. (Default: true)</td>
+        <td>Whether the dialog should focus the first focusable element on open.
+          (Default: true)</td>
       </tr>
       <tr>
         <td>restoreFocus: boolean</td>
-        <td>Whether the dialog should restore focus to the previously-focused element, after it's closed. (Default: true)</td>
+        <td>Whether the dialog should restore focus to the previously-focused
+          element, after it's closed. (Default: true)</td>
       </tr>
       <tr>
         <td>componentFactoryResolver?: ComponentFactoryResolver</td>
-        <td>Allows for a different `ComponentFactoryResolver` to be passed in when creating a dialog.</td>
+        <td>Allows for a different `ComponentFactoryResolver` to be passed in
+          when creating a dialog.</td>
       </tr>
       <tr>
         <td>data: any</td>

demos/src/app/components/dialog-demo/routing.module.ts

@@ -20,7 +20,7 @@ export class Api {}
 export class Sass {}
 
 @Component({template: '<component-viewer></component-viewer>'})
-export class DialogDemo implements OnInit {
+export class Dialog implements OnInit {
   @ViewChild(ComponentViewer, {static: true}) _componentViewer: ComponentViewer;
 
   ngOnInit(): void {
@@ -84,7 +84,6 @@ export class Examples {
     const dialogRef = this.dialog.open(DialogConfirmation);
 
     dialogRef.afterOpened().subscribe(() => {
-      dialogRef.componentInstance.dialogComponent.layout();
       console.log('Dialog opened');
     });
 

…/src/app/components/dialog-demo/api.html demos/src/app/components/dialog/api.htmldemos/src/app/components/dialog-demo/api.html renamed to demos/src/app/components/dialog/api.html demos/src/app/components/dialog-demo/api.html renamed to demos/src/app/components/dialog/api.html

@@ -0,0 +1,28 @@
+import {NgModule} from '@angular/core';
+
+import {SharedModule} from '../../shared.module';
+import {RoutingModule, ROUTE_DECLARATIONS} from './routing.module';
+
+import {
+  DialogAlert,
+  DialogConfirmation,
+  DialogScrollable,
+  DialogForm,
+  DialogSimple
+} from './dialog';
+
+@NgModule({
+  imports: [
+    RoutingModule,
+    SharedModule
+  ],
+  declarations: [
+    ROUTE_DECLARATIONS,
+    DialogAlert,
+    DialogConfirmation,
+    DialogForm,
+    DialogScrollable,
+    DialogSimple
+  ]
+})
+export class DialogModule {}

…onents/dialog-demo/dialog-form-demo.html …/components/dialog/dialog-form-demo.htmldemos/src/app/components/dialog-demo/dialog-form-demo.html renamed to demos/src/app/components/dialog/dialog-form-demo.html demos/src/app/components/dialog-demo/dialog-form-demo.html renamed to demos/src/app/components/dialog/dialog-form-demo.html

@@ -0,0 +1,31 @@
+import {NgModule} from '@angular/core';
+import {Routes, RouterModule} from '@angular/router';
+
+import {Api, Examples, Usage, Sass, Dialog} from './dialog';
+
+export const ROUTE_DECLARATIONS = [
+  Api,
+  Examples,
+  Sass,
+  Usage,
+  Dialog
+];
+
+const ROUTES: Routes = [
+  {
+    path: '', component: Dialog,
+    children: [
+      {path: '', redirectTo: 'usage'},
+      {path: 'usage', component: Usage},
+      {path: 'api', component: Api},
+      {path: 'sass', component: Sass},
+      {path: 'examples', component: Examples}
+    ]
+  }
+];
+
+@NgModule({
+  imports: [RouterModule.forChild(ROUTES)],
+  exports: [RouterModule]
+})
+export class RoutingModule {}