fix: fix test ts

NG-ZORRO · Apr 28, 2020 · 10d1740 · 10d1740
1 parent c6bd419
commit 10d1740
Show file tree

Hide file tree

Showing 68 changed files with 330 additions and 379 deletions.
diff --git a/components/affix/affix.spec.ts b/components/affix/affix.spec.ts
@@ -501,9 +501,8 @@ describe('affix-extra', () => {
   `
 })
 class TestAffixComponent {
-  @ViewChild(NzAffixComponent, { static: true })
-  nzAffixComponent: NzAffixComponent;
+  @ViewChild(NzAffixComponent, { static: true }) nzAffixComponent!: NzAffixComponent;
   fakeTarget: string | Element | Window | null = null;
-  newOffset: {};
-  newOffsetBottom: {};
+  newOffset: {} = {};
+  newOffsetBottom: {} = {};
 }
diff --git a/components/alert/alert.spec.ts b/components/alert/alert.spec.ts
@@ -135,21 +135,19 @@ describe('alert', () => {
   `
 })
 export class NzDemoTestBasicComponent {
-  @ViewChild('template', { static: false }) template: TemplateRef<void>;
+  @ViewChild('template', { static: false }) template!: TemplateRef<void>;
   banner = false;
   closeable = false;
-  closeText: string | TemplateRef<void>;
+  closeText?: string | TemplateRef<void>;
   description: string | TemplateRef<void> = 'description';
   message: string | TemplateRef<void> = 'message';
   showIcon = false;
-  iconType: string;
+  iconType?: string;
   type = 'info';
   onClose = jasmine.createSpy('close callback');
 }
 
 @Component({
-  template: `
-    <nz-alert nzBanner> </nz-alert>
-  `
+  template: ` <nz-alert nzBanner> </nz-alert> `
 })
 export class NzDemoTestBannerComponent {}
diff --git a/components/anchor/anchor.spec.ts b/components/anchor/anchor.spec.ts
@@ -286,7 +286,7 @@ describe('anchor', () => {
   styleUrls: ['./style/patch.less']
 })
 export class TestComponent {
-  @ViewChild(NzAnchorComponent, { static: false }) comp: NzAnchorComponent;
+  @ViewChild(NzAnchorComponent, { static: false }) comp!: NzAnchorComponent;
   nzAffix = true;
   nzBounds = 5;
   nzOffsetTop = 0;

diff --git a/components/auto-complete/autocomplete-trigger.directive.ts b/components/auto-complete/autocomplete-trigger.directive.ts
@@ -268,7 +268,7 @@ export class NzAutocompleteTriggerDirective implements ControlValueAccessor, OnD
       throw getNzAutocompleteMissingPanelError();
     }
 
-    if (!this.portal) {
+    if (!this.portal && this.nzAutocomplete.template) {
       this.portal = new TemplatePortal(this.nzAutocomplete.template, this.viewContainerRef);
     }
 

diff --git a/components/auto-complete/autocomplete.component.ts b/components/auto-complete/autocomplete.component.ts
@@ -99,7 +99,7 @@ export class NzAutocompleteComponent implements AfterContentInit, AfterViewInit,
 
   showPanel: boolean = true;
   isOpen: boolean = false;
-  activeItem?: NzAutocompleteOptionComponent;
+  activeItem!: NzAutocompleteOptionComponent;
   dropDownPosition: NzDropDownPosition = 'bottom';
 
   /**

diff --git a/components/auto-complete/autocomplete.spec.ts b/components/auto-complete/autocomplete.spec.ts
@@ -917,14 +917,14 @@ describe('auto-complete', () => {
   `
 })
 class NzTestSimpleAutocompleteComponent {
-  inputValue: string;
+  inputValue!: string;
   filteredOptions: Array<string | number>;
   inputControl = new FormControl();
   options: Array<string | number> = ['Burns Bay Road', 'Downing Street', 'Wall Street'];
 
-  @ViewChild(NzAutocompleteComponent, { static: false }) panel: NzAutocompleteComponent;
-  @ViewChild(NzAutocompleteTriggerDirective, { static: false }) trigger: NzAutocompleteTriggerDirective;
-  @ViewChildren(NzAutocompleteOptionComponent) optionComponents: QueryList<NzAutocompleteOptionComponent>;
+  @ViewChild(NzAutocompleteComponent, { static: false }) panel!: NzAutocompleteComponent;
+  @ViewChild(NzAutocompleteTriggerDirective, { static: false }) trigger!: NzAutocompleteTriggerDirective;
+  @ViewChildren(NzAutocompleteOptionComponent) optionComponents!: QueryList<NzAutocompleteOptionComponent>;
 
   constructor() {
     this.filteredOptions = this.options;
@@ -953,13 +953,13 @@ class NzTestSimpleAutocompleteComponent {
   `
 })
 class NzTestAutocompletePropertyComponent {
-  inputValue: string;
-  width: number;
+  inputValue?: string;
+  width?: number;
   overlayClassName = '';
   overlayStyle = {};
   options = ['Burns Bay Road', 'Downing Street', 'Wall Street'];
-  @ViewChild(NzAutocompleteComponent, { static: false }) panel: NzAutocompleteComponent;
-  @ViewChild(NzAutocompleteTriggerDirective, { static: false }) trigger: NzAutocompleteTriggerDirective;
+  @ViewChild(NzAutocompleteComponent, { static: false }) panel!: NzAutocompleteComponent;
+  @ViewChild(NzAutocompleteTriggerDirective, { static: false }) trigger!: NzAutocompleteTriggerDirective;
 
   constructor() {}
 }
@@ -968,7 +968,7 @@ class NzTestAutocompletePropertyComponent {
   template: ` <input [nzAutocomplete]="auto" /> `
 })
 class NzTestAutocompleteWithoutPanelComponent {
-  @ViewChild(NzAutocompleteTriggerDirective, { static: false }) trigger: NzAutocompleteTriggerDirective;
+  @ViewChild(NzAutocompleteTriggerDirective, { static: false }) trigger!: NzAutocompleteTriggerDirective;
 }
 
 @Component({
@@ -982,7 +982,7 @@ class NzTestAutocompleteWithoutPanelComponent {
 })
 class NzTestAutocompleteWithOnPushDelayComponent implements OnInit {
   options: string[] = [];
-  @ViewChild(NzAutocompleteTriggerDirective, { static: false }) trigger: NzAutocompleteTriggerDirective;
+  @ViewChild(NzAutocompleteTriggerDirective, { static: false }) trigger!: NzAutocompleteTriggerDirective;
 
   constructor(private cdr: ChangeDetectorRef) {}
 
@@ -1014,7 +1014,7 @@ class NzTestAutocompleteWithOnPushDelayComponent implements OnInit {
   `
 })
 class NzTestAutocompleteGroupComponent {
-  inputValue: string;
+  inputValue!: string;
   optionGroups = [
     {
       title: '话题',
@@ -1054,7 +1054,7 @@ class NzTestAutocompleteGroupComponent {
     }
   ];
 
-  @ViewChild(NzAutocompleteTriggerDirective, { static: false }) trigger: NzAutocompleteTriggerDirective;
+  @ViewChild(NzAutocompleteTriggerDirective, { static: false }) trigger!: NzAutocompleteTriggerDirective;
 }
 
 @Component({
@@ -1070,7 +1070,7 @@ class NzTestAutocompleteGroupComponent {
 class NzTestAutocompleteWithFormComponent {
   form: FormGroup;
   options = ['Burns Bay Road', 'Downing Street', 'Wall Street'];
-  @ViewChild(NzAutocompleteTriggerDirective, { static: false }) trigger: NzAutocompleteTriggerDirective;
+  @ViewChild(NzAutocompleteTriggerDirective, { static: false }) trigger!: NzAutocompleteTriggerDirective;
 
   constructor(private fb: FormBuilder) {
     this.form = this.fb.group({ formControl: 'Burns' });
@@ -1095,7 +1095,7 @@ class NzTestAutocompleteDifferentValueWithFormComponent {
     { label: 'Lucy', value: 'lucy' },
     { label: 'Jack', value: 'jack' }
   ];
-  @ViewChild(NzAutocompleteTriggerDirective) trigger: NzAutocompleteTriggerDirective;
+  @ViewChild(NzAutocompleteTriggerDirective) trigger!: NzAutocompleteTriggerDirective;
 
   constructor(private fb: FormBuilder) {
     this.form = this.fb.group({ formControl: 'lucy' });
@@ -1118,7 +1118,7 @@ class NzTestAutocompleteWithObjectOptionComponent {
     { label: 'Lucy', value: 'lucy' },
     { label: 'Jack', value: 'jack' }
   ];
-  @ViewChild(NzAutocompleteTriggerDirective) trigger: NzAutocompleteTriggerDirective;
+  @ViewChild(NzAutocompleteTriggerDirective) trigger!: NzAutocompleteTriggerDirective;
 
   // tslint:disable-next-line: no-any
   compareFun = (o1: any, o2: any) => {

diff --git a/components/avatar/avatar.spec.ts b/components/avatar/avatar.spec.ts
@@ -216,12 +216,12 @@ describe('avatar', () => {
   styleUrls: ['./style/index.less']
 })
 class TestAvatarComponent {
-  @ViewChild('comp', { static: false }) comp: NzAvatarComponent;
+  @ViewChild('comp', { static: false }) comp!: NzAvatarComponent;
   nzShape = 'square';
   nzSize: string | number = 'large';
   nzIcon: string | null = 'user';
   nzText: string | null = 'A';
   nzSrc: string | null = imageBase64;
-  nzSrcSet: string;
-  nzAlt: string;
+  nzSrcSet?: string;
+  nzAlt?: string;
 }
diff --git a/components/back-top/back-top.spec.ts b/components/back-top/back-top.spec.ts
@@ -203,7 +203,7 @@ describe('Component:nz-back-top', () => {
 })
 class TestBackTopComponent {
   @ViewChild(NzBackTopComponent, { static: true })
-  nzBackTopComponent: NzBackTopComponent;
+  nzBackTopComponent!: NzBackTopComponent;
 
   target: HTMLElement | null = null;
 
@@ -224,11 +224,11 @@ class TestBackTopComponent {
 })
 class TestBackTopTemplateComponent {
   @ViewChild(NzBackTopComponent, { static: false })
-  nzBackTopComponent: NzBackTopComponent;
+  nzBackTopComponent!: NzBackTopComponent;
 }
 
 class MockNzScrollService {
-  mockTopOffset: number;
+  mockTopOffset: number = 0;
 
   getScroll(): number {
     return this.mockTopOffset;

diff --git a/components/badge/badge.spec.ts b/components/badge/badge.spec.ts
@@ -165,9 +165,9 @@ export class NzTestBadgeBasicComponent {
   inner = true;
   overflow = 20;
   showZero = false;
-  status: string;
-  style: NgStyleInterface;
-  text: string;
-  title: string | null;
-  offset: [number, number];
+  status!: string;
+  style!: NgStyleInterface;
+  text!: string;
+  title: string | null = null;
+  offset?: [number, number];
 }
diff --git a/components/button/button.spec.ts b/components/button/button.spec.ts
@@ -178,7 +178,7 @@ export class TestButtonBindingComponent {
   `
 })
 export class TestButtonWithIconComponent implements OnInit {
-  title: string;
+  title?: string;
   ngOnInit(): void {
     setTimeout(() => (this.title = 'button'), 5000);
   }

diff --git a/components/carousel/carousel.spec.ts b/components/carousel/carousel.spec.ts
@@ -33,7 +33,7 @@ import { NZ_CAROUSEL_CUSTOM_STRATEGIES } from './typings';
   `
 })
 export class NzTestCarouselBasicComponent {
-  @ViewChild(NzCarouselComponent, { static: false }) nzCarouselComponent: NzCarouselComponent;
+  @ViewChild(NzCarouselComponent, { static: false }) nzCarouselComponent!: NzCarouselComponent;
   dots = true;
   dotPosition = 'bottom';
   effect = 'scrollx';
@@ -211,7 +211,7 @@ describe('carousel', () => {
     }));
 
     it('should resize content after window resized', fakeAsync(() => {
-      const resizeSpy = spyOn(testComponent.nzCarouselComponent.strategy, 'withCarouselContents');
+      const resizeSpy = spyOn(testComponent.nzCarouselComponent.strategy!, 'withCarouselContents');
       window.dispatchEvent(new Event('resize'));
       tick(16);
       expect(resizeSpy).toHaveBeenCalledTimes(1);

diff --git a/components/cascader/cascader.spec.ts b/components/cascader/cascader.spec.ts
@@ -1977,8 +1977,8 @@ const options5: any[] = []; // tslint:disable-line:no-any
   ]
 })
 export class NzDemoCascaderDefaultComponent {
-  @ViewChild(NzCascaderComponent, { static: true }) cascader: NzCascaderComponent;
-  @ViewChild('renderTpl', { static: true }) renderTpl: TemplateRef<any>;
+  @ViewChild(NzCascaderComponent, { static: true }) cascader!: NzCascaderComponent;
+  @ViewChild('renderTpl', { static: true }) renderTpl!: TemplateRef<any>;
 
   public nzOptions: any[] | null = options1;
   public values: string[] | number[] | null = null;
@@ -2038,7 +2038,7 @@ export class NzDemoCascaderDefaultComponent {
   ]
 })
 export class NzDemoCascaderLoadDataComponent {
-  @ViewChild(NzCascaderComponent, { static: true }) cascader: NzCascaderComponent;
+  @ViewChild(NzCascaderComponent, { static: true }) cascader!: NzCascaderComponent;
 
   public nzOptions: any[] | null = null;
   public values: string[] | null = null;

diff --git a/components/checkbox/checkbox.spec.ts b/components/checkbox/checkbox.spec.ts
@@ -312,7 +312,7 @@ describe('checkbox', () => {
   `
 })
 export class NzTestCheckboxSingleComponent {
-  @ViewChild(NzCheckboxComponent, { static: false }) nzCheckboxComponent: NzCheckboxComponent;
+  @ViewChild(NzCheckboxComponent, { static: false }) nzCheckboxComponent!: NzCheckboxComponent;
   disabled = false;
   autoFocus = false;
   checked = false;
@@ -321,9 +321,7 @@ export class NzTestCheckboxSingleComponent {
 }
 
 @Component({
-  template: `
-    <nz-checkbox-group [nzDisabled]="disabled" [ngModel]="options" (ngModelChange)="modelChange($event)"></nz-checkbox-group>
-  `
+  template: ` <nz-checkbox-group [nzDisabled]="disabled" [ngModel]="options" (ngModelChange)="modelChange($event)"></nz-checkbox-group> `
 })
 export class NzTestCheckboxGroupComponent {
   options = [

diff --git a/components/collapse/collapse.spec.ts b/components/collapse/collapse.spec.ts
@@ -191,7 +191,7 @@ describe('collapse', () => {
   `
 })
 export class NzTestCollapseBasicComponent {
-  @ViewChild('headerTemplate', { static: false }) headerTemplate: TemplateRef<void>;
+  @ViewChild('headerTemplate', { static: false }) headerTemplate!: TemplateRef<void>;
   accordion = false;
   bordered = true;
   disabled = false;

diff --git a/components/core/outlet/string_template_outlet.spec.ts b/components/core/outlet/string_template_outlet.spec.ts
@@ -103,10 +103,10 @@ describe('string template outlet', () => {
   `
 })
 export class StringTemplateOutletTestComponent {
-  @ViewChild('stringTpl') stringTpl: TemplateRef<NzSafeAny>;
-  @ViewChild('emptyTpl') emptyTpl: TemplateRef<NzSafeAny>;
-  @ViewChild('dataTimeTpl') dataTimeTpl: TemplateRef<NzSafeAny>;
-  @ViewChild(NzStringTemplateOutletDirective) nzStringTemplateOutletDirective: NzStringTemplateOutletDirective;
+  @ViewChild('stringTpl') stringTpl!: TemplateRef<NzSafeAny>;
+  @ViewChild('emptyTpl') emptyTpl!: TemplateRef<NzSafeAny>;
+  @ViewChild('dataTimeTpl') dataTimeTpl!: TemplateRef<NzSafeAny>;
+  @ViewChild(NzStringTemplateOutletDirective) nzStringTemplateOutletDirective!: NzStringTemplateOutletDirective;
   stringTemplateOutlet: TemplateRef<NzSafeAny> | string | null = null;
   context: NzSafeAny = { $implicit: '' };
 }
diff --git a/components/core/services/scroll.spec.ts b/components/core/services/scroll.spec.ts
@@ -24,7 +24,7 @@ describe('NzScrollService', () => {
   }
 
   class MockPlatformLocation {
-    hash: string;
+    hash!: string;
   }
 
   beforeEach(() => {

diff --git a/components/core/wave/nz-wave.spec.ts b/components/core/wave/nz-wave.spec.ts
@@ -274,8 +274,8 @@ class WaveContainerWithButtonComponent {
   isDestroyed = false;
   borderColor = 'rgb(0,255,0)';
   backgroundColor = 'rgb(255,255,255)';
-  @ViewChild('trigger', { static: false }) trigger: ElementRef<HTMLElement>;
-  @ViewChild(NzWaveDirective, { static: false }) wave: NzWaveDirective;
+  @ViewChild('trigger', { static: false }) trigger!: ElementRef<HTMLElement>;
+  @ViewChild(NzWaveDirective, { static: false }) wave!: NzWaveDirective;
 }
 
 @Component({
@@ -298,6 +298,6 @@ class WaveContainerWithExtraNodeComponent {
   isDestroyed = false;
   borderColor = 'rgb(0,255,0)';
   backgroundColor = 'rgb(255,255,255)';
-  @ViewChild('trigger', { static: false }) trigger: ElementRef<HTMLElement>;
-  @ViewChild(NzWaveDirective, { static: false }) wave: NzWaveDirective;
+  @ViewChild('trigger', { static: false }) trigger!: ElementRef<HTMLElement>;
+  @ViewChild(NzWaveDirective, { static: false }) wave!: NzWaveDirective;
 }