Fix issues with updated emoji mart (#7761)

* Changes behavior from search light to keep custom search
The new version of emoji-mart needs the search function to constantly
receive the custom emoji array. This changes the behavior back to the
previous one in the search light method to keep the emoji autocomplete
as before

* Change test order
The order was breaking the tests

* Fix codeclimate issues

* Update to emoji mart to version without issue in picker

Fixes #7760
This commit is contained in:
Renato "Lond" Cerqueira 2018-06-08 13:28:04 +02:00 committed by Eugen Rochko
parent 6671297e7a
commit 85470ec872
3 changed files with 41 additions and 11 deletions

View File

@ -44,7 +44,12 @@ describe('emoji_index', () => {
expect(emojiIndex.search('apple').map(trimEmojis)).toEqual(expected); expect(emojiIndex.search('apple').map(trimEmojis)).toEqual(expected);
}); });
it('erases custom emoji if not passed again', () => { it('can include/exclude categories', () => {
expect(search('flag', { include: ['people'] })).toEqual([]);
expect(emojiIndex.search('flag', { include: ['people'] })).toEqual([]);
});
it('(different behavior from emoji-mart) do not erases custom emoji if not passed again', () => {
const custom = [ const custom = [
{ {
id: 'mastodon', id: 'mastodon',
@ -60,7 +65,33 @@ describe('emoji_index', () => {
search('', { custom }); search('', { custom });
emojiIndex.search('', { custom }); emojiIndex.search('', { custom });
const expected = []; const expected = [];
expect(search('masto').map(trimEmojis)).toEqual(expected); const lightExpected = [
{
id: 'mastodon',
custom: true,
},
];
expect(search('masto').map(trimEmojis)).toEqual(lightExpected);
expect(emojiIndex.search('masto').map(trimEmojis)).toEqual(expected);
});
it('(different behavior from emoji-mart) erases custom emoji if another is passed', () => {
const custom = [
{
id: 'mastodon',
name: 'mastodon',
short_names: ['mastodon'],
text: '',
emoticons: [],
keywords: ['mastodon'],
imageUrl: 'http://example.com',
custom: true,
},
];
search('', { custom });
emojiIndex.search('', { custom });
const expected = [];
expect(search('masto', { custom: [] }).map(trimEmojis)).toEqual(expected);
expect(emojiIndex.search('masto').map(trimEmojis)).toEqual(expected); expect(emojiIndex.search('masto').map(trimEmojis)).toEqual(expected);
}); });
@ -97,11 +128,6 @@ describe('emoji_index', () => {
.not.toContain('pineapple'); .not.toContain('pineapple');
}); });
it('can include/exclude categories', () => {
expect(search('flag', { include: ['people'] })).toEqual([]);
expect(emojiIndex.search('flag', { include: ['people'] })).toEqual([]);
});
it('does an emoji whose unified name is irregular', () => { it('does an emoji whose unified name is irregular', () => {
const expected = [ const expected = [
{ {

View File

@ -54,9 +54,13 @@ function addCustomToPool(custom, pool) {
index = {}; index = {};
} }
function search(value, { emojisToShowFilter, maxResults, include, exclude, custom = [] } = {}) { function search(value, { emojisToShowFilter, maxResults, include, exclude, custom } = {}) {
if (customEmojisList !== custom) if (custom !== undefined) {
addCustomToPool(custom, originalPool); if (customEmojisList !== custom)
addCustomToPool(custom, originalPool);
} else {
custom = [];
}
maxResults = maxResults || 75; maxResults = maxResults || 75;
include = include || []; include = include || [];

View File

@ -2305,7 +2305,7 @@ elliptic@^6.0.0:
emoji-mart@Gargron/emoji-mart#build: emoji-mart@Gargron/emoji-mart#build:
version "2.6.1" version "2.6.1"
resolved "https://codeload.github.com/Gargron/emoji-mart/tar.gz/227c56c8a1cd89a475d4cf8d9605096555e12484" resolved "https://codeload.github.com/Gargron/emoji-mart/tar.gz/829d042f1c059b15050e5acb3c77adf1a851c666"
emoji-regex@^6.1.0: emoji-regex@^6.1.0:
version "6.5.1" version "6.5.1"