Fix thinking_face emoji autocomplete (#5238)

remotes/1727458204337373841/tmp_refs/heads/signup-info-prompt
Nolan Lawson 2017-10-06 03:03:13 -07:00 committed by Eugen Rochko
parent 97b3d0cd56
commit 72d939b69f
2 changed files with 17 additions and 7 deletions

View File

@ -125,13 +125,16 @@ function getData(emoji) {
} }
function intersect(a, b) { function intersect(a, b) {
let aSet = new Set(a); let set;
let bSet = new Set(b); let list;
let intersection = new Set( if (a.length < b.length) {
[...aSet].filter(x => bSet.has(x)) set = new Set(a);
); list = b;
} else {
return Array.from(intersection); set = new Set(b);
list = a;
}
return Array.from(new Set(list.filter(x => set.has(x))));
} }
export { getData, getSanitizedData, intersect }; export { getData, getSanitizedData, intersect };

View File

@ -96,4 +96,11 @@ describe('emoji_index', () => {
expect(search('polo').map(trimEmojis)).to.deep.equal(expected); expect(search('polo').map(trimEmojis)).to.deep.equal(expected);
expect(emojiIndex.search('polo').map(trimEmojis)).to.deep.equal(expected); expect(emojiIndex.search('polo').map(trimEmojis)).to.deep.equal(expected);
}); });
it('can search for thinking_face', () => {
let expected = [ { id: 'thinking_face', unified: '1f914', native: '🤔' } ];
expect(search('thinking_fac').map(trimEmojis)).to.deep.equal(expected);
// this is currently broken in emoji-mart
// expect(emojiIndex.search('thinking_fac').map(trimEmojis)).to.deep.equal(expected);
});
}); });