[MM-14278] Update unit tests for emoji actions (#2433)

* [MM-14278] Display same recent emoji only once

* A recent change had recent emojis stored by alias, resulting in identical emoji such as 👍 or 👍 displayed twice
* This reverts that change to store by the first alias only, de-duplicating the recent history
* Additinally, `emoji.id` was undefined, using `emojiIndex` instead

* [MM-14278] Update unit tests for emoji actions

actions/emoji_actions.test.js

@@ -13,9 +13,17 @@ describe('Actions.Emojis', () => {
  store = await configureStore();
  });
 
- it('addRecentEmoji', async () => {
+ test('Emoji alias is stored in recent emojis', async () => {
  store.dispatch(Actions.addRecentEmoji('grinning'));
  assert.ok(getRecentEmojis(store.getState()).includes('grinning'));
+ });
+
+ test('First alias is stored in recent emojis even if second alias used', async () => {
+ store.dispatch(Actions.addRecentEmoji('cop'));
+ assert.ok(getRecentEmojis(store.getState()).includes('policeman'));
+ });
+
+ test('Invalid emoji are not stored in recents', async () => {
  store.dispatch(Actions.addRecentEmoji('joramwilander'));
  assert.ok(!getRecentEmojis(store.getState()).includes('joramwilander'));
  });