Skip to content

Commit 016552e

Browse files
author
Sarah Allen
committed
Fix connected component to use renamed action
1 parent d481b82 commit 016552e

File tree

2 files changed

+13
-13
lines changed

2 files changed

+13
-13
lines changed

packages/lib-classifier/src/components/Classifier/components/FieldGuide/components/FieldGuide/components/FieldGuideItems/FieldGuideItemAnchor.js

+6-6
Original file line numberDiff line numberDiff line change
@@ -43,24 +43,24 @@ export function AnchorLabel({ className, icons, item }) {
4343
}
4444

4545
function storeMapper(stores) {
46-
const { attachedMedia: icons, setActiveItem } = stores.classifierStore.fieldGuide
46+
const { attachedMedia: icons, setActiveItemIndex } = stores.classifierStore.fieldGuide
4747
return {
4848
icons,
49-
setActiveItem,
49+
setActiveItemIndex,
5050
}
5151
}
5252

5353
@inject(storeMapper)
5454
@observer
5555
class FieldGuideItemAnchor extends React.Component {
5656
onClick (event, item) {
57-
const { setActiveItem } = this.props
57+
const { setActiveItemIndex } = this.props
5858
event.preventDefault()
59-
setActiveItem(item)
59+
setActiveItemIndex(item)
6060
}
6161

6262
render () {
63-
const { className, icons, row, setActiveItem } = this.props
63+
const { className, icons, row } = this.props
6464

6565
return (
6666
row.map((item) => {
@@ -90,7 +90,7 @@ FieldGuideItemAnchor.wrappedComponent.propTypes = {
9090
className: PropTypes.string,
9191
icons: MobXPropTypes.observableMap,
9292
row: PropTypes.arrayOf(PropTypes.object).isRequired,
93-
setActiveItem: PropTypes.func.isRequired
93+
setActiveItemIndex: PropTypes.func.isRequired
9494
}
9595

9696
export default FieldGuideItemAnchor

packages/lib-classifier/src/components/Classifier/components/FieldGuide/components/FieldGuide/components/FieldGuideItems/FieldGuideItemAnchor.spec.js

+7-7
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ describe('Component > FieldGuideItemAnchor', function () {
2121
<FieldGuideItemAnchor.wrappedComponent
2222
icons={attachedMedia}
2323
row={row}
24-
setActiveItem={() => {}}
24+
setActiveItemIndex={() => {}}
2525
/>)
2626
expect(wrapper).to.be.ok
2727
})
@@ -31,7 +31,7 @@ describe('Component > FieldGuideItemAnchor', function () {
3131
<FieldGuideItemAnchor.wrappedComponent
3232
icons={attachedMedia}
3333
row={row}
34-
setActiveItem={() => { }}
34+
setActiveItemIndex={() => { }}
3535
/>)
3636
expect(wrapper).to.have.lengthOf(row.length)
3737
})
@@ -41,22 +41,22 @@ describe('Component > FieldGuideItemAnchor', function () {
4141
<FieldGuideItemAnchor.wrappedComponent
4242
icons={attachedMedia}
4343
row={row}
44-
setActiveItem={() => { }}
44+
setActiveItemIndex={() => { }}
4545
/>)
4646
expect(wrapper.props().label.type).to.equal(AnchorLabel)
4747
})
4848

49-
it('should call setActiveItem on click', function () {
50-
const setActiveItemSpy = sinon.spy()
49+
it('should call setActiveItemIndex on click', function () {
50+
const setActiveItemIndexSpy = sinon.spy()
5151
const wrapper = shallow(
5252
<FieldGuideItemAnchor.wrappedComponent
5353
icons={attachedMedia}
5454
row={row}
55-
setActiveItem={setActiveItemSpy}
55+
setActiveItemIndex={setActiveItemIndexSpy}
5656
/>)
5757

5858
wrapper.simulate('click', { preventDefault: () => {} })
59-
expect(setActiveItemSpy).to.have.been.calledOnceWith(row[0])
59+
expect(setActiveItemIndexSpy).to.have.been.calledOnceWith(row[0])
6060
})
6161

6262
describe('Component > AnchorLabel', function () {

0 commit comments

Comments
 (0)