Merge pull request Expensify#73530 from Expensify/revert-72984-fix_72212

Revert "fix: Delete option is missing in More menu of an unpaid invoice report"

Author: cristipaval

src/libs/ReportUtils.ts

@@ -2743,7 +2743,7 @@ function canDeleteMoneyRequestReport(report: Report, reportTransactions: Transac
     }
 
     if (isInvoiceReport(report)) {
-        return report?.ownerAccountID === currentUserAccountID && isReportOpenOrProcessing;
+        return report?.ownerAccountID === currentUserAccountID && isReportOpenOrProcessing && policy?.approvalMode !== CONST.POLICY.APPROVAL_MODE.OPTIONAL;
     }
 
     // Users cannot delete a report in the unreported or IOU cases, but they can delete individual transactions.

tests/unit/ReportUtilsTest.ts

@@ -26,7 +26,6 @@ import {
     buildTransactionThread,
     canAddTransaction,
     canCreateRequest,
-    canDeleteMoneyRequestReport,
     canDeleteReportAction,
     canDeleteTransaction,
     canEditMoneyRequest,
@@ -3053,13 +3052,6 @@ describe('ReportUtils', () => {
         });
     });
 
-    describe('canDeleteMoneyRequestReport', () => {
-        it('should allow deletion if the report is open invoice report', () => {
-            const invoiceReport = {...createInvoiceReport(343), ownerAccountID: currentUserAccountID, stateNum: CONST.REPORT.STATE_NUM.OPEN, statusNum: CONST.REPORT.STATUS_NUM.OPEN};
-            expect(canDeleteMoneyRequestReport(invoiceReport, [], [])).toBe(true);
-        });
-    });
-
     describe('shouldDisableThread', () => {
         const reportID = '1';