Commit 4da199f

authored

fix #29

1 parent b00007c commit 4da199fCopy full SHA for 4da199f

File tree

+37

-2

lines changed

packages/react-bootstrap-table2
- src
  - header-cell.js
- test
  - header-cell.test.js

+37

-2

lines changed

Lines changed: 11 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -5,8 +5,16 @@ import _ from './utils';`
`5`	`5`
`6`	`6`
`7`	`7`	`const HeaderCell = ({ column, index }) => {`
`8`		`- const { headerTitle, text, headerAlign, headerFormatter } = column;`
`9`		`- const attrs = {};`
	`8`	`+ const {`
	`9`	`+ text,`
	`10`	`+ headerTitle,`
	`11`	`+ headerAlign,`
	`12`	`+ headerFormatter,`
	`13`	`+ headerEvents`
	`14`	`+ } = column;`
	`15`	`+ const attrs = {`
	`16`	`+ ...headerEvents`
	`17`	`+ };`
`10`	`18`	`const headerStyle = {};`
`11`	`19`	`const children = headerFormatter ? headerFormatter(column, index) : text;`
`12`	`20`
`@@ -39,6 +47,7 @@ HeaderCell.propTypes = {`
`39`	`47`	`style: PropTypes.oneOfType([PropTypes.object, PropTypes.func]),`
`40`	`48`	`headerTitle: PropTypes.oneOfType([PropTypes.bool, PropTypes.func]),`
`41`	`49`	`title: PropTypes.oneOfType([PropTypes.bool, PropTypes.func]),`
	`50`	`+ headerEvents: PropTypes.object,`
`42`	`51`	`events: PropTypes.object,`
`43`	`52`	`headerAlign: PropTypes.oneOfType([PropTypes.string, PropTypes.func]),`
`44`	`53`	`align: PropTypes.oneOfType([PropTypes.string, PropTypes.func])`

Lines changed: 26 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -147,4 +147,30 @@ describe('HeaderCell', () => {`
`147`	`147`	`expect(formatter.calledWith(column, index)).toBe(true);`
`148`	`148`	`});`
`149`	`149`	`});`
	`150`	`+`
	`151`	`+ describe('when column.headerEvents prop is defined', () => {`
	`152`	`+ let column;`
	`153`	`+`
	`154`	`+ beforeEach(() => {`
	`155`	`+ column = {`
	`156`	`+ dataField: 'id',`
	`157`	`+ text: 'ID',`
	`158`	`+ headerEvents: {`
	`159`	`+ onClick: sinon.stub()`
	`160`	`+ }`
	`161`	`+ };`
	`162`	`+`
	`163`	`+ wrapper = shallow(<HeaderCell column={ column } index={ index } />);`
	`164`	`+ });`
	`165`	`+`
	`166`	`+ it('should attachs DOM event successfully', () => {`
	`167`	`+ expect(wrapper.length).toBe(1);`
	`168`	`+ expect(wrapper.find('th').prop('onClick')).toBeDefined();`
	`169`	`+ });`
	`170`	`+`
	`171`	`+ it('event hook should be called when triggering', () => {`
	`172`	`+ wrapper.find('th').simulate('click');`
	`173`	`+ expect(column.headerEvents.onClick.callCount).toBe(1);`
	`174`	`+ });`
	`175`	`+ });`
`150`	`176`	`});`

Comments

(0)