remove redundant test case

Author: dominictb

src/libs/ReportSecondaryActionUtils.ts

@@ -546,14 +546,6 @@ function isRemoveHoldAction(report: Report, chatReport: OnyxEntry<Report>, repor
     return policy?.role === CONST.POLICY.ROLE.ADMIN;
 }
 
-function isRemoveHoldActionForTransaction(report: Report, reportTransaction: Transaction, policy?: Policy): boolean {
-    if (!isOnHoldTransactionUtils(reportTransaction)) {
-        return false;
-    }
-
-    return policy?.role === CONST.POLICY.ROLE.ADMIN;
-}
-
 function getSecondaryReportActions({
     report,
     chatReport,

tests/unit/ReportSecondaryActionUtilsTest.ts

@@ -1247,7 +1247,7 @@ describe('getSecondaryExportReportActions', () => {
         expect(result.includes(CONST.REPORT.EXPORT_OPTIONS.MARK_AS_EXPORTED)).toBe(true);
     });
 
-    it('includes REMOVE HOLD option for admin', () => {
+    it('includes REMOVE HOLD option for admin in expense report but not in transaction thread report', () => {
         const report = {} as unknown as Report;
         const policy = {
             role: CONST.POLICY.ROLE.ADMIN,
@@ -1310,21 +1310,6 @@ describe('getSecondaryTransactionThreadActions', () => {
         expect(result.includes(CONST.REPORT.SECONDARY_ACTIONS.HOLD)).toBe(true);
     });
 
-    it('includes REMOVE HOLD option for admin', () => {
-        const report = {} as unknown as Report;
-        const policy = {
-            role: CONST.POLICY.ROLE.ADMIN,
-        } as unknown as Policy;
-        const transaction = {
-            comment: {
-                hold: 'REPORT_ACTION_ID',
-            },
-        } as unknown as Transaction;
-
-        const result = getSecondaryTransactionThreadActions(report, transaction, [], policy);
-        expect(result).toContain(CONST.REPORT.TRANSACTION_SECONDARY_ACTIONS.REMOVE_HOLD);
-    });
-
     it('includes DELETE option for expense report submitter', async () => {
         const report = {
             reportID: REPORT_ID,