Addressed suggestion

Author: shubham1206agra

src/libs/actions/User.ts

@@ -1452,9 +1452,9 @@ function lockAccount(accountID?: number, domainAccountID?: number, domainName?:
     return API.makeRequestWithSideEffects(SIDE_EFFECT_REQUEST_COMMANDS.LOCK_ACCOUNT, params, {optimisticData, successData, failureData});
 }
 
-function requestUnlockAccount(currentAccountIDParam: number, accountID: number | undefined) {
+function requestUnlockAccount(accountID: number) {
     const params: LockAccountParams = {
-        accountID: accountID ?? currentAccountIDParam,
+        accountID,
     };
 
     API.write(WRITE_COMMANDS.REQUEST_UNLOCK_ACCOUNT, params);

src/pages/domain/Members/DomainMemberDetailsPage.tsx

@@ -10,7 +10,6 @@ import MenuItem from '@components/MenuItem';
 import {ModalActions} from '@components/Modal/Global/ModalContext';
 import VacationDelegateMenuItem from '@components/VacationDelegateMenuItem';
 import useConfirmModal from '@hooks/useConfirmModal';
-import useCurrentUserPersonalDetails from '@hooks/useCurrentUserPersonalDetails';
 import {useMemoizedLazyExpensifyIcons} from '@hooks/useLazyAsset';
 import useLocalize from '@hooks/useLocalize';
 import useOnyx from '@hooks/useOnyx';
@@ -37,7 +36,6 @@ function DomainMemberDetailsPage({route}: DomainMemberDetailsPageProps) {
     const {domainAccountID, accountID} = route.params;
     const styles = useThemeStyles();
     const {translate} = useLocalize();
-    const currentUserPersonalDetails = useCurrentUserPersonalDetails();
     const icons = useMemoizedLazyExpensifyIcons(['RemoveMembers', 'Flag', 'Unlock', 'CircularArrowBackwards']);
     const [isModalVisible, setIsModalVisible] = useState(false);
     const [shouldForceCloseAccount, setShouldForceCloseAccount] = useState<boolean>();
@@ -121,7 +119,7 @@ function DomainMemberDetailsPage({route}: DomainMemberDetailsPageProps) {
     );
 
     const showUnlockAccountModal = () => {
-        requestUnlockAccount(currentUserPersonalDetails.accountID, accountID);
+        requestUnlockAccount(accountID);
         showConfirmModal({
             title: translate('lockAccountPage.unlockTitle'),
             prompt: translate('lockAccountPage.unlockDescription'),

src/pages/settings/Security/LockAccount/UnlockAccountPage.tsx

@@ -35,7 +35,7 @@ function UnlockAccountPage() {
                     descriptionStyle={styles.colorMuted}
                     buttonText={translate('unlockAccountPage.chatWithConcierge')}
                     onButtonPress={() => {
-                        requestUnlockAccount(currentUserPersonalDetails.accountID, undefined);
+                        requestUnlockAccount(currentUserPersonalDetails.accountID);
                         Navigation.navigate(ROUTES.CONCIERGE);
                     }}
                     containerStyle={styles.h100}

tests/actions/UserTest.ts

@@ -792,24 +792,12 @@ describe('actions/User', () => {
 
     describe('requestUnlockAccount', () => {
         it('should call API.write with REQUEST_UNLOCK_ACCOUNT command and explicit accountID', async () => {
-            const currentAccountID = 888;
             const accountID = 123456;
-            UserActions.requestUnlockAccount(currentAccountID, accountID);
+            UserActions.requestUnlockAccount(accountID);
             await waitForBatchedUpdates();
 
             expect(mockAPI.write).toHaveBeenCalledWith(WRITE_COMMANDS.REQUEST_UNLOCK_ACCOUNT, {accountID});
         });
-
-        it('should fall back to currentUserAccountID when no accountID is provided', async () => {
-            const currentAccountID = 888;
-            await Onyx.merge(ONYXKEYS.SESSION, {accountID: currentAccountID, email: 'user@expensify.com'});
-            await waitForBatchedUpdates();
-
-            UserActions.requestUnlockAccount(currentAccountID, undefined);
-            await waitForBatchedUpdates();
-
-            expect(mockAPI.write).toHaveBeenCalledWith(WRITE_COMMANDS.REQUEST_UNLOCK_ACCOUNT, {accountID: currentAccountID});
-        });
     });
 
     describe('respondToProactiveAppReview', () => {