Fix: Update non-reimbursable badge test to use different managerID

After merging main, canApproveIOU now receives currentUserAccountID
explicitly. The test was setting managerID to RORY_ACCOUNT_ID and
passing the same ID as currentUserAccountID, causing canApproveIOU
to return true and the APPROVE badge to appear. Use a different
managerID so the test isolates the non-reimbursable PAY badge skip
behavior.

Co-authored-by: Situ Chandra Shil <situchan@users.noreply.github.com>

Author: MelvinBot

tests/actions/IOUTest/ReportWorkflowTest.ts

@@ -3682,15 +3682,16 @@ describe('actions/IOU/ReportWorkflow', () => {
             // Non-reimbursable-only report: total=0, nonReimbursableTotal=-5000
             // canBePaidNow returns false (REIMBURSEMENT_NO early-exits when onlyShowPayElsewhere=false)
             // canBePaidElsewhere returns true (SUBMITTED status + non-reimbursable spend)
-            // But since all transactions are non-reimbursable, PAY badge should be skipped
+            // But since all transactions are non-reimbursable, PAY badge should be skipped.
+            // Use a different managerID so canApproveIOU also returns false (current user is not the manager).
             const fakeIouReport: Report = {
                 ...createRandomReport(Number(iouReportID), CONST.REPORT.CHAT_TYPE.POLICY_EXPENSE_CHAT),
                 reportID: iouReportID,
                 type: CONST.REPORT.TYPE.EXPENSE,
                 policyID,
                 stateNum: CONST.REPORT.STATE_NUM.SUBMITTED,
                 statusNum: CONST.REPORT.STATUS_NUM.SUBMITTED,
-                managerID: RORY_ACCOUNT_ID,
+                managerID: 999999,
                 total: 0,
                 nonReimbursableTotal: -5000,
                 isWaitingOnBankAccount: false,