[TS migration][G3] Feedback

Expensify · Mar 8, 2024 · e6cdbd9 · e6cdbd9
1 parent de50342
commit e6cdbd9
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 6 deletions.
diff --git a/tests/unit/SidebarTest.ts b/tests/unit/SidebarTest.ts
@@ -59,11 +59,11 @@ describe('Sidebar', () => {
                 waitForBatchedUpdates()
                     // When Onyx is updated with the data and the sidebar re-renders
                     .then(() => {
-                        const reportCollection = {
+                        const reportCollection: ReportActionCollectionDataSet = {
                             [`${ONYXKEYS.COLLECTION.REPORT}${report.reportID}`]: report,
                         } as ReportCollectionDataSet;
 
-                        const reportAction = {
+                        const reportAction: ReportActionCollectionDataSet = {
                             [`${ONYXKEYS.COLLECTION.REPORT_ACTIONS}${report.reportID}`]: {[action.reportActionID]: action},
                         } as ReportActionCollectionDataSet;
 
@@ -111,11 +111,11 @@ describe('Sidebar', () => {
                 waitForBatchedUpdates()
                     // When Onyx is updated with the data and the sidebar re-renders
                     .then(() => {
-                        const reportCollection = {
+                        const reportCollection: ReportCollectionDataSet = {
                             [`${ONYXKEYS.COLLECTION.REPORT}${report.reportID}`]: report,
                         } as ReportCollectionDataSet;
 
-                        const reportAction = {
+                        const reportAction: ReportActionCollectionDataSet = {
                             [`${ONYXKEYS.COLLECTION.REPORT_ACTIONS}${report.reportID}`]: {[action.reportActionID]: action},
                         } as ReportActionCollectionDataSet;
 

diff --git a/tests/utils/TestHelper.ts b/tests/utils/TestHelper.ts
@@ -1,4 +1,5 @@
 import Str from 'expensify-common/lib/str';
+import type {OnyxUpdate} from 'react-native-onyx';
 import Onyx from 'react-native-onyx';
 import CONST from '@src/CONST';
 import * as Session from '@src/libs/actions/Session';
@@ -18,7 +19,13 @@ type MockFetch = ReturnType<typeof jest.fn> & {
     resume?: () => Promise<void>;
 };
 
-type Response = {ok: boolean; json: () => Promise<{jsonCode: number}>};
+type Response = {
+    ok?: boolean;
+    json?: () => Promise<{jsonCode: number}>;
+    jsonCode?: number;
+    onyxData?: OnyxUpdate[];
+};
+
 type QueueItem = (value: Response | PromiseLike<Response>) => void;
 
 type FormData = {
@@ -51,7 +58,7 @@ function signInWithTestUser(accountID = 1, login = '[email protected]', password = '
 
     HttpUtils.xhr = jest.fn().mockImplementation(() => {
         // Your mocked response object
-        const mockedResponse = {
+        const mockedResponse: Response = {
             onyxData: [
                 {
                     onyxMethod: Onyx.METHOD.MERGE,