fix errors and warnings when running unit tests (mattermost#2909)

mattermost-build · Jun 6, 2019 · 76601d4 · 76601d4
1 parent 5dd2b25
commit 76601d4
Show file tree

Hide file tree

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

diff --git a/components/integrations/bots/bots.test.jsx b/components/integrations/bots/bots.test.jsx
@@ -13,7 +13,7 @@ import Bots from './bots.jsx';
 describe('components/integrations/bots/Bots', () => {
  const team = TestHelper.fakeTeam();
  const actions = {
- loadBots: jest.fn().mockReturnValue(Promise.resolve()),
+ loadBots: jest.fn().mockReturnValue(Promise.resolve({})),
  getUserAccessTokensForUser: jest.fn(),
  createUserAccessToken: jest.fn(),
  revokeUserAccessToken: jest.fn(),

diff --git a/components/list_modal.jsx b/components/list_modal.jsx
@@ -139,8 +139,8 @@ export default class ListModal extends React.PureComponent {
 
  onPageChange = async (page) => {
  this.setState({loading: true});
- const items = await this.props.loadItems(page, this.state.searchTerm);
- this.setState({page, items, loading: false});
+ const result = await this.props.loadItems(page, this.state.searchTerm);
+ this.setState({page, items: result.items, loading: false});
  }
 
  onSearchInput = async (event) => {
@@ -245,4 +245,4 @@ export default class ListModal extends React.PureComponent {
  </div >
  );
  }
-}
+}
diff --git a/components/permalink_view/permalink_view.test.jsx b/components/permalink_view/permalink_view.test.jsx
@@ -129,7 +129,7 @@ describe('components/PermalinkView', () => {
  },
  };
 
- describe('focusPost', async () => {
+ describe('focusPost', () => {
  test('should focus post in already loaded channel', async () => {
  const testStore = await mockStore(initialState);
  await testStore.dispatch(focusPost('postid1'));