From b5e9794232e25314032e8c78914ac631486f0756 Mon Sep 17 00:00:00 2001
From: Karishma Chadha <kchadha@media.mit.edu>
Date: Wed, 10 Oct 2018 19:01:42 -0400
Subject: [PATCH] Fix unit tests

---
 src/lib/get-costume-url.js                         |  5 ++++-
 test/unit/containers/sprite-selector-item.test.jsx |  9 ---------
 test/unit/util/get-costume-url.test.js             | 11 +++++++++++
 3 files changed, 15 insertions(+), 10 deletions(-)
 create mode 100644 test/unit/util/get-costume-url.test.js

diff --git a/src/lib/get-costume-url.js b/src/lib/get-costume-url.js
index b4d779d04..7d12edf80 100644
--- a/src/lib/get-costume-url.js
+++ b/src/lib/get-costume-url.js
@@ -25,4 +25,7 @@ const getCostumeUrl = function (assetId, vm) {
     return vm.runtime.storage.get(assetId).encodeDataURI();
 };
 
-export default getCostumeUrl;
+export {
+    getCostumeUrl as default,
+    HAS_FONT_REGEXP
+};
diff --git a/test/unit/containers/sprite-selector-item.test.jsx b/test/unit/containers/sprite-selector-item.test.jsx
index 25200a1fb..f133c02a2 100644
--- a/test/unit/containers/sprite-selector-item.test.jsx
+++ b/test/unit/containers/sprite-selector-item.test.jsx
@@ -4,7 +4,6 @@ import configureStore from 'redux-mock-store';
 import {Provider} from 'react-redux';
 
 import SpriteSelectorItem from '../../../src/containers/sprite-selector-item';
-import {HAS_FONT_REGEXP} from '../../../src/containers/sprite-selector-item';
 import CloseButton from '../../../src/components/close-button/close-button';
 
 describe('SpriteSelectorItem Container', () => {
@@ -56,12 +55,4 @@ describe('SpriteSelectorItem Container', () => {
         wrapper.find(CloseButton).simulate('click');
         expect(onDeleteButtonClick).toHaveBeenCalledWith(1337);
     });
-
-    test('Has font regexp works', () => {
-        expect('font-family="Sans Serif"'.match(HAS_FONT_REGEXP)).toBeTruthy();
-        expect('font-family="none" font-family="Sans Serif"'.match(HAS_FONT_REGEXP)).toBeTruthy();
-        expect('font-family = "Sans Serif"'.match(HAS_FONT_REGEXP)).toBeTruthy();
-
-        expect('font-family="none"'.match(HAS_FONT_REGEXP)).toBeFalsy();
-    });
 });
diff --git a/test/unit/util/get-costume-url.test.js b/test/unit/util/get-costume-url.test.js
new file mode 100644
index 000000000..93154acca
--- /dev/null
+++ b/test/unit/util/get-costume-url.test.js
@@ -0,0 +1,11 @@
+import {HAS_FONT_REGEXP} from '../../../src/lib/get-costume-url';
+
+describe('SVG Font Parsing', () => {
+    test('Has font regexp works', () => {
+        expect('font-family="Sans Serif"'.match(HAS_FONT_REGEXP)).toBeTruthy();
+        expect('font-family="none" font-family="Sans Serif"'.match(HAS_FONT_REGEXP)).toBeTruthy();
+        expect('font-family = "Sans Serif"'.match(HAS_FONT_REGEXP)).toBeTruthy();
+
+        expect('font-family="none"'.match(HAS_FONT_REGEXP)).toBeFalsy();
+    });
+});
-- 
GitLab