fix(module:table): should prevent tr click trigger when clicking expa…

…nd (#2618)

close #2419

components/table/nz-td.component.html

@@ -13,7 +13,7 @@
 <span
   *ngIf="nzShowExpand"
   class="ant-table-row-expand-icon"
-  (click)="expandChange()"
+  (click)="expandChange($event)"
   [class.ant-table-row-expanded]="nzExpand"
   [class.ant-table-row-collapsed]="!nzExpand"></span>
 <ng-content></ng-content>

components/table/nz-td.component.ts

@@ -102,7 +102,8 @@ export class NzTdComponent {
     }
   }
 
-  expandChange(): void {
+  expandChange(e: Event): void {
+    e.stopPropagation();
     this.nzExpand = !this.nzExpand;
     this.nzExpandChange.emit(this.nzExpand);
   }

components/table/nz-td.spec.ts

@@ -1,6 +1,7 @@
 import { Component } from '@angular/core';
 import { fakeAsync, flush, TestBed } from '@angular/core/testing';
 import { By } from '@angular/platform-browser';
+import { createFakeEvent } from '../core/testing';
 import { NzTableModule } from './nz-table.module';
 import { NzTdComponent } from './nz-td.component';
 
@@ -100,6 +101,16 @@ describe('nz-td', () => {
       expect(td.nativeElement.querySelector('.ant-table-row-expand-icon').classList).toContain('ant-table-row-expanded');
       expect(testComponent.expandChange).toHaveBeenCalledTimes(1);
     });
+    it('should click expand event stopPropagation', () => {
+      testComponent.showExpand = true;
+      fixture.detectChanges();
+      const input: HTMLElement = td.nativeElement.querySelector('.ant-table-row-expand-icon');
+      const fakeInputChangeEvent = createFakeEvent('click', true, true);
+      spyOn(fakeInputChangeEvent, 'stopPropagation');
+      input.dispatchEvent(fakeInputChangeEvent);
+      fixture.detectChanges();
+      expect(fakeInputChangeEvent.stopPropagation).toHaveBeenCalled();
+    });
     it('should be row index when index-size is 0', () => {
       testComponent.indentSize = 0;
       fixture.detectChanges();