Remove erroneous parameter for debounce (mattermost#4408)

* Remove erroneous parameter for debounce * Fixing system_user tests
Revanth47 · Dec 3, 2019 · 91da11a · 91da11a
1 parent c344cc1
commit 91da11a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/components/admin_console/system_users/system_users.jsx b/components/admin_console/system_users/system_users.jsx
@@ -222,7 +222,7 @@ export default class SystemUsers extends React.Component {
  }
 
  this.setState({loading: false});
- }, Constants.SEARCH_TIMEOUT_MILLISECONDS, true);
+ }, Constants.SEARCH_TIMEOUT_MILLISECONDS);
 
  getUserById = async (id) => {
  if (this.props.users[id]) {

diff --git a/components/admin_console/system_users/system_users.test.jsx b/components/admin_console/system_users/system_users.test.jsx
@@ -123,6 +123,7 @@ describe('components/admin_console/system_users', () => {
 
  await wrapper.instance().doSearch('searchterm', '', '');
 
+ jest.runAllTimers();
  expect(searchProfiles).toHaveBeenCalled();
  expect(searchProfiles).toHaveBeenCalledWith('searchterm', {allow_inactive: true});
  });
@@ -138,6 +139,7 @@ describe('components/admin_console/system_users', () => {
 
  await wrapper.instance().doSearch('searchterm', '', 'system_admin');
 
+ jest.runAllTimers();
  expect(searchProfiles).toHaveBeenCalled();
  expect(searchProfiles).toHaveBeenCalledWith('searchterm', {allow_inactive: true, role: 'system_admin'});
  });