From cff8517bd3fae16545ba6cddb04f16c37780218a Mon Sep 17 00:00:00 2001
From: DD <liudi08@gmail.com>
Date: Fri, 23 Feb 2018 16:04:16 -0500
Subject: [PATCH] fix tests
---
src/components/sprite-selector/sprite-selector.jsx | 6 +++---
src/containers/sprite-selector-item.jsx | 2 +-
test/unit/containers/sprite-selector-item.test.jsx | 5 ++++-
3 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/src/components/sprite-selector/sprite-selector.jsx b/src/components/sprite-selector/sprite-selector.jsx
index 906479d71..fdec937a9 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 4b81507a0..d1a3e4b3f 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 2a371a1a0..a423fe816 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);
--
GitLab