Merge pull request #133 from mattermost/test-email

Add config argument to test email action
mattermost · Aug 28, 2020 · b57f1ad · b57f1ad
1 parent 051223b
commit b57f1ad
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/packages/mattermost-redux/src/actions/admin.js b/packages/mattermost-redux/src/actions/admin.js
@@ -58,12 +58,13 @@ export function reloadConfig() {
  );
 }
 
-export function testEmail() {
+export function testEmail(config) {
  return bindClientFunc(
  Client4.testEmail,
  AdminTypes.TEST_EMAIL_REQUEST,
  AdminTypes.TEST_EMAIL_SUCCESS,
- AdminTypes.TEST_EMAIL_FAILURE
+ AdminTypes.TEST_EMAIL_FAILURE,
+ config
  );
 }
 

diff --git a/packages/mattermost-redux/src/client/client4.js b/packages/mattermost-redux/src/client/client4.js
@@ -1087,10 +1087,10 @@ export default class Client4 {
  );
  };
 
- testEmail = async () => {
+ testEmail = async (config) => {
  return this.doFetch(
  `${this.getBaseRoute()}/email/test`,
- {method: 'post'}
+ {method: 'post', body: JSON.stringify(config)}
  );
  };
 

diff --git a/packages/mattermost-redux/test/actions/admin.test.js b/packages/mattermost-redux/test/actions/admin.test.js
@@ -150,7 +150,7 @@ describe('Actions.Admin', () => {
  post('/email/test').
  reply(200, OK_RESPONSE);
 
- await Actions.testEmail()(store.dispatch, store.getState);
+ await Actions.testEmail({})(store.dispatch, store.getState);
 
  const state = store.getState();
  const request = state.requests.admin.testEmail;