Skip to content

Commit

Permalink
Fix deleteUserMultifactor naming
Browse files Browse the repository at this point in the history
  • Loading branch information
luisrudge committed Apr 17, 2018
1 parent 9d7b07b commit 9a0e013
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 2 deletions.
12 changes: 11 additions & 1 deletion src/management/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -998,7 +998,7 @@ utils.wrapPropertyMethod(ManagementClient, 'updateAppMetadata', 'users.updateApp
* @example
* var params = { id: USER_ID, provider: MULTIFACTOR_PROVIDER };
*
* management.deleteUserMultifcator(params, function (err, user) {
* management.deleteUserMultifactor(params, function (err, user) {
* if (err) {
* // Handle error.
* }
Expand All @@ -1013,6 +1013,16 @@ utils.wrapPropertyMethod(ManagementClient, 'updateAppMetadata', 'users.updateApp
*
* @return {Promise|undefined}
*/
utils.wrapPropertyMethod(
ManagementClient,
'deleteUserMultifactor',
'users.deleteMultifactorProvider'
);

/**
* @deprecated The function name has a typo. We're shipping this so it
* doesn't break compatibility. Use {@link deleteUserMultifactor} instead.
*/
utils.wrapPropertyMethod(
ManagementClient,
'deleteUserMultifcator',
Expand Down
4 changes: 3 additions & 1 deletion test/management/management-client.tests.js
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,9 @@ describe('ManagementClient', function() {
'getActiveUsersCount',
'getDailyStats',
'getTenantSettings',
'updateTenantSettings'
'updateTenantSettings',
'deleteUserMultifactor',
'deleteUserMultifcator'
];

before(function() {
Expand Down

0 comments on commit 9a0e013

Please # to comment.