[No QA] [TS migration] Migrate 'E2E' lib to TypeScript

metro.config.js

@@ -27,8 +27,8 @@ const config = {
             if (isUsingMockAPI && moduleName.includes('/API')) {
                 return {
                     ...resolution,
-                    // TODO: Change API.mock.js extension once it is migrated to TypeScript
-                    filePath: resolution.filePath.replace(/src\/libs\/API.js/, 'src/libs/E2E/API.mock.js'),
+                    // TODO: Change API.js extension once it is migrated to TypeScript
+                    filePath: resolution.filePath.replace(/src\/libs\/API.js/, 'src/libs/E2E/API.mock.ts'),
                 };
             }
             return resolution;

src/CONST.ts

@@ -240,6 +240,76 @@ const CONST = {
         CUSTOM_STATUS: 'customStatus',
         NEW_DOT_CATEGORIES: 'newDotCategories',
         NEW_DOT_TAGS: 'newDotTags',
+        PDF_META_STORE: 'pdfMetaStore',
+        REPORT_ACTION_CONTEXT_MENU: 'reportActionContextMenu',
+        SUBMIT_POLICY: 'submitPolicy',
+        ATTENDEES: 'attendees',
+        AUTO_EXPORT: 'autoExport',
+        AUTO_EXPORT_INTACCT: 'autoExportIntacct',
+        AUTO_EXPORT_QBO: 'autoExportQbo',
+        AUTO_EXPORT_XERO: 'autoExportXero',
+        AUTO_JOIN_POLICY: 'autoJoinPolicy',
+        AUTOMATED_TAX_EXEMPTION: 'automatedTaxExemption',
+        BILL_PAY: 'billPay',
+        CATEGORY_DEFAULT_TAX: 'categoryDefaultTax',
+        COLLECTABLE_DEPOSIT_ACCOUNTS: 'collectableDepositAccounts',
+        CONCIERGE_TRAVEL: 'conciergeTravel',
+        CONNECTED_CARDS: 'connectedCards',
+        DISCREPANCY: 'discrepancy',
+        DOMAIN_CONTACT_BILLING: 'domainContactBilling',
+        DOMAIN_TWO_FACTOR_AUTH: 'domainTwoFactorAuth',
+        DUPLICATE_DETECTION: 'duplicateDetection',
+        EMAIL_SUPPRESSION_BETA: 'emailSuppressionBeta',
+        EXPENSES_V2: 'expensesV2',
+        EXPENSIFY_CARD: 'expensifyCard',
+        EXPENSIFY_CARD_INTACCT_RECONCILIATION: 'expensifyCardIntacctReconciliation',
+        EXPENSIFY_CARD_NETSUITE_RECONCILIATION: 'expensifyCardNetSuiteReconciliation',
+        EXPENSIFY_CARD_QBO_RECONCILIATION: 'expensifyCardQBOReconciliation',
+        EXPENSIFY_CARD_RAPID_INCREASE_FRAUD: 'expensifyCardRapidIncreaseFraud',
+        EXPENSIFY_CARD_XERO_RECONCILIATION: 'expensifyCardXeroReconciliation',
+        EXPENSIFY_ORG: 'expensifyOrg',
+        FIX_VIOLATION_PUSH_NOTIFICATION: 'fixViolationPushNotification',
+        FREE_PLAN_FULL_LAUNCH: 'freePlanFullLaunch',
+        FREE_PLAN_SOFT_LAUNCH: 'freePlanSoftLaunch',
+        GUSTO: 'gusto',
+        INBOX_CACHE: 'inboxCache',
+        INBOX_HIDDEN_TASKS: 'inboxHiddenTasks',
+        INDIRECT_INTEGRATION_SETUP: 'indirectIntegrationSetup',
+        IOU: 'IOU',
+        JOIN_POLICY: 'joinPolicy',
+        LOAD_POLICY_ASYNC: 'loadPolicyAsync',
+        MAP_RECEIPT: 'mapReceipt',
+        MERGE_API: 'mergeAPI',
+        MOBILE_REALTIME_REPORT_COMMENTS: 'mobileRealtimeReportComments',
+        MOBILE_SECURE_RECEIPTS: 'mobileSecureReceipts',
+        MONTHLY_SETTLEMENT: 'monthlySettlement',
+        NAMES_AND_AVATARS: 'namesAndAvatars',
+        NATIVE_CHAT: 'nativeChat',
+        NEW_PRICING: 'newPricing',
+        NEWSLETTER_THREE: 'newsletterThree',
+        NEXT_STEPS: 'nextSteps',
+        OPEN_FACE_HAMBURGER: 'openFaceHamburger',
+        PER_DIEM: 'perDiem',
+        PER_DIEM_INTERNATIONAL: 'perDiemInternational',
+        PRICING_COPY_CHANGES: 'pricingCopyChanges',
+        QBO_INVOICES: 'qboInvoices',
+        QUICKBOOKS_DESKTOP_V2: 'quickbooksDesktopV2',
+        REALTIME_REPORT_COMMENTS: 'realtimeReportComments',
+        S2W_ANNOUNCEMENT: 's2wAnnouncement',
+        SCHEDULED_AUTO_REPORTING: 'scheduledAutoReporting',
+        SECURE_RECEIPTS: 'secureReceipts',
+        SECURE_RECEIPTS_REPORTS: 'secureReceiptsReports',
+        SELF_SERVICE_HARD_LAUNCH: 'selfServiceHardLaunch',
+        SEND_MONEY: 'sendMoney',
+        SMART_SCAN_USER_DISPUTES: 'smartScanUserDisputes',
+        SMS_SIGN_UP: 'smsSignUp',
+        STRIPE_CONNECT: 'stripeConnect',
+        SUMMARY_EMAIL: 'summaryEmail',
+        SWIPE_TO_WIN: 'swipeToWin',
+        TAX_FOR_MILEAGE: 'taxForMileage',
+        TWO_FACTOR_AUTH: 'twoFactorAuth',
+        VENMO_INTEGRATION: 'venmoIntegration',
+        ZENEFITS_INTEGRATION: 'zenefitsIntegration',
     },
     BUTTON_STATES: {
         DEFAULT: 'default',

src/ONYXKEYS.ts

@@ -321,7 +321,7 @@ type OnyxValues = {
     [ONYXKEYS.COUNTRY_CODE]: number;
     [ONYXKEYS.COUNTRY]: string;
     [ONYXKEYS.USER]: OnyxTypes.User;
-    [ONYXKEYS.LOGIN_LIST]: OnyxTypes.Login;
+    [ONYXKEYS.LOGIN_LIST]: Record<string, OnyxTypes.Login>;
     [ONYXKEYS.SESSION]: OnyxTypes.Session;
     [ONYXKEYS.BETAS]: OnyxTypes.Beta[];
     [ONYXKEYS.NVP_PRIORITY_MODE]: ValueOf<typeof CONST.PRIORITY_MODE>;
@@ -380,7 +380,7 @@ type OnyxValues = {
     [ONYXKEYS.COLLECTION.WORKSPACE_INVITE_MEMBERS_DRAFT]: Record<string, number>;
     [ONYXKEYS.COLLECTION.REPORT]: OnyxTypes.Report;
     [ONYXKEYS.COLLECTION.REPORT_METADATA]: OnyxTypes.ReportMetadata;
-    [ONYXKEYS.COLLECTION.REPORT_ACTIONS]: OnyxTypes.ReportAction;
+    [ONYXKEYS.COLLECTION.REPORT_ACTIONS]: Record<number, OnyxTypes.ReportAction>;
     [ONYXKEYS.COLLECTION.REPORT_ACTIONS_DRAFTS]: string;
     [ONYXKEYS.COLLECTION.REPORT_ACTIONS_REACTIONS]: OnyxTypes.ReportActionReactions;
     [ONYXKEYS.COLLECTION.REPORT_DRAFT_COMMENT]: string;

src/libs/E2E/API.mock.js → src/libs/E2E/API.mock.ts

@@ -1,7 +1,7 @@
 /* eslint-disable rulesdir/no-api-in-views */
-import _ from 'underscore';
 import Onyx from 'react-native-onyx';
 import Log from '../Log';
+import Response from '../../types/onyx/Response';
 
 // mock functions
 import mockBeginSignin from './apiMocks/beginSignin';
@@ -10,12 +10,16 @@ import mockAuthenticatePusher from './apiMocks/authenticatePusher';
 import mockOpenApp from './apiMocks/openApp';
 import mockOpenReport from './apiMocks/openReport';
 
+type ApiCommandParameters = Record<string, unknown>;
+
+type Mocks = Record<string, (params: ApiCommandParameters) => Response>;
+
 /**
  * A dictionary which has the name of a API command as key, and a function which
  * receives the api command parameters as value and is expected to return a response
  * object.
  */
-const mocks = {
+const mocks: Mocks = {
     BeginSignIn: mockBeginSignin,
     SigninUser: mockSigninUser,
     OpenApp: mockOpenApp,
@@ -24,9 +28,9 @@ const mocks = {
     AuthenticatePusher: mockAuthenticatePusher,
 };
 
-function mockCall(command, apiCommandParameters, tag) {
-    const mockResponse = mocks[command] && mocks[command](apiCommandParameters);
-    if (!mockResponse || !_.isArray(mockResponse.onyxData)) {
+function mockCall(command: string, apiCommandParameters: ApiCommandParameters, tag: string): Promise<void> | undefined {
+    const mockResponse = mocks[command]?.(apiCommandParameters);
+    if (!mockResponse || !Array.isArray(mockResponse.onyxData)) {
         Log.warn(`[${tag}] for command ${command} is not mocked yet!`);
         return;
     }
@@ -38,12 +42,10 @@ function mockCall(command, apiCommandParameters, tag) {
  * All calls to API.write() will be persisted to disk as JSON with the params, successData, and failureData.
  * This is so that if the network is unavailable or the app is closed, we can send the WRITE request later.
  *
- * @param {String} command - Name of API command to call.
- * @param {Object} apiCommandParameters - Parameters to send to the API.
- *
- * @returns {Promise}
+ * @param command - Name of API command to call.
+ * @param apiCommandParameters - Parameters to send to the API.
  */
-function write(command, apiCommandParameters = {}) {
+function write(command: string, apiCommandParameters: ApiCommandParameters = {}): Promise<void> | undefined {
     return mockCall(command, apiCommandParameters, 'API.write');
 }
 
@@ -55,24 +57,20 @@ function write(command, apiCommandParameters = {}) {
  * Using this method is discouraged and will throw an ESLint error. Use it sparingly and only when all other alternatives have been exhausted.
  * It is best to discuss it in Slack anytime you are tempted to use this method.
  *
- * @param {String} command - Name of API command to call.
- * @param {Object} apiCommandParameters - Parameters to send to the API.
- *
- * @returns {Promise}
+ * @param command - Name of API command to call.
+ * @param apiCommandParameters - Parameters to send to the API.
  */
-function makeRequestWithSideEffects(command, apiCommandParameters = {}) {
+function makeRequestWithSideEffects(command: string, apiCommandParameters: ApiCommandParameters = {}): Promise<void> | undefined {
     return mockCall(command, apiCommandParameters, 'API.makeRequestWithSideEffects');
 }
 
 /**
  * Requests made with this method are not be persisted to disk. If there is no network connectivity, the request is ignored and discarded.
  *
- * @param {String} command - Name of API command to call.
- * @param {Object} apiCommandParameters - Parameters to send to the API.
- *
- * @returns {Promise}
+ * @param command - Name of API command to call.
+ * @param apiCommandParameters - Parameters to send to the API.
  */
-function read(command, apiCommandParameters) {
+function read(command: string, apiCommandParameters: ApiCommandParameters): Promise<void> | undefined {
     return mockCall(command, apiCommandParameters, 'API.read');
 }
 

src/libs/E2E/actions/e2eLogin.js → src/libs/E2E/actions/e2eLogin.ts

@@ -8,18 +8,16 @@ import * as Session from '../../actions/Session';
  * If the user is already logged in the function will simply
  * resolve.
  *
- * @param {String} email
- * @param {String} password
- * @return {Promise<boolean>} Resolved true when the user was actually signed in. Returns false if the user was already logged in.
+ * @return Resolved true when the user was actually signed in. Returns false if the user was already logged in.
  */
-export default function (email = 'fake@email.com', password = 'Password123') {
-    const waitForBeginSignInToFinish = () =>
+export default function (email = 'fake@email.com', password = 'Password123'): Promise<boolean> {
+    const waitForBeginSignInToFinish = (): Promise<void> =>
         new Promise((resolve) => {
             const id = Onyx.connect({
                 key: ONYXKEYS.CREDENTIALS,
                 callback: (credentials) => {
                     // beginSignUp writes to credentials.login once the API call is complete
-                    if (!credentials.login) {
+                    if (!credentials?.login) {
                         return;
                     }
 
@@ -36,7 +34,7 @@ export default function (email = 'fake@email.com', password = 'Password123') {
         const connectionId = Onyx.connect({
             key: ONYXKEYS.SESSION,
             callback: (session) => {
-                if (session.authToken == null || session.authToken.length === 0) {
+                if (session?.authToken == null || session.authToken.length === 0) {
                     neededLogin = true;
 
                     // authenticate with a predefined user

src/libs/E2E/apiMocks/authenticatePusher.ts

@@ -0,0 +1,11 @@
+import Response from '../../../types/onyx/Response';
+
+const authenticatePusher = (): Response => ({
+    auth: 'auth',
+    // eslint-disable-next-line @typescript-eslint/naming-convention
+    shared_secret: 'secret',
+    jsonCode: 200,
+    requestID: '783ef7fc3991969a-SJC',
+});
+
+export default authenticatePusher;

src/libs/E2E/apiMocks/beginSignin.js → src/libs/E2E/apiMocks/beginSignin.ts

@@ -1,4 +1,7 @@
-export default ({email}) => ({
+import Response from '../../../types/onyx/Response';
+import {SigninParams} from '../types';
+
+const beginSignin = ({email}: SigninParams): Response => ({
     onyxData: [
         {
             onyxMethod: 'merge',
@@ -23,3 +26,5 @@ export default ({email}) => ({
     jsonCode: 200,
     requestID: '783e54ef4b38cff5-SJC',
 });
+
+export default beginSignin;