Skip to content

Commit

Permalink
Merge pull request #112 from sproogen/f/#105-button-tests
Browse files Browse the repository at this point in the history
F/#105 button tests
  • Loading branch information
sproogen authored Aug 14, 2017
2 parents 006092d + 9a2bc92 commit 135b7ae
Show file tree
Hide file tree
Showing 2 changed files with 130 additions and 4 deletions.
10 changes: 6 additions & 4 deletions app/Resources/client/src/components/Button/Button.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ class Button extends React.Component {
if (event) {
event.preventDefault()
}
if (!this.isDisabled()) {
if (!this.isDisabled() && this.props.callback) {
this.setState({
disabled : true,
loading : true
Expand Down Expand Up @@ -64,15 +64,17 @@ class Button extends React.Component {
}

Button.propTypes = {
text : PropTypes.string.isRequired,
text : PropTypes.string,
className : PropTypes.string,
disabled : PropTypes.bool,
callback : PropTypes.func.isRequired,
callback : PropTypes.func,
submitEvent : PropTypes.string
}

Button.defaultProps = {
disabled : false,
text : '',
className : '',
disabled : false,
}

export default Button
124 changes: 124 additions & 0 deletions app/Resources/client/tests/components/Button.spec.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,124 @@
import React from 'react'
import Button from 'components/Button/Button'
import { shallow, mount } from 'enzyme'
import EventBus from 'components/EventBus'

describe('(Component) Button', () => {
it('renders as a button', () => {
const wrapper = shallow(<Button />)
const inst = wrapper.instance()
expect(inst).to.be.instanceOf(Button)
expect(wrapper.name()).to.equal('button')
})

describe('(State)', () => {
it('Should have initial state', () => {
const wrapper = mount(<Button />)
expect(wrapper.state()).to.deep.equal({ disabled : false, loading : false })
})
})

describe('(Props)', () => {
it('Should have default props', () => {
const wrapper = mount(<Button />)
expect(wrapper.props().text).to.equal('')
expect(wrapper.props().className).to.equal('')
expect(wrapper.props().disabled).to.equal(false)
})

it('Should have props', () => {
const wrapper = mount(
<Button text='Click Me!' className='button' disabled callback={() => {}} submitEvent='onSubmit' />
)
expect(wrapper.props().text).to.equal('Click Me!')
expect(wrapper.props().className).to.equal('button')
expect(wrapper.props().disabled).to.equal(true)
expect(wrapper.props().callback).to.be.a('function')
expect(wrapper.props().submitEvent).to.equal('onSubmit')
})
})

describe('(Method) isDisabled', () => {
it('Should return false as default', () => {
const wrapper = shallow(<Button />).instance()
expect(wrapper.isDisabled()).to.equal(false)
})

it('Should return true for prop', () => {
const wrapper = shallow(<Button disabled />).instance()
expect(wrapper.isDisabled()).to.equal(true)
})

it('Should return true for state', () => {
const wrapper = shallow(<Button />).instance()
wrapper.setState({ disabled: true })
expect(wrapper.isDisabled()).to.equal(true)
})
})

describe('(Method) handlePress', () => {
it('Should update the state', () => {
const wrapper = shallow(<Button callback={() => Promise.resolve()} />)
const instance = wrapper.instance()
instance.handlePress()
expect(wrapper.state()).to.deep.equal({ disabled : true, loading : true })
})

it('Should call callback', () => {
const callback = sinon.stub().returns(Promise.resolve())
const wrapper = shallow(<Button callback={callback} />)
const instance = wrapper.instance()
instance.handlePress()
callback.should.have.been.calledOnce()
})

it('Shouldn\'t update the state when disabled', () => {
const wrapper = shallow(<Button disabled callback={() => Promise.resolve()} />)
const instance = wrapper.instance()
instance.handlePress()
expect(wrapper.state()).to.deep.equal({ disabled : false, loading : false })
})

it('Shouldn\'t update the state with no callback', () => {
const wrapper = shallow(<Button />)
const instance = wrapper.instance()
instance.handlePress()
expect(wrapper.state()).to.deep.equal({ disabled : false, loading : false })
})

// Note : I don't seem to be able to test the promise callback in handlePress
})

describe('(EventListener) submitEvent', () => {
const wrapper = mount(<Button submitEvent='submitButton' />)
const spy = sinon.spy(wrapper.instance(), 'handlePress')
it('Should call handlePress', () => {
EventBus.getEventBus().emit('submitButton')
spy.should.have.been.calledOnce()
})
})

describe('(Render) button', () => {
it('Should render the text', () => {
const wrapper = shallow(<Button text='Click Me!' />)
expect(wrapper.text()).to.equal('Click Me!')
})

it('Should be disabled', () => {
const wrapper = shallow(<Button text='Click Me!' disabled />)
expect(wrapper.props().disabled).to.equal(true)
expect(wrapper.props().className).to.equal('btn disabled')
})

it('Should have className', () => {
const wrapper = shallow(<Button className='button button--class' disabled />)
expect(wrapper.props().className).to.equal('btn button button--class disabled')
})

it('Should render spinner when loading', () => {
const wrapper = shallow(<Button text='Click Me!' />)
wrapper.setState({ disabled : true, loading : true })
expect(wrapper.text()).to.equal('<Spinner />')
})
})
})

0 comments on commit 135b7ae

Please sign in to comment.