allow ModifiedExpenseContent to subscribe to policy

Author: LukasMod

src/pages/inbox/report/PureReportActionItem.tsx

@@ -1158,7 +1158,6 @@ function PureReportActionItem({
                     report={report}
                     childReport={childReport}
                     originalReport={originalReport}
-                    policy={policy}
                     currentUserEmail={currentUserEmail}
                 />
             );

src/pages/inbox/report/actionContents/ModifiedExpenseContent.tsx

@@ -7,20 +7,20 @@ import {getForReportAction, getMovedReportID} from '@libs/ModifiedExpenseMessage
 import ReportActionItemMessageWithExplain from '@pages/inbox/report/ReportActionItemMessageWithExplain';
 import CONST from '@src/CONST';
 import ONYXKEYS from '@src/ONYXKEYS';
-import type {Policy, Report, ReportAction} from '@src/types/onyx';
+import type {Report, ReportAction} from '@src/types/onyx';
 
 type ModifiedExpenseContentProps = {
     action: ReportAction;
     report: OnyxEntry<Report>;
     childReport: OnyxEntry<Report>;
     originalReport: OnyxEntry<Report>;
-    policy: OnyxEntry<Policy>;
     currentUserEmail: string | undefined;
 };
 
-function ModifiedExpenseContent({action, report, childReport, originalReport, policy, currentUserEmail}: ModifiedExpenseContentProps) {
+function ModifiedExpenseContent({action, report, childReport, originalReport, currentUserEmail}: ModifiedExpenseContentProps) {
     const {translate} = useLocalize();
     const {policyForMovingExpensesID} = usePolicyForMovingExpenses();
+    const [policy] = useOnyx(`${ONYXKEYS.COLLECTION.POLICY}${report?.policyID}`);
 
     // When expense is moved from self-DM to workspace, policyID is temporarily OWNER_EMAIL_FAKE.
     // Fall back to policyForMovingExpensesID (actual destination workspace) for correct tag list.

tests/ui/PureReportActionItemTest.tsx

@@ -779,6 +779,14 @@ describe('PureReportActionItem', () => {
         it('MODIFIED_EXPENSE with policyRulesModifiedFields renders billable message and workspace rules link', async () => {
             const policyID = 'policy123';
 
+            // Set up policy in Onyx so ModifiedExpenseContent self-subscribes to it
+            await act(async () => {
+                await Onyx.merge(`${ONYXKEYS.COLLECTION.POLICY}${policyID}`, {
+                    id: policyID,
+                    name: 'Test Policy',
+                });
+            });
+
             const action = createReportAction(CONST.REPORT.ACTIONS.TYPE.MODIFIED_EXPENSE, {
                 policyID,
                 policyRulesModifiedFields: {billable: true},
@@ -812,7 +820,6 @@ describe('PureReportActionItem', () => {
                                     betas={undefined}
                                     draftTransactionIDs={[]}
                                     userBillingGracePeriodEnds={undefined}
-                                    policy={{id: policyID} as Policy}
                                 />
                             </PortalProvider>
                         </ScreenWrapper>