Update the logic of handling the username recovery configs.

wso2-extensions · Nov 4, 2024 · a7a13d5 · a7a13d5
1 parent 8d5f4c4
commit a7a13d5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/...ance/src/main/java/org/wso2/carbon/identity/governance/IdentityGovernanceServiceImpl.java b/...ance/src/main/java/org/wso2/carbon/identity/governance/IdentityGovernanceServiceImpl.java
@@ -435,7 +435,11 @@ private void updateUsernameRecoveryPropertyValues(Map<String, String> configurat
 
             if(usernameRecoveryProperty) {
                 configurationDetails.put(USERNAME_RECOVERY_EMAIL_ENABLE,
-                        String.valueOf(usernameRecoveryEmailProperty || StringUtils.isBlank(usernameRecoveryEmailProp)));
+                        String.valueOf(usernameRecoveryEmailProperty ||
+                                StringUtils.isBlank(usernameRecoveryEmailProp)));
+                configurationDetails.put(USERNAME_RECOVERY_SMS_ENABLE,
+                        String.valueOf(usernameRecoverySmsProperty ||
+                                StringUtils.isBlank(usernameRecoverySmsProp)));
             } else if (StringUtils.isBlank(usernameRecoveryProp)) {
                 // Connector is not explicitly enabled or disabled. The connector state is derived from new and existing
                 // configurations.

diff --git a/.../src/test/java/org/wso2/carbon/identity/governance/IdentityGovernanceServiceImplTest.java b/.../src/test/java/org/wso2/carbon/identity/governance/IdentityGovernanceServiceImplTest.java
@@ -175,7 +175,7 @@ public Object[][] buildConfigurations() {
 
         IdentityProviderProperty[] identityProviderProperties10 = getIdentityProviderProperties(
                 false, false, false);
-        Map<String, String> expected10 = getExpectedPropertyValues(true, true, false);
+        Map<String, String> expected10 = getExpectedPropertyValues(true, true, true);
 
         // Only username config false. Preconditions: all the configs true.
         Map<String, String> usernameConfig11 = new HashMap<>();