fix unit test and browser JS warning (mattermost#3087)

mattermost-build · Jul 12, 2019 · 13f9f35 · 13f9f35
1 parent 6a80b99
commit 13f9f35
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/components/channel_selector_modal/channel_selector_modal.test.jsx b/components/channel_selector_modal/channel_selector_modal.test.jsx
@@ -46,7 +46,7 @@ describe('components/ChannelSelectorModal', () => {
  onChannelsSelected: jest.fn(),
  groupID: '',
  actions: {
- loadChannels: jest.fn(() => Promise.resolve({})),
+ loadChannels: jest.fn(() => Promise.resolve({data: []})),
  setModalSearchTerm: jest.fn(() => Promise.resolve()),
  searchChannels: jest.fn(() => Promise.resolve({})),
  },

diff --git a/components/post_view/channel_intro_message/channel_intro_message.jsx b/components/post_view/channel_intro_message/channel_intro_message.jsx
@@ -570,6 +570,7 @@ function createSetHeaderButton(channel) {
  className={'intro-links color--link'}
  dialogType={EditChannelHeaderModal}
  dialogProps={{channel}}
+ modalId={'setChannelHeader'}
  >
  <EditIcon/>
  {message}

diff --git a/components/post_view/post_list/post_list.test.jsx b/components/post_view/post_list/post_list.test.jsx
@@ -13,7 +13,7 @@ const actionsProp = {
  loadPostsAround: jest.fn().mockImplementation(() => Promise.resolve({atLatestMessage: true, atOldestmessage: true})),
  loadUnreads: jest.fn().mockImplementation(() => Promise.resolve({atLatestMessage: true, atOldestmessage: true})),
  loadPosts: jest.fn().mockImplementation(() => Promise.resolve({moreToLoad: false})),
- syncPostsInChannel: jest.fn(),
+ syncPostsInChannel: jest.fn().mockResolvedValue({}),
  loadLatestPosts: jest.fn().mockImplementation(() => Promise.resolve({atLatestMessage: true, atOldestmessage: true})),
  checkAndSetMobileView: jest.fn(),
 };