Fix tests

Author: parasharrajat

tests/actions/IOUTest/DuplicateTest.ts

@@ -7,6 +7,7 @@ import {bulkDuplicateExpenses, duplicateExpenseTransaction, duplicateReport, mer
 import type {DuplicateReportParams} from '@libs/actions/IOU/Duplicate';
 import initOnyxDerivedValues from '@libs/actions/OnyxDerived';
 import {addComment, openReport} from '@libs/actions/Report';
+import type {MergeDuplicatesParams} from '@libs/API/parameters';
 import {WRITE_COMMANDS} from '@libs/API/types';
 import Navigation from '@libs/Navigation/Navigation';
 import {getLoginsByAccountIDs} from '@libs/PersonalDetailsUtils';
@@ -186,7 +187,7 @@ describe('actions/Duplicate', () => {
             await Onyx.set(`${ONYXKEYS.COLLECTION.REPORT_ACTIONS}${childReportID}`, {});
             await waitForBatchedUpdates();
 
-            const mergeParams = {
+            const mergeParams: MergeDuplicatesParams = {
                 transactionID: mainTransactionID,
                 transactionIDList: duplicateTransactionIDs,
                 created: '2024-01-01 12:00:00',
@@ -200,12 +201,10 @@ describe('actions/Duplicate', () => {
                 tag: 'UpdatedProject',
                 receiptID: 123,
                 reportID,
-                currentUserLogin: RORY_EMAIL,
-                currentUserAccountID: RORY_ACCOUNT_ID,
             };
 
             // When: Call mergeDuplicates
-            mergeDuplicates(mergeParams);
+            mergeDuplicates({...mergeParams, currentUserLogin: RORY_EMAIL, currentUserAccountID: RORY_ACCOUNT_ID});
             await waitForBatchedUpdates();
 
             // Then: Verify main transaction was updated
@@ -269,7 +268,7 @@ describe('actions/Duplicate', () => {
             await Onyx.set(`${ONYXKEYS.COLLECTION.REPORT_ACTIONS}${reportID}`, {});
             await waitForBatchedUpdates();
 
-            const mergeParams = {
+            const mergeParams: MergeDuplicatesParams = {
                 transactionID: mainTransactionID,
                 transactionIDList: [],
                 created: '2024-01-01 12:00:00',
@@ -283,12 +282,10 @@ describe('actions/Duplicate', () => {
                 tag: 'UpdatedProject',
                 receiptID: 123,
                 reportID,
-                currentUserLogin: RORY_EMAIL,
-                currentUserAccountID: RORY_ACCOUNT_ID,
             };
 
             // When: Call mergeDuplicates with empty duplicate list
-            mergeDuplicates(mergeParams);
+            mergeDuplicates({...mergeParams, currentUserLogin: RORY_EMAIL, currentUserAccountID: RORY_ACCOUNT_ID});
             await waitForBatchedUpdates();
 
             // Then: Verify main transaction was still updated
@@ -322,7 +319,7 @@ describe('actions/Duplicate', () => {
             await Onyx.set(`${ONYXKEYS.COLLECTION.REPORT_ACTIONS}${reportID}`, {});
             await waitForBatchedUpdates();
 
-            const mergeParams = {
+            const mergeParams: MergeDuplicatesParams = {
                 transactionID: mainTransactionID,
                 transactionIDList: duplicateTransactionIDs,
                 created: '2024-01-01 12:00:00',
@@ -336,12 +333,10 @@ describe('actions/Duplicate', () => {
                 tag: 'UpdatedProject',
                 receiptID: 123,
                 reportID,
-                currentUserLogin: RORY_EMAIL,
-                currentUserAccountID: RORY_ACCOUNT_ID,
             };
 
             // When: Call mergeDuplicates without expense report
-            mergeDuplicates(mergeParams);
+            mergeDuplicates({...mergeParams, currentUserLogin: RORY_EMAIL, currentUserAccountID: RORY_ACCOUNT_ID});
             await waitForBatchedUpdates();
 
             // Then: Verify function completed without errors
@@ -525,7 +520,7 @@ describe('actions/Duplicate', () => {
 
             await waitForBatchedUpdates();
 
-            const mergeParams = {
+            const mergeParams: MergeDuplicatesParams = {
                 transactionID: mainTransactionID,
                 transactionIDList: duplicateTransactionIDs,
                 created: '2024-01-01 12:00:00',
@@ -539,12 +534,10 @@ describe('actions/Duplicate', () => {
                 tag: 'UpdatedProject',
                 receiptID: 123,
                 reportID,
-                currentUserLogin: RORY_EMAIL,
-                currentUserAccountID: RORY_ACCOUNT_ID,
             };
 
             // When: Call mergeDuplicates
-            mergeDuplicates(mergeParams);
+            mergeDuplicates({...mergeParams, currentUserLogin: RORY_EMAIL, currentUserAccountID: RORY_ACCOUNT_ID});
             await waitForBatchedUpdates();
 
             // Then we expect the reportPreview to update with new childVisibleActionCount
@@ -626,7 +619,7 @@ describe('actions/Duplicate', () => {
             });
             await waitForBatchedUpdates();
 
-            const mergeParams = {
+            const mergeParams: MergeDuplicatesParams = {
                 transactionID: mainTransactionID,
                 transactionIDList: duplicateTransactionIDs,
                 transactionThreadReportID: optimisticTransactionThreadReportID,
@@ -641,12 +634,10 @@ describe('actions/Duplicate', () => {
                 tag: 'UpdatedProject',
                 receiptID: 123,
                 reportID,
-                currentUserLogin: RORY_EMAIL,
-                currentUserAccountID: RORY_ACCOUNT_ID,
             };
 
             // When: Call mergeDuplicates with transactionThreadReportID
-            mergeDuplicates(mergeParams);
+            mergeDuplicates({...mergeParams, currentUserLogin: RORY_EMAIL, currentUserAccountID: RORY_ACCOUNT_ID});
             await waitForBatchedUpdates();
 
             // Then: Verify the optimistic transaction thread report was created