[Due for payment 2025-11-19] [$250] Fix flaky ReportUtils › getReportActionActorAccountID test

[github-actions]

🚨 Failure Summary 🚨:

• 📋 Job Name: test / test (job 1)
• 🔧 Failure in Workflow: Process new code merged to main
• 🔗 Triggered by PR: PR Link
• 👤 PR Author: @kstroz
• 🤝 Merged by: @mountiny
• 🐛 Error Message:
  failure: Process completed with exit code 1.

FAIL tests/unit/ReportUtilsTest.ts (167.791 s)
  ● ReportUtils › getReportActionActorAccountID › should return report manager account id if action is REPORTPREVIEW and report is not a policy expense chat

    expect(received).toEqual(expected) // deep equality

    Expected: 20
    Received: 10

      7157 |             };
      7158 |             const actorAccountID = getReportActionActorAccountID(reportAction, iouReport, report);
    > 7159 |             expect(actorAccountID).toEqual(20);
           |                                    ^
      7160 |         });
      7161 |
      7162 |         it('should return admin account id if action is SUBMITTED taken by an admin on behalf the submitter', () => {

      at Object.toEqual (tests/unit/ReportUtilsTest.ts:7159:36)

⚠️ Action Required ⚠️:

🛠️ A recent merge appears to have caused a failure in the job named test / test (job 1).
This issue has been automatically created and labeled with Workflow Failure for investigation.

👀 Please look into the following:

1. Why the PR caused the job to fail?
2. Address any underlying issues.

🐛 We appreciate your help in squashing this bug!

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021973046764328218376
• Upwork Job ID: 1973046764328218376
• Last Price Increase: 2025-11-19
• Automatic offers:
• nkdengineer | Contributor | 108957961

Issue Owner

Current Issue Owner: @mallenexpensify

[nkdengineer]

Proposal

Please re-state the problem that we are trying to solve in this issue.

Investigate workflow job failing on main: test / test (job 1)

What is the root cause of that problem?

We don't set the chat type then if it's randomized to policyExpenseChat type, the test fails

App/tests/unit/ReportUtilsTest.ts

Lines 7174 to 7180 in a426bbf

	const report: Report = {
	...createRandomReport(1),
	type: CONST.REPORT.TYPE.CHAT,
	};
	const actorAccountID = getReportActionActorAccountID(reportAction, iouReport, report);
	expect(actorAccountID).toEqual(20);
	});

What changes do you think we should make in order to solve the problem?

Add the chatType that is not policyExpenseChat here

chatType: undefined

App/tests/unit/ReportUtilsTest.ts

Lines 7174 to 7180 in a426bbf

	const report: Report = {
	...createRandomReport(1),
	type: CONST.REPORT.TYPE.CHAT,
	};
	const actorAccountID = getReportActionActorAccountID(reportAction, iouReport, report);
	expect(actorAccountID).toEqual(20);
	});

What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?

None

What alternative solutions did you explore? (Optional)

Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~021973046764328218376

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @brunovjk (External)

[melvin-bot]

⚠️ Failed to update price automatically. The BZ team member will need to update the price manually in Upwork.