Merge pull request #154 from mwadden/master

Bug Fix: Ensure category ids are not dropped after emojisToShowFilter call
missive · Dec 28, 2017 · 9675658 · 9675658
2 parents 048838c + 3c924a1
commit 9675658
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/spec/picker-spec.js b/spec/picker-spec.js
@@ -33,5 +33,11 @@ describe('Picker', () => {
       subject = render({ emojisToShowFilter: unified => false })
       expect(subject.categories.length).toEqual(2)
     })
+
+    it('maintains category ids after it is filtered', () => {
+      subject = render({emojisToShowFilter: emoji => true});
+      const categoriesWithIds = subject.categories.filter(category => category.id);
+      expect(categoriesWithIds.length).toEqual(10);
+    });
   })
 })
diff --git a/src/components/picker.js b/src/components/picker.js
@@ -108,6 +108,7 @@ export default class Picker extends React.PureComponent {
           let newCategory = {
             emojis: newEmojis,
             name: category.name,
+            id: category.id
           }
 
           this.categories.push(newCategory)