fix: Hmm it's not here in RHP is displayed after deleting a workspace

src/libs/PolicyUtils.ts

@@ -62,6 +62,14 @@ Onyx.connect({
     callback: (value) => (allPolicies = value),
 });
 
+let currentUserEmail: string;
+Onyx.connect({
+    key: ONYXKEYS.SESSION,
+    callback: (val) => {
+        currentUserEmail = val?.email ?? '';
+    },
+});
+
 Onyx.connect({
     key: ONYXKEYS.NVP_ACTIVE_POLICY_ID,
     callback: (value) => (activePolicyId = value),
@@ -71,9 +79,15 @@ Onyx.connect({
  * Filter out the active policies, which will exclude policies with pending deletion
  * These are policies that we can use to create reports with in NewDot.
  */
-function getActivePolicies(policies: OnyxCollection<Policy> | null): Policy[] {
+function getActivePolicies(policies: OnyxCollection<Policy> | null, excludePersonalPolicy = false): Policy[] {
     return Object.values(policies ?? {}).filter<Policy>(
-        (policy): policy is Policy => !!policy && policy.pendingAction !== CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE && !!policy.name && !!policy.id,
+        (policy): policy is Policy =>
+            !!policy &&
+            policy.pendingAction !== CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE &&
+            !!policy.name &&
+            !!policy.id &&
+            (!excludePersonalPolicy || policy.type !== CONST.POLICY.TYPE.PERSONAL) &&
+            !!getPolicyRole(policy, currentUserEmail),
     );
 }
 /**

src/libs/ReportUtils.ts

@@ -8216,9 +8216,7 @@ function createDraftTransactionAndNavigateToParticipantSelector(transactionID: s
         mccGroup,
     } as Transaction);
 
-    const filteredPolicies = Object.values(allPolicies ?? {}).filter(
-        (policy) => policy && policy.type !== CONST.POLICY.TYPE.PERSONAL && policy.pendingAction !== CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE,
-    );
+    const filteredPolicies = PolicyUtils.getActivePolicies(allPolicies, true);
 
     if (actionName === CONST.IOU.ACTION.CATEGORIZE) {
         const activePolicy = getPolicy(activePolicyID);

src/pages/workspace/WorkspaceNewRoomPage.tsx

@@ -64,8 +64,7 @@ function WorkspaceNewRoomPage() {
 
     const workspaceOptions = useMemo(
         () =>
-            PolicyUtils.getActivePolicies(policies)
-                ?.filter((policy) => policy.type !== CONST.POLICY.TYPE.PERSONAL)
+            PolicyUtils.getActivePolicies(policies, true)
                 .map((policy) => ({
                     label: policy.name,
                     value: policy.id,

tests/unit/PolicyUtilsTest.ts

@@ -1,5 +1,15 @@
+import Onyx from 'react-native-onyx';
+import type {OnyxCollection} from 'react-native-onyx';
 import * as PolicyUtils from '@libs/PolicyUtils';
+import CONST from '@src/CONST';
+import ONYXKEYS from '@src/ONYXKEYS';
+import type {Policy} from '@src/types/onyx';
+import createRandomPolicy from '../utils/collections/policies';
+import * as TestHelper from '../utils/TestHelper';
+import waitForBatchedUpdates from '../utils/waitForBatchedUpdates';
 
+const CARLOS_EMAIL = '[email protected]';
+const CARLOS_ACCOUNT_ID = 1;
 function toLocaleDigitMock(dot: string): string {
     return dot;
 }
@@ -81,4 +91,57 @@ describe('PolicyUtils', () => {
             });
         });
     });
+    describe('getActivePolicies', () => {
+        beforeAll(() => {
+            Onyx.init({
+                keys: ONYXKEYS,
+                initialKeyStates: {
+                    // Given mock data for session
+                    [ONYXKEYS.SESSION]: {accountID: CARLOS_ACCOUNT_ID, email: CARLOS_EMAIL},
+                },
+            });
+        });
+
+        beforeEach(() => {
+            global.fetch = TestHelper.getGlobalFetchMock();
+            return Onyx.clear().then(waitForBatchedUpdates);
+        });
+        it('should return empty array', () => {
+            // Given mock data for policies, contains only one workspace does not has role prop.
+            const policies = {
+                // eslint-disable-next-line @typescript-eslint/naming-convention
+                1: {
+                    ...createRandomPolicy(1, CONST.POLICY.TYPE.CORPORATE),
+                    role: '',
+                },
+            };
+            const result = PolicyUtils.getActivePolicies(policies as OnyxCollection<Policy>);
+            // The result should be empty array since the policies contains only one policy which does not have role field.
+            expect(result.length).toBe(0);
+        });
+        it('should return array contains policy which has id = 1', () => {
+            // Given mock data for policies, contains only one control workspace
+            const randomPolicy1 = {...createRandomPolicy(1, CONST.POLICY.TYPE.CORPORATE), pendingAction: null};
+            const policies = {
+                // eslint-disable-next-line @typescript-eslint/naming-convention
+                1: randomPolicy1,
+            };
+            const result = PolicyUtils.getActivePolicies(policies as OnyxCollection<Policy>);
+            // The result should contains the policy which has id = 1 since it is a valid policy.
+            expect(result).toContainEqual(randomPolicy1);
+        });
+        it('should return empty array', () => {
+            // Given mock data for policies, contains only one control workspace which is pending delete.
+            const policies = {
+                // eslint-disable-next-line @typescript-eslint/naming-convention
+                1: {
+                    ...createRandomPolicy(1, CONST.POLICY.TYPE.CORPORATE),
+                    pendingAction: CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE,
+                },
+            };
+            const result = PolicyUtils.getActivePolicies(policies as OnyxCollection<Policy>);
+            // The result should be empty array since the policies contains only one policy which has pendingAction is 'delete'.
+            expect(result).toEqual([]);
+        });
+    });
 });