fix(Accordion): log error if incorrect activeIndex type

src/modules/Accordion/AccordionAccordion.js

@@ -13,6 +13,19 @@ import {
 } from '../../lib'
 import AccordionPanel from './AccordionPanel'
 
+const warnIfPropsAreInvalid = (props, state) => {
+  const { exclusive } = props
+  const { activeIndex } = state
+
+  /* eslint-disable no-console */
+  if (exclusive && typeof activeIndex !== 'number') {
+    console.error('`activeIndex` must be a number if `exclusive` is true')
+  } else if (!exclusive && !_.isArray(activeIndex)) {
+    console.error('`activeIndex` must be an array if `exclusive` is false')
+  }
+  /* eslint-enable no-console */
+}
+
 /**
  * An Accordion can contain sub-accordions.
  */
@@ -72,6 +85,18 @@ export default class AccordionAccordion extends Component {
     return { activeIndex: exclusive ? -1 : [] }
   }
 
+  componentDidMount() {
+    if (process.env.NODE_ENV !== 'production') {
+      warnIfPropsAreInvalid(this.props, this.state)
+    }
+  }
+
+  componentDidUpdate() {
+    if (process.env.NODE_ENV !== 'production') {
+      warnIfPropsAreInvalid(this.props, this.state)
+    }
+  }
+
   computeNewIndex = (index) => {
     const { exclusive } = this.props
     const { activeIndex } = this.state

test/specs/modules/Accordion/AccordionAccordion-test.js

@@ -4,7 +4,7 @@ import AccordionAccordion from 'src/modules/Accordion/AccordionAccordion'
 import AccordionContent from 'src/modules/Accordion/AccordionContent'
 import AccordionTitle from 'src/modules/Accordion/AccordionTitle'
 import * as common from 'test/specs/commonTests'
-import { sandbox } from 'test/utils'
+import { consoleUtil, sandbox } from 'test/utils'
 
 describe('AccordionAccordion', () => {
   common.isConformant(AccordionAccordion)
@@ -135,6 +135,24 @@ describe('AccordionAccordion', () => {
         .simulate('click')
       wrapper.should.have.state('activeIndex').that.is.empty()
     })
+
+    it('warns if is `exclusive` and is given an array', () => {
+      consoleUtil.disableOnce()
+
+      const consoleError = sandbox.spy(console, 'error')
+      mount(<AccordionAccordion exclusive activeIndex={[1]} />)
+
+      consoleError.should.have.been.calledOnce()
+    })
+
+    it('warns if not `exclusive` and is given a number', () => {
+      consoleUtil.disableOnce()
+
+      const consoleError = sandbox.spy(console, 'error')
+      mount(<AccordionAccordion exclusive={false} activeIndex={1} />)
+
+      consoleError.should.have.been.calledOnce()
+    })
   })
 
   describe('defaultActiveIndex', () => {