diff --git a/src/components/sprite-selector/sprite-selector.jsx b/src/components/sprite-selector/sprite-selector.jsx index 906479d717f9ef2efeb386427f6810f727235499..fdec937a9fdd146e919ade7dc6d56319ac99567f 100644 --- a/src/components/sprite-selector/sprite-selector.jsx +++ b/src/components/sprite-selector/sprite-selector.jsx @@ -79,9 +79,9 @@ const SpriteSelectorComponent = function (props) { className={hoveredTarget.sprite === sprite.id && sprite.id !== editingTarget && hoveredTarget.receivedBlocks ? - classNames(styles.sprite, styles.receivedBlocks) : - raised && sprite.id !== editingTarget ? - classNames(styles.sprite, styles.raised) : styles.sprite} + classNames(styles.sprite, styles.receivedBlocks) : + raised && sprite.id !== editingTarget ? + classNames(styles.sprite, styles.raised) : styles.sprite} id={sprite.id} key={sprite.id} name={sprite.name} diff --git a/src/containers/sprite-selector-item.jsx b/src/containers/sprite-selector-item.jsx index 4b81507a0b656caf01301d443680cd0aea738803..d1a3e4b3fac0a29cc74fafbdd8d19dd799b2a534 100644 --- a/src/containers/sprite-selector-item.jsx +++ b/src/containers/sprite-selector-item.jsx @@ -73,7 +73,7 @@ SpriteSelectorItem.propTypes = { onClick: PropTypes.func, onDeleteButtonClick: PropTypes.func, onDuplicateButtonClick: PropTypes.func, - receivedBlocks: PropTypes.bool, + receivedBlocks: PropTypes.bool.isRequired, selected: PropTypes.bool }; diff --git a/test/unit/containers/sprite-selector-item.test.jsx b/test/unit/containers/sprite-selector-item.test.jsx index 2a371a1a0dfe763c32737ee832a9a3009e57b0e7..a423fe81635a3685b6521a73f12824f3a5b380e0 100644 --- a/test/unit/containers/sprite-selector-item.test.jsx +++ b/test/unit/containers/sprite-selector-item.test.jsx @@ -12,6 +12,7 @@ describe('SpriteSelectorItem Container', () => { let costumeURL; let name; let onClick; + let dispatchSetHoveredSprite; let onDeleteButtonClick; let selected; let id; @@ -23,6 +24,7 @@ describe('SpriteSelectorItem Container', () => { <SpriteSelectorItem className={className} costumeURL={costumeURL} + dispatchSetHoveredSprite={dispatchSetHoveredSprite} id={id} name={name} selected={selected} @@ -34,13 +36,14 @@ describe('SpriteSelectorItem Container', () => { }; beforeEach(() => { - store = mockStore(); + store = mockStore({hoveredTarget: {receivedBlocks: false, sprite: null}}); className = 'ponies'; costumeURL = 'https://scratch.mit.edu/foo/bar/pony'; id = 1337; name = 'Pony sprite'; onClick = jest.fn(); onDeleteButtonClick = jest.fn(); + dispatchSetHoveredSprite = jest.fn(); selected = true; // Mock window.confirm() which is called when the close button is clicked. global.confirm = jest.fn(() => true);