diff --git a/spec/javascript/components/button.test.jsx b/spec/javascript/components/button.test.jsx index 0f16ebe8e..13bbe95a2 100644 --- a/spec/javascript/components/button.test.jsx +++ b/spec/javascript/components/button.test.jsx @@ -2,13 +2,72 @@ import { expect } from 'chai'; import { shallow } from 'enzyme'; import sinon from 'sinon'; -import Button from '../../../app/assets/javascripts/components/components/button' +import Button from '../../../app/assets/javascripts/components/components/button'; describe('', () => { - it('simulates click events', () => { - const onClick = sinon.spy(); - const wrapper = shallow(); + it('renders a button element', () => { + const wrapper = shallow(); + expect(wrapper).to.match('button'); + }); + + it('renders the given text', () => { + const text = 'foo'; + const wrapper = shallow(); + expect(wrapper.find('button')).to.have.text(text); + }); + + it('handles click events using the given handler', () => { + const handler = sinon.spy(); + const wrapper = shallow(); wrapper.find('button').simulate('click'); - expect(onClick.calledOnce).to.equal(true); + expect(handler.calledOnce).to.equal(true); + }); + + it('does not handle click events if props.disabled given', () => { + const handler = sinon.spy(); + const wrapper = shallow(); + wrapper.find('button').simulate('click'); + expect(handler.called).to.equal(false); + }); + + it('renders a disabled attribute if props.disabled given', () => { + const wrapper = shallow(); + expect(wrapper.find('button')).to.be.disabled(); + }); + + it('renders the children', () => { + const children =
children
; + const wrapper = shallow(); + expect(wrapper.find('button')).to.contain(children); + }); + + it('renders the props.text instead of children', () => { + const text = 'foo'; + const children =children
; + const wrapper = shallow(); + expect(wrapper.find('button')).to.have.text(text); + expect(wrapper.find('button')).to.not.contain(children); + }); + + it('renders style="display: block; width: 100%;" if props.block given', () => { + const wrapper = shallow(); + expect(wrapper.find('button')).to.have.style('display', 'block'); + expect(wrapper.find('button')).to.have.style('width', '100%'); + }); + + it('renders style="display: inline-block; width: auto;" by default', () => { + const wrapper = shallow(); + expect(wrapper.find('button')).to.have.style('display', 'inline-block'); + expect(wrapper.find('button')).to.have.style('width', 'auto'); + }); + + it('adds class "button-secondary" if props.secondary given', () => { + const wrapper = shallow(); + expect(wrapper.find('button')).to.have.className('button-secondary'); + }); + + it('does not add class "button-secondary" by default', () => { + const wrapper = shallow(); + expect(wrapper.find('button')).to.not.have.className('button-secondary'); }); }); diff --git a/spec/javascript/components/display_name.test.jsx b/spec/javascript/components/display_name.test.jsx index c25eb8530..e5a932f4b 100644 --- a/spec/javascript/components/display_name.test.jsx +++ b/spec/javascript/components/display_name.test.jsx @@ -5,15 +5,23 @@ import Immutable from 'immutable'; import DisplayName from '../../../app/assets/javascripts/components/components/display_name' describe('