Uncomment PR #43 specs

missive · Feb 17, 2017 · 5c27f17 · 5c27f17
1 parent 00fdf61
commit 5c27f17
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 18 deletions.
diff --git a/src/components/pickerSpec.js b/src/components/pickerSpec.js
@@ -20,18 +20,17 @@ describe('Picker', () => {
     expect(subject).toBeDefined();
   });
 
-  // This is the unit test for pull request (https://github.com/missive/emoji-mart/pull/43)
-  // describe('categories', () => {
-  //   it('shows 10 by default', () => {
-  //     subject = render();
-  //     expect(subject.categories.length).toEqual(10);
-  //   });
-  //
-  //   it('will not show some based upon our filter', () => {
-  //     subject = render({emojisToShowFilter: (unified) => false});
-  //     expect(subject.categories.length).toEqual(2);
-  //   });
-  // });
+  describe('categories', () => {
+    it('shows 10 by default', () => {
+      subject = render();
+      expect(subject.categories.length).toEqual(10);
+    });
+
+    it('will not show some based upon our filter', () => {
+      subject = render({emojisToShowFilter: (unified) => false});
+      expect(subject.categories.length).toEqual(2);
+    });
+  });
 
   function render(props = {}) {
     const defaultProps = {

diff --git a/src/utils/emoji-indexSpec.js b/src/utils/emoji-indexSpec.js
@@ -13,11 +13,10 @@ describe('#emojiIndex', () => {
       }]);
     });
 
-    // This is the unit test for pull request (https://github.com/missive/emoji-mart/pull/43)
-    // it('should filter only emojis we care about, exclude pineapple', () => {
-    //   let emojisToShowFilter = (unified) => unified !== '1F34D';
-    //   expect(emojiIndex.search('apple', emojisToShowFilter).map((obj) => obj.id))
-    //     .not.toContain('pineapple');
-    // });
+    it('should filter only emojis we care about, exclude pineapple', () => {
+      let emojisToShowFilter = (unified) => unified !== '1F34D';
+      expect(emojiIndex.search('apple', emojisToShowFilter).map((obj) => obj.id))
+        .not.toContain('pineapple');
+    });
   });
 });