From 7138b685590b17406932a19a65873446f2f315fa Mon Sep 17 00:00:00 2001 From: AllenFang Date: Sat, 9 Nov 2019 14:49:42 +0800 Subject: [PATCH] fix test cases --- .../test/header-cell.test.js | 47 ++++++++++++------- 1 file changed, 29 insertions(+), 18 deletions(-) diff --git a/packages/react-bootstrap-table2/test/header-cell.test.js b/packages/react-bootstrap-table2/test/header-cell.test.js index c7ea736..465bde1 100644 --- a/packages/react-bootstrap-table2/test/header-cell.test.js +++ b/packages/react-bootstrap-table2/test/header-cell.test.js @@ -148,7 +148,7 @@ describe('HeaderCell', () => { it('should call custom headerFormatter correctly', () => { expect(formatter.callCount).toBe(1); expect(formatter.calledWith( - column, index, { sortElement: undefined })).toBe(true); + column, index, { sortElement: undefined, filterElement: undefined })).toBe(true); }); }); @@ -729,7 +729,13 @@ describe('HeaderCell', () => { } }; wrapper = shallow( - + ); }); @@ -738,14 +744,14 @@ describe('HeaderCell', () => { expect(wrapper.find('th').length).toBe(1); }); - // it('should render filter correctly', () => { - // expect(wrapper.find(Filter).length).toBe(1); - // expect(wrapper.find(Filter).props()).toEqual({ - // column, - // onFilter, - // ...filterProps - // }); - // }); + it('should render filter correctly', () => { + expect(wrapper.find(Filter).length).toBe(1); + expect(wrapper.find(Filter).props()).toEqual({ + column, + onFilter, + ...filterProps + }); + }); }); describe('when column.filter and column.filterRenderer is defined', () => { @@ -767,7 +773,12 @@ describe('HeaderCell', () => { filterRenderer }; wrapper = shallow( - ); + ); }); it('should render successfully', () => { @@ -775,12 +786,12 @@ describe('HeaderCell', () => { expect(wrapper.find('th').length).toBe(1); }); - // it('should render filter correctly', () => { - // expect(wrapper.find(Filter).length).toBe(1); - // }); - // - // it('should call filterRenderer function correctly', () => { - // expect(filterRenderer).toHaveBeenCalledTimes(1); - // }); + it('should render filter correctly', () => { + expect(wrapper.find(Filter).length).toBe(1); + }); + + it('should call filterRenderer function correctly', () => { + expect(filterRenderer).toHaveBeenCalledTimes(1); + }); }); });