diff --git a/__mocks__/reportData/policies.ts b/__mocks__/reportData/policies.ts index d082a3770dc5..d801f9c3e0d6 100644 --- a/__mocks__/reportData/policies.ts +++ b/__mocks__/reportData/policies.ts @@ -73,7 +73,6 @@ const policy420A: Policy = { outputCurrency: 'USD', avatarURL: '', employeeList: {}, - isPolicyExpenseChatEnabled: true, lastModified: '1744880511066991', chatReportIDAnnounce: '0', chatReportIDAdmins: '1552825053855260', diff --git a/src/types/onyx/Policy.ts b/src/types/onyx/Policy.ts index c0f1fec9a344..d535b5b8a247 100644 --- a/src/types/onyx/Policy.ts +++ b/src/types/onyx/Policy.ts @@ -2880,9 +2880,6 @@ type Policy = OnyxCommon.OnyxValueWithOfflineFeedback< /** The custom units data for this policy */ customUnits?: Record; - /** Deprecated legacy flag for policy expense chat compatibility. */ - isPolicyExpenseChatEnabled?: boolean; - /** Whether the auto reporting is enabled */ autoReporting?: boolean; diff --git a/tests/actions/IOU/MoneyRequestTest.ts b/tests/actions/IOU/MoneyRequestTest.ts index 6fea13f2c48c..bfce0a849170 100644 --- a/tests/actions/IOU/MoneyRequestTest.ts +++ b/tests/actions/IOU/MoneyRequestTest.ts @@ -1255,11 +1255,6 @@ describe('MoneyRequest', () => { it('should navigate to confirmation page for CREATE flow from global menu', async () => { await Onyx.set(`${ONYXKEYS.COLLECTION.POLICY}${fakePolicy.id}`, fakePolicy); - const defaultExpensePolicy = { - ...fakePolicy, - isPolicyExpenseChatEnabled: true, - }; - // getPolicyExpenseChat resolves reports via ReportUtils' internal Onyx cache, which isn't populated // in this bare unit-test harness, so return the policy expense chat report directly. jest.mocked(getPolicyExpenseChat).mockReturnValueOnce(fakeReport); @@ -1268,7 +1263,7 @@ describe('MoneyRequest', () => { getCurrencyDecimals: getCurrencyDecimalsLocal, ...baseParams, report: undefined, - defaultExpensePolicy, + defaultExpensePolicy: fakePolicy, isAutoReporting: true, iouType: CONST.IOU.TYPE.CREATE, draftTransactionIDs: [baseParams.transactionID], @@ -1306,7 +1301,6 @@ describe('MoneyRequest', () => { const defaultExpensePolicy = { ...fakePolicy, autoReporting: false, - isPolicyExpenseChatEnabled: true, }; handleMoneyRequestStepDistanceNavigation({ @@ -1799,11 +1793,10 @@ describe('MoneyRequest', () => { describe('shouldUseDefaultExpensePolicy', () => { const fakePolicy = createRandomPolicy(1, CONST.POLICY.TYPE.TEAM); - it('should return true when iouType is CREATE with a paid group policy that has expense chat enabled and no billing restrictions', () => { + it('should return true when iouType is CREATE with a paid group policy and no billing restrictions', () => { const policy = { ...fakePolicy, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, }; expect(shouldUseDefaultExpensePolicy(CONST.IOU.TYPE.CREATE, policy, 0, undefined, undefined, currentUserAccountID)).toBe(true); @@ -1813,7 +1806,6 @@ describe('MoneyRequest', () => { const policy = { ...fakePolicy, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, }; expect(shouldUseDefaultExpensePolicy(CONST.IOU.TYPE.SUBMIT, policy, 0, undefined, undefined, currentUserAccountID)).toBe(false); @@ -1825,7 +1817,6 @@ describe('MoneyRequest', () => { const policy = { ...fakePolicy, type: CONST.POLICY.TYPE.PERSONAL, - isPolicyExpenseChatEnabled: true, }; expect(shouldUseDefaultExpensePolicy(CONST.IOU.TYPE.CREATE, policy, 0, undefined, undefined, currentUserAccountID)).toBe(false); @@ -1835,7 +1826,6 @@ describe('MoneyRequest', () => { const policy = { ...fakePolicy, type: CONST.POLICY.TYPE.PERSONAL, - isPolicyExpenseChatEnabled: false, }; expect(shouldUseDefaultExpensePolicy(CONST.IOU.TYPE.CREATE, policy, 0, undefined, undefined, currentUserAccountID)).toBe(false); @@ -1853,7 +1843,6 @@ describe('MoneyRequest', () => { const policy = { ...fakePolicy, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, }; expect(shouldUseDefaultExpensePolicy(CONST.IOU.TYPE.CREATE, policy, undefined, undefined, undefined, currentUserAccountID)).toBe(true); @@ -1863,7 +1852,6 @@ describe('MoneyRequest', () => { const policy = { ...fakePolicy, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, ownerAccountID: TEST_USER_ACCOUNT_ID, }; const pastDate = Math.floor(Date.now() / 1000) - 86400 * 30; diff --git a/tests/actions/IOU/RequestMoneyTest.ts b/tests/actions/IOU/RequestMoneyTest.ts index dd6d2cc4190d..3ed90edba91a 100644 --- a/tests/actions/IOU/RequestMoneyTest.ts +++ b/tests/actions/IOU/RequestMoneyTest.ts @@ -1229,7 +1229,6 @@ describe('actions/IOU', () => { type: CONST.POLICY.TYPE.TEAM, owner: RORY_EMAIL, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, employeeList: { [CARLOS_EMAIL]: { role: CONST.POLICY.ROLE.ADMIN, diff --git a/tests/actions/IOU/SplitExpenseItemsTest.ts b/tests/actions/IOU/SplitExpenseItemsTest.ts index f067232912cd..43b1d5583131 100644 --- a/tests/actions/IOU/SplitExpenseItemsTest.ts +++ b/tests/actions/IOU/SplitExpenseItemsTest.ts @@ -23,7 +23,6 @@ describe('initSplitExpenseItemData stale tax handling', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'owner@test.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, tax: {trackingEnabled: true}, taxRates: { @@ -45,7 +44,6 @@ describe('initSplitExpenseItemData stale tax handling', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'owner@test.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, tax: {trackingEnabled: true}, taxRates: { @@ -68,7 +66,6 @@ describe('initSplitExpenseItemData stale tax handling', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'owner@test.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, tax: {trackingEnabled: true}, taxRates: { @@ -91,7 +88,6 @@ describe('initSplitExpenseItemData stale tax handling', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'owner@test.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, tax: {trackingEnabled: true}, taxRates: { @@ -113,7 +109,6 @@ describe('initSplitExpenseItemData stale tax handling', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'owner@test.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, tax: {trackingEnabled: true}, taxRates: { diff --git a/tests/actions/IOUTest/DeleteMoneyRequestTest.ts b/tests/actions/IOUTest/DeleteMoneyRequestTest.ts index 76e5b74cf28a..8ec1d3e453ca 100644 --- a/tests/actions/IOUTest/DeleteMoneyRequestTest.ts +++ b/tests/actions/IOUTest/DeleteMoneyRequestTest.ts @@ -704,7 +704,6 @@ describe('actions/IOU/DeleteMoneyRequest', () => { name: '', owner: '', outputCurrency: '', - isPolicyExpenseChatEnabled: false, }, policyTagList: {}, policyCategories: {}, diff --git a/tests/actions/IOUTest/PayMoneyRequestTest.ts b/tests/actions/IOUTest/PayMoneyRequestTest.ts index 1ee41471d74c..157807c05c56 100644 --- a/tests/actions/IOUTest/PayMoneyRequestTest.ts +++ b/tests/actions/IOUTest/PayMoneyRequestTest.ts @@ -920,7 +920,6 @@ describe('actions/IOU/PayMoneyRequest', () => { role: CONST.POLICY.ROLE.ADMIN, owner: adminEmail, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, type: CONST.POLICY.TYPE.CORPORATE, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, employeeList: { @@ -1748,7 +1747,6 @@ describe('actions/IOU/PayMoneyRequest', () => { owner: adminEmail, ownerAccountID: adminAccountID, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, type: CONST.POLICY.TYPE.CORPORATE, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, reimbursementChoice: CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_YES, @@ -1822,7 +1820,6 @@ describe('actions/IOU/PayMoneyRequest', () => { owner: adminEmail, ownerAccountID: adminAccountID, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, type: CONST.POLICY.TYPE.CORPORATE, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, reimbursementChoice: CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_YES, diff --git a/tests/actions/IOUTest/ReportWorkflowTest.ts b/tests/actions/IOUTest/ReportWorkflowTest.ts index 741d3b5d2820..db633b29545e 100644 --- a/tests/actions/IOUTest/ReportWorkflowTest.ts +++ b/tests/actions/IOUTest/ReportWorkflowTest.ts @@ -329,7 +329,6 @@ describe('actions/IOU/ReportWorkflow', () => { role: CONST.POLICY.ROLE.ADMIN, owner: CARLOS_EMAIL, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.BASIC, }; @@ -452,7 +451,6 @@ describe('actions/IOU/ReportWorkflow', () => { role: CONST.POLICY.ROLE.ADMIN, owner: CARLOS_EMAIL, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.DYNAMICEXTERNAL, }; @@ -567,7 +565,6 @@ describe('actions/IOU/ReportWorkflow', () => { role: CONST.POLICY.ROLE.ADMIN, owner: CARLOS_EMAIL, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.BASIC, }; @@ -669,7 +666,6 @@ describe('actions/IOU/ReportWorkflow', () => { role: CONST.POLICY.ROLE.ADMIN, owner: CARLOS_EMAIL, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, }; @@ -3970,7 +3966,6 @@ describe('actions/IOU/ReportWorkflow', () => { owner: adminEmail, ownerAccountID: adminAccountID, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, type: CONST.POLICY.TYPE.CORPORATE, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, reimbursementChoice: CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_MANUAL, @@ -4215,7 +4210,6 @@ describe('actions/IOU/ReportWorkflow', () => { role: CONST.POLICY.ROLE.ADMIN, owner: adminEmail, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, type: CONST.POLICY.TYPE.CORPORATE, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, employeeList: { @@ -4528,7 +4522,6 @@ describe('actions/IOU/ReportWorkflow', () => { role: CONST.POLICY.ROLE.ADMIN, owner: adminEmail, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, type: CONST.POLICY.TYPE.CORPORATE, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, employeeList: { diff --git a/tests/actions/IOUTest/UpdateMoneyRequestTest.ts b/tests/actions/IOUTest/UpdateMoneyRequestTest.ts index e6af5d5d4be6..1b67da776c2d 100644 --- a/tests/actions/IOUTest/UpdateMoneyRequestTest.ts +++ b/tests/actions/IOUTest/UpdateMoneyRequestTest.ts @@ -445,7 +445,6 @@ describe('actions/IOU/UpdateMoneyRequest', () => { name: '', owner: '', outputCurrency: '', - isPolicyExpenseChatEnabled: false, }, policyTagList: {}, policyCategories: {}, @@ -519,7 +518,6 @@ describe('actions/IOU/UpdateMoneyRequest', () => { name: '', owner: '', outputCurrency: '', - isPolicyExpenseChatEnabled: false, }, policyTagList: {}, policyCategories: {}, diff --git a/tests/actions/ReportPreviewActionUtilsTest.ts b/tests/actions/ReportPreviewActionUtilsTest.ts index 061f477dd237..6ab3a4fda6b2 100644 --- a/tests/actions/ReportPreviewActionUtilsTest.ts +++ b/tests/actions/ReportPreviewActionUtilsTest.ts @@ -397,7 +397,6 @@ describe('getReportPreviewAction', () => { owner: OWNER_EMAIL, ownerAccountID: OWNER_ACCOUNT_ID, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, approver: OWNER_EMAIL, preventSelfApproval: false, diff --git a/tests/actions/ReportTest.ts b/tests/actions/ReportTest.ts index 6e2f45c60765..3b7eac9501c7 100644 --- a/tests/actions/ReportTest.ts +++ b/tests/actions/ReportTest.ts @@ -2780,7 +2780,6 @@ describe('actions/Report', () => { // Given a policy with harvesting is disabled const policy = { ...createRandomPolicy(Number(policyID)), - isPolicyExpenseChatEnabled: true, type: CONST.POLICY.TYPE.TEAM, autoReporting: false, autoReportingFrequency: CONST.POLICY.AUTO_REPORTING_FREQUENCIES.IMMEDIATE, @@ -2869,7 +2868,6 @@ describe('actions/Report', () => { global.fetch = mockFetchData; const policy = { ...createRandomPolicy(Number(policyID)), - isPolicyExpenseChatEnabled: true, type: CONST.POLICY.TYPE.TEAM, harvesting: { enabled: false, @@ -2902,7 +2900,6 @@ describe('actions/Report', () => { // Given a policy with harvesting is enabled const policy = { ...createRandomPolicy(Number(policyID)), - isPolicyExpenseChatEnabled: true, type: CONST.POLICY.TYPE.TEAM, autoReportingFrequency: CONST.POLICY.AUTO_REPORTING_FREQUENCIES.IMMEDIATE, harvesting: { @@ -2939,7 +2936,6 @@ describe('actions/Report', () => { // Given a policy with instant submission and approval disabled const policy: OnyxTypes.Policy = { ...createRandomPolicy(Number(policyID)), - isPolicyExpenseChatEnabled: true, type: CONST.POLICY.TYPE.TEAM, autoReportingFrequency: CONST.POLICY.AUTO_REPORTING_FREQUENCIES.INSTANT, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, @@ -2983,7 +2979,6 @@ describe('actions/Report', () => { const policy = { ...createRandomPolicy(Number(policyID)), - isPolicyExpenseChatEnabled: true, type: CONST.POLICY.TYPE.TEAM, autoReporting: false, autoReportingFrequency: CONST.POLICY.AUTO_REPORTING_FREQUENCIES.IMMEDIATE, @@ -4117,7 +4112,6 @@ describe('actions/Report', () => { role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, employeeList: { [adminEmail]: { role: CONST.POLICY.ROLE.ADMIN, @@ -4362,7 +4356,6 @@ describe('actions/Report', () => { id: 'targetPolicy', role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, employeeList: { [existingAdminEmail]: {email: existingAdminEmail, role: CONST.POLICY.ROLE.ADMIN}, [existingUserEmail]: {email: existingUserEmail, role: CONST.POLICY.ROLE.USER}, @@ -4654,7 +4647,6 @@ describe('actions/Report', () => { role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, employeeList: { 'admin@test.com': { role: CONST.POLICY.ROLE.ADMIN, diff --git a/tests/data/Invoice.ts b/tests/data/Invoice.ts index 66c0f7d36e2c..dff24b7c9c5b 100644 --- a/tests/data/Invoice.ts +++ b/tests/data/Invoice.ts @@ -11,7 +11,6 @@ const policy: OnyxEntry = { role: 'admin', owner: 'a1@53019.com', ownerAccountID: 32, - isPolicyExpenseChatEnabled: true, outputCurrency: 'USD', autoReporting: true, autoReportingFrequency: 'instant', diff --git a/tests/perf-test/OptionsListUtils.perf-test.ts b/tests/perf-test/OptionsListUtils.perf-test.ts index ef2273a93805..7572cc653871 100644 --- a/tests/perf-test/OptionsListUtils.perf-test.ts +++ b/tests/perf-test/OptionsListUtils.perf-test.ts @@ -87,7 +87,6 @@ const allPolicies = { type: CONST.POLICY.TYPE.TEAM, owner: 'test@expensify.com', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: false, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, } as Policy, }; diff --git a/tests/ui/AgentsPromoBannersTest.tsx b/tests/ui/AgentsPromoBannersTest.tsx index 47b9d0099445..488783fde0d7 100644 --- a/tests/ui/AgentsPromoBannersTest.tsx +++ b/tests/ui/AgentsPromoBannersTest.tsx @@ -81,7 +81,6 @@ function buildPolicy(): Policy { owner: USER_EMAIL, outputCurrency: 'USD', approvalMode: CONST.POLICY.APPROVAL_MODE.BASIC, - isPolicyExpenseChatEnabled: true, areWorkflowsEnabled: true, areRulesEnabled: true, pendingAction: null, diff --git a/tests/ui/DistanceRequestStartPageTest.tsx b/tests/ui/DistanceRequestStartPageTest.tsx index 81a42ca1b07e..dda213ad7565 100644 --- a/tests/ui/DistanceRequestStartPageTest.tsx +++ b/tests/ui/DistanceRequestStartPageTest.tsx @@ -238,7 +238,6 @@ describe('DistanceRequestStartPage', () => { type: CONST.POLICY.TYPE.TEAM, name: 'Workspace', role: CONST.POLICY.ROLE.USER, - isPolicyExpenseChatEnabled: true, autoReporting: true, commuterExclusions: { method: CONST.POLICY.COMMUTER_EXCLUSION_METHOD.FIXED_DISTANCE, @@ -342,7 +341,6 @@ describe('DistanceRequestStartPage', () => { type: CONST.POLICY.TYPE.TEAM, name: policyID, role: CONST.POLICY.ROLE.USER, - isPolicyExpenseChatEnabled: true, autoReporting: false, commuterExclusions: { method: CONST.POLICY.COMMUTER_EXCLUSION_METHOD.FIXED_DISTANCE, diff --git a/tests/ui/IOURequestStepDistanceRateTest.tsx b/tests/ui/IOURequestStepDistanceRateTest.tsx index d669efa99abf..1b12830b6d7c 100644 --- a/tests/ui/IOURequestStepDistanceRateTest.tsx +++ b/tests/ui/IOURequestStepDistanceRateTest.tsx @@ -134,7 +134,6 @@ function buildPolicy(rateCount: number): Policy { type: CONST.POLICY.TYPE.CORPORATE, owner: ACCOUNT_LOGIN, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, customUnits: { [CUSTOM_UNIT_ID]: { attributes: {unit: CONST.CUSTOM_UNITS.DISTANCE_UNIT_MILES}, diff --git a/tests/ui/ImportedMembersPageTest.tsx b/tests/ui/ImportedMembersPageTest.tsx index 3b8315ef04f2..713593ed257f 100644 --- a/tests/ui/ImportedMembersPageTest.tsx +++ b/tests/ui/ImportedMembersPageTest.tsx @@ -73,7 +73,6 @@ function buildSubmitPolicy(): Policy { [ADMIN_EMAIL]: {email: ADMIN_EMAIL, role: CONST.POLICY.ROLE.ADMIN}, }, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, pendingAction: null, errors: {}, } as Policy; diff --git a/tests/ui/MoneyRequestReportActionsListRejectModalTest.tsx b/tests/ui/MoneyRequestReportActionsListRejectModalTest.tsx index 7bfd77c6c2bc..da7168b6e9c3 100644 --- a/tests/ui/MoneyRequestReportActionsListRejectModalTest.tsx +++ b/tests/ui/MoneyRequestReportActionsListRejectModalTest.tsx @@ -195,7 +195,6 @@ const mockPolicy: Policy = { role: CONST.POLICY.ROLE.ADMIN, }, }, - isPolicyExpenseChatEnabled: true, } as Policy; const mockTransaction: Transaction = { diff --git a/tests/ui/MoneyRequestViewReceiptTest.tsx b/tests/ui/MoneyRequestViewReceiptTest.tsx index 12f794ce263d..baa1fa7d3797 100644 --- a/tests/ui/MoneyRequestViewReceiptTest.tsx +++ b/tests/ui/MoneyRequestViewReceiptTest.tsx @@ -127,7 +127,6 @@ const expensePolicy: Policy = { role: CONST.POLICY.ROLE.ADMIN, owner: currentUserEmail, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }; const renderMoneyRequestView = () => @@ -179,7 +178,6 @@ const setupTestData = async () => { name: 'Test Policy', owner: currentUserEmail, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }); await Onyx.merge(`${ONYXKEYS.COLLECTION.REPORT}${expenseReportID}`, { reportID: expenseReportID, diff --git a/tests/ui/MoneyRequestViewTest.tsx b/tests/ui/MoneyRequestViewTest.tsx index fcdb1dc16786..82c7fd92e59c 100644 --- a/tests/ui/MoneyRequestViewTest.tsx +++ b/tests/ui/MoneyRequestViewTest.tsx @@ -113,7 +113,6 @@ const renderMoneyRequestView = (threadReport: ReturnType { name: 'Test Policy', owner: currentUserEmail, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }); await Onyx.merge(`${ONYXKEYS.COLLECTION.REPORT}${expenseReportID}`, { reportID: expenseReportID, diff --git a/tests/ui/NewReportWorkspaceSelectionPageTest.tsx b/tests/ui/NewReportWorkspaceSelectionPageTest.tsx index 518278b0c4c3..a5e6d00f91c5 100644 --- a/tests/ui/NewReportWorkspaceSelectionPageTest.tsx +++ b/tests/ui/NewReportWorkspaceSelectionPageTest.tsx @@ -82,7 +82,6 @@ async function seedBaseOnyx(policyOverrides?: Partial) { name: POLICY_NAME, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, owner: EMAIL, employeeList: {[EMAIL]: {email: EMAIL, role: CONST.POLICY.ROLE.ADMIN}}, pendingAction: null, diff --git a/tests/ui/ProductMarketingWindowNavigationTest.tsx b/tests/ui/ProductMarketingWindowNavigationTest.tsx index 7ec6a8de079b..dd4cd0413e1d 100644 --- a/tests/ui/ProductMarketingWindowNavigationTest.tsx +++ b/tests/ui/ProductMarketingWindowNavigationTest.tsx @@ -44,7 +44,6 @@ function buildAdminPolicy(): Policy { role: CONST.POLICY.ROLE.ADMIN, owner: USER_EMAIL, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, employeeList: { [USER_EMAIL]: { email: USER_EMAIL, diff --git a/tests/ui/ReportActionComposeTest.tsx b/tests/ui/ReportActionComposeTest.tsx index e216cb8abf77..6b44853c85c1 100644 --- a/tests/ui/ReportActionComposeTest.tsx +++ b/tests/ui/ReportActionComposeTest.tsx @@ -334,7 +334,6 @@ describe('ReportActionCompose Integration Tests', () => { name: 'Test Policy', owner: 'test@test.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }); // Parent expense report (the IOUReportID in the action's originalMessage) await Onyx.merge(`${ONYXKEYS.COLLECTION.REPORT}${expenseReportID}`, { diff --git a/tests/ui/ReportActionItemTest.tsx b/tests/ui/ReportActionItemTest.tsx index 5093ee57c645..b896707dc9c4 100644 --- a/tests/ui/ReportActionItemTest.tsx +++ b/tests/ui/ReportActionItemTest.tsx @@ -424,7 +424,6 @@ describe('ReportActionItem', () => { role: CONST.POLICY.ROLE.ADMIN, owner: 'owner@test.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.DYNAMICEXTERNAL, } as const; @@ -476,7 +475,6 @@ describe('ReportActionItem', () => { role: CONST.POLICY.ROLE.ADMIN, owner: 'owner@test.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.BASIC, } as const; @@ -523,7 +521,6 @@ describe('ReportActionItem', () => { role: CONST.POLICY.ROLE.ADMIN, owner: 'owner@test.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.DYNAMICEXTERNAL, } as const; @@ -571,7 +568,6 @@ describe('ReportActionItem', () => { role: CONST.POLICY.ROLE.ADMIN, owner: 'owner@test.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.DYNAMICEXTERNAL, } as const; diff --git a/tests/ui/TransactionPreviewContentTest.tsx b/tests/ui/TransactionPreviewContentTest.tsx index 31fd576d866f..64ffdeda82cb 100644 --- a/tests/ui/TransactionPreviewContentTest.tsx +++ b/tests/ui/TransactionPreviewContentTest.tsx @@ -51,7 +51,6 @@ const corporatePolicy = { name: 'Corporate Policy', owner: '', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, employeeList: { [currentUserEmail]: { email: currentUserEmail, diff --git a/tests/ui/WorkspaceWorkflowsApprovalsEditPageTest.tsx b/tests/ui/WorkspaceWorkflowsApprovalsEditPageTest.tsx index cf30e4ff92cb..c6f2134ef8d7 100644 --- a/tests/ui/WorkspaceWorkflowsApprovalsEditPageTest.tsx +++ b/tests/ui/WorkspaceWorkflowsApprovalsEditPageTest.tsx @@ -61,7 +61,6 @@ function buildPolicy(): Policy { employeeList, approver: ALICE_EMAIL, areWorkflowsEnabled: true, - isPolicyExpenseChatEnabled: true, outputCurrency: 'USD', avatarURL: '', lastModified: new Date().toISOString(), diff --git a/tests/ui/components/AttachmentPickerWithMenuItemsTest.tsx b/tests/ui/components/AttachmentPickerWithMenuItemsTest.tsx index b5e46639dcd5..23b984913882 100644 --- a/tests/ui/components/AttachmentPickerWithMenuItemsTest.tsx +++ b/tests/ui/components/AttachmentPickerWithMenuItemsTest.tsx @@ -143,7 +143,6 @@ describe('AttachmentPickerWithMenuItems - empty report confirmation', () => { name: 'Test Workspace', type: CONST.POLICY.TYPE.TEAM, role: CONST.POLICY.ROLE.ADMIN, - isPolicyExpenseChatEnabled: true, pendingAction: null, owner: CURRENT_USER_EMAIL, outputCurrency: CONST.CURRENCY.USD, diff --git a/tests/ui/components/EmptySearchViewTest.tsx b/tests/ui/components/EmptySearchViewTest.tsx index 61e1b75ed57e..adbbe07a16ca 100644 --- a/tests/ui/components/EmptySearchViewTest.tsx +++ b/tests/ui/components/EmptySearchViewTest.tsx @@ -259,7 +259,6 @@ describe('EmptySearchView', () => { type: CONST.POLICY.TYPE.TEAM, role: CONST.POLICY.ROLE.ADMIN, areInvoicesEnabled: true, - isPolicyExpenseChatEnabled: true, }); it('should display correct buttons and subtitle when user has not clicked on "Take a test drive"', async () => { diff --git a/tests/ui/components/QuickCreationActionsBarTest.tsx b/tests/ui/components/QuickCreationActionsBarTest.tsx index 69211417b7cc..6efb2cd67ee2 100644 --- a/tests/ui/components/QuickCreationActionsBarTest.tsx +++ b/tests/ui/components/QuickCreationActionsBarTest.tsx @@ -72,7 +72,6 @@ describe('QuickCreationActionsBar - empty report confirmation', () => { name: 'Test Workspace', type: CONST.POLICY.TYPE.TEAM, role: CONST.POLICY.ROLE.ADMIN, - isPolicyExpenseChatEnabled: true, pendingAction: null, owner: CURRENT_USER_EMAIL, outputCurrency: CONST.CURRENCY.USD, diff --git a/tests/ui/components/SearchActionsBarCreateButtonTest.tsx b/tests/ui/components/SearchActionsBarCreateButtonTest.tsx index 3196941fd6c6..cdb1d30417f4 100644 --- a/tests/ui/components/SearchActionsBarCreateButtonTest.tsx +++ b/tests/ui/components/SearchActionsBarCreateButtonTest.tsx @@ -75,7 +75,6 @@ const MOCK_POLICY = { name: 'Test Workspace', type: CONST.POLICY.TYPE.TEAM, role: CONST.POLICY.ROLE.ADMIN, - isPolicyExpenseChatEnabled: true, pendingAction: null, avatarURL: '', areInvoicesEnabled: false, @@ -177,7 +176,7 @@ describe('SearchActionsBarCreateButton', () => { shouldNavigateToUpgradePath: false, }); - // Set up multiple policies with chat enabled + // Set up multiple eligible group workspaces await act(async () => { await Onyx.merge(`${ONYXKEYS.COLLECTION.POLICY}${MOCK_POLICY_ID}`, MOCK_POLICY); await Onyx.merge(`${ONYXKEYS.COLLECTION.POLICY}policy-456`, { diff --git a/tests/ui/components/SettlementButtonTest.tsx b/tests/ui/components/SettlementButtonTest.tsx index ad578dfdd306..de2b135ec238 100644 --- a/tests/ui/components/SettlementButtonTest.tsx +++ b/tests/ui/components/SettlementButtonTest.tsx @@ -188,7 +188,6 @@ function createTestPolicy(overrides?: Partial): Policy { owner: ACCOUNT_LOGIN, ownerAccountID: ACCOUNT_ID, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, ...overrides, } as Policy; } diff --git a/tests/unit/CardFeedUtilsTest.ts b/tests/unit/CardFeedUtilsTest.ts index c5b70afe4d81..d0f0e5c3eff1 100644 --- a/tests/unit/CardFeedUtilsTest.ts +++ b/tests/unit/CardFeedUtilsTest.ts @@ -43,7 +43,6 @@ function createTestPolicy(overrides: Partial & Pick): Poli type: CONST.POLICY.TYPE.TEAM, owner: 'admin@test.com', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: false, ...overrides, }; } diff --git a/tests/unit/CompanyCardExportTest.ts b/tests/unit/CompanyCardExportTest.ts index 78146ff0f7ad..e9f52b421d91 100644 --- a/tests/unit/CompanyCardExportTest.ts +++ b/tests/unit/CompanyCardExportTest.ts @@ -55,7 +55,6 @@ function createQBDPolicy(overrides?: Partial, nonReimbursableAccount = ' role: CONST.POLICY.ROLE.ADMIN, owner: 'test@qbdcc.com', ownerAccountID: 1, - isPolicyExpenseChatEnabled: false, outputCurrency: 'USD', connections: { quickbooksDesktop: { diff --git a/tests/unit/CreateReportMenuItemTest.tsx b/tests/unit/CreateReportMenuItemTest.tsx index b18886934e86..d71c115b3581 100644 --- a/tests/unit/CreateReportMenuItemTest.tsx +++ b/tests/unit/CreateReportMenuItemTest.tsx @@ -87,7 +87,7 @@ jest.mock('@navigation/helpers/isOnSearchMoneyRequestReportPage', () => ({ jest.mock('@pages/inbox/sidebar/FABPopoverContent/FABFocusableMenuItem', () => jest.fn(() => null)); -function makePolicy(id: string, type: Policy['type'], isPolicyExpenseChatEnabled: boolean): Policy { +function makePolicy(id: string, type: Policy['type']): Policy { return { id, name: `${id} workspace`, @@ -97,15 +97,14 @@ function makePolicy(id: string, type: Policy['type'], isPolicyExpenseChatEnabled owner: 'user@test.com', ownerAccountID: 1, employeeList: {}, - isPolicyExpenseChatEnabled, isJoinRequestPending: false, } as Policy; } function setupUseOnyx() { - const personalPolicy = makePolicy('personal-1', CONST.POLICY.TYPE.PERSONAL, true); - const groupPolicy = makePolicy('team-1', CONST.POLICY.TYPE.TEAM, true); - const submitPolicy = makePolicy('submit-1', CONST.POLICY.TYPE.SUBMIT, true); + const personalPolicy = makePolicy('personal-1', CONST.POLICY.TYPE.PERSONAL); + const groupPolicy = makePolicy('team-1', CONST.POLICY.TYPE.TEAM); + const submitPolicy = makePolicy('submit-1', CONST.POLICY.TYPE.SUBMIT); const values = new Map([ [ONYXKEYS.NVP_ACTIVE_POLICY_ID, personalPolicy.id], [`${ONYXKEYS.COLLECTION.POLICY}${personalPolicy.id}`, personalPolicy], diff --git a/tests/unit/DistanceRequestUtilsTest.ts b/tests/unit/DistanceRequestUtilsTest.ts index 7af8bee6363a..74edb8aec4de 100644 --- a/tests/unit/DistanceRequestUtilsTest.ts +++ b/tests/unit/DistanceRequestUtilsTest.ts @@ -34,7 +34,6 @@ const FAKE_POLICY: Policy = { type: 'corporate', owner: 'work.sa1206+travel@gmail.com', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, customUnits: { C9031B6F4725D: { ...distanceCustomUnitBase, diff --git a/tests/unit/ExpensifyCardFeedSelectorUtilsTest.ts b/tests/unit/ExpensifyCardFeedSelectorUtilsTest.ts index d3107188fd46..58ccc40cfccc 100644 --- a/tests/unit/ExpensifyCardFeedSelectorUtilsTest.ts +++ b/tests/unit/ExpensifyCardFeedSelectorUtilsTest.ts @@ -34,7 +34,6 @@ function createAdminPolicy(overrides: Partial & Pick): Pol type: CONST.POLICY.TYPE.TEAM, owner: 'admin@workspace.com', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: false, ...overrides, }; } diff --git a/tests/unit/HomePage/YourSpendSection/useYourSpendDataTest.ts b/tests/unit/HomePage/YourSpendSection/useYourSpendDataTest.ts index 279ec9bcbb17..1fcec1cbb4bc 100644 --- a/tests/unit/HomePage/YourSpendSection/useYourSpendDataTest.ts +++ b/tests/unit/HomePage/YourSpendSection/useYourSpendDataTest.ts @@ -140,7 +140,6 @@ function makeCorporatePolicy(overrides: Partial = {}): Policy { role: 'admin', owner: 'test@example.com', ownerAccountID: ACCOUNT_ID, - isPolicyExpenseChatEnabled: true, outputCurrency: CONST.CURRENCY.USD, approvalMode: CONST.POLICY.APPROVAL_MODE.BASIC, reimbursementChoice: CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_YES, diff --git a/tests/unit/HrUtilsTest.ts b/tests/unit/HrUtilsTest.ts index 725a556ba033..8c17f7717d42 100644 --- a/tests/unit/HrUtilsTest.ts +++ b/tests/unit/HrUtilsTest.ts @@ -59,7 +59,6 @@ function makePolicy(overrides: Partial = {}): Policy { role: CONST.POLICY.ROLE.ADMIN, owner: 'owner@test.com', ownerAccountID: 1, - isPolicyExpenseChatEnabled: true, outputCurrency: 'USD', ...overrides, }; diff --git a/tests/unit/IOUAmountSubmissionTest.ts b/tests/unit/IOUAmountSubmissionTest.ts index 8996bdb9ef82..e250621c1290 100644 --- a/tests/unit/IOUAmountSubmissionTest.ts +++ b/tests/unit/IOUAmountSubmissionTest.ts @@ -506,7 +506,6 @@ describe('AmountSubmission', () => { name: 'Test Workspace', owner: 'me@test.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, autoReporting: false, } as Policy; diff --git a/tests/unit/ImportedMerchantRulesPageTest.tsx b/tests/unit/ImportedMerchantRulesPageTest.tsx index 26ffe0248544..758b28682a41 100644 --- a/tests/unit/ImportedMerchantRulesPageTest.tsx +++ b/tests/unit/ImportedMerchantRulesPageTest.tsx @@ -76,7 +76,6 @@ function buildRulesEnabledControlPolicy(): Policy { [ADMIN_EMAIL]: {email: ADMIN_EMAIL, role: CONST.POLICY.ROLE.ADMIN}, }, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, pendingAction: null, errors: {}, } as Policy; diff --git a/tests/unit/LHNAvatarPipelineTest.ts b/tests/unit/LHNAvatarPipelineTest.ts index c9932b3def0c..25c159bc3048 100644 --- a/tests/unit/LHNAvatarPipelineTest.ts +++ b/tests/unit/LHNAvatarPipelineTest.ts @@ -49,7 +49,6 @@ const TEST_POLICY: Policy = { role: CONST.POLICY.ROLE.ADMIN, owner: 'user1@test.com', ownerAccountID: CURRENT_USER_ACCOUNT_ID, - isPolicyExpenseChatEnabled: true, outputCurrency: 'USD', } as Policy; diff --git a/tests/unit/ModifiedExpenseMessageTest.ts b/tests/unit/ModifiedExpenseMessageTest.ts index 9e10e687fdfa..587a5efb1bc3 100644 --- a/tests/unit/ModifiedExpenseMessageTest.ts +++ b/tests/unit/ModifiedExpenseMessageTest.ts @@ -127,7 +127,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: CURRENT_USER_LOGIN, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }; const result = getMovedFromOrToReportMessage(translateLocal, undefined, selfDMReport, CURRENT_USER_LOGIN, policy); const expectedResult = translate(CONST.LOCALES.EN as 'en', 'iou.movedToPersonalSpace'); @@ -160,7 +159,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: CURRENT_USER_LOGIN, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }; const result = getMovedFromOrToReportMessage(translateLocal, undefined, policyExpenseReport, CURRENT_USER_LOGIN, policy); const expectedResult = translate(CONST.LOCALES.EN as 'en', 'iou.movedFromPersonalSpace', policyExpenseReport.reportName, policy.name); @@ -188,7 +186,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: CURRENT_USER_LOGIN, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }; const result = getMovedFromOrToReportMessage(translateLocal, undefined, policyExpenseReport, CURRENT_USER_LOGIN, policy); // When a valid policy provides a name, the movedFromPersonalSpace message is returned @@ -237,7 +234,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: CURRENT_USER_LOGIN, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }; const result = getMovedFromOrToReportMessage(translateLocal, movedFromReport, undefined, CURRENT_USER_LOGIN, policy); const expectedResult = translate(CONST.LOCALES.EN as 'en', 'iou.movedFromReport', movedFromReport.reportName ?? ''); @@ -1117,7 +1113,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'test@example.com', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }; jest.spyOn(PolicyUtils, 'isPolicyAdmin').mockReturnValue(true); @@ -1553,7 +1548,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'test@example.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }, policyTags: undefined, currentUserLogin: 'test@example.com', @@ -1760,7 +1754,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'test@example.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }, policyTags: undefined, currentUserLogin: 'test@example.com', @@ -1810,7 +1803,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'test@example.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }, policyTags: undefined, currentUserLogin: 'test@example.com', @@ -1862,7 +1854,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'owner@example.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }, policyTags: undefined, currentUserLogin: 'test@example.com', @@ -1879,7 +1870,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'test@example.com', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }; jest.spyOn(PolicyUtils, 'isPolicyAdmin').mockReturnValue(true); @@ -1972,7 +1962,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: CURRENT_USER_LOGIN, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }; const result = getForReportAction({ @@ -2006,7 +1995,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: CURRENT_USER_LOGIN, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }; const result = getForReportAction({ @@ -2078,7 +2066,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'test@example.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }, policyTags: undefined, currentUserLogin: 'test@example.com', @@ -2124,7 +2111,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'test@example.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }, policyTags: undefined, currentUserLogin: 'test@example.com', @@ -2173,7 +2159,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'test@example.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }, policyTags: undefined, currentUserLogin: 'test@example.com', @@ -2222,7 +2207,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'test@example.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }, policyTags: undefined, currentUserLogin: 'test@example.com', @@ -2268,7 +2252,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'test@example.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, }, policyTags: undefined, currentUserLogin: 'test@example.com', @@ -2291,7 +2274,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'test@example.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, connections: { quickbooksOnline: { data: { @@ -2430,7 +2412,6 @@ describe('ModifiedExpenseMessage', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'test@example.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, connections: { xero: { config: {isConfigured: true}, diff --git a/tests/unit/MoneyRequestReportUtilsTest.ts b/tests/unit/MoneyRequestReportUtilsTest.ts index afd485380575..43f3176a73fe 100644 --- a/tests/unit/MoneyRequestReportUtilsTest.ts +++ b/tests/unit/MoneyRequestReportUtilsTest.ts @@ -14,7 +14,6 @@ const policyBaseMock: Policy = { outputCurrency: 'USD', type: 'team', owner: 'admin@test.com', - isPolicyExpenseChatEnabled: true, }; const reportBaseMock: Report = { diff --git a/tests/unit/NextStepUtilsTest.ts b/tests/unit/NextStepUtilsTest.ts index 3d386f25af39..0f98d46ec8f8 100644 --- a/tests/unit/NextStepUtilsTest.ts +++ b/tests/unit/NextStepUtilsTest.ts @@ -49,7 +49,6 @@ describe('libs/NextStepUtils', () => { role: 'admin', type: 'team', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, reimbursementChoice: CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_YES, }; const report = buildOptimisticExpenseReport({ @@ -938,7 +937,6 @@ describe('libs/NextStepUtils', () => { type: CONST.POLICY.TYPE.TEAM, owner: currentUserEmail, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, reimbursementChoice: CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_YES, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, approver: currentUserEmail, @@ -992,7 +990,6 @@ describe('libs/NextStepUtils', () => { type: CONST.POLICY.TYPE.TEAM, owner: currentUserEmail, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, reimbursementChoice: CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_YES, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, approver: currentUserEmail, @@ -1211,7 +1208,6 @@ describe('libs/NextStepUtils', () => { type: CONST.POLICY.TYPE.TEAM, owner: currentUserEmail, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, reimbursementChoice: CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_YES, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, approver: currentUserEmail, diff --git a/tests/unit/OptionsListUtilsTest.tsx b/tests/unit/OptionsListUtilsTest.tsx index b6827f42c1cb..953fdeb44c98 100644 --- a/tests/unit/OptionsListUtilsTest.tsx +++ b/tests/unit/OptionsListUtilsTest.tsx @@ -151,7 +151,6 @@ describe('OptionsListUtils', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'reedrichards@expensify.com', outputCurrency: '', - isPolicyExpenseChatEnabled: false, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, }; @@ -8610,7 +8609,6 @@ describe('OptionsListUtils', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'owner@test.com', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, areCategoriesEnabled: true, }; @@ -8674,7 +8672,6 @@ describe('OptionsListUtils', () => { type: CONST.POLICY.TYPE.TEAM, owner: 'owner@test.com', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, areCategoriesEnabled: true, }; @@ -8713,7 +8710,6 @@ describe('OptionsListUtils', () => { owner: 'owner@test.com', outputCurrency: 'USD', approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, - isPolicyExpenseChatEnabled: false, }; const report: Report = { reportID, @@ -8789,7 +8785,6 @@ describe('OptionsListUtils', () => { owner: 'owner@test.com', outputCurrency: 'USD', approvalMode: CONST.POLICY.APPROVAL_MODE.BASIC, - isPolicyExpenseChatEnabled: false, }; const report: Report = { reportID, @@ -9401,7 +9396,6 @@ describe('OptionsListUtils', () => { role: 'user', approvalMode: CONST.POLICY.APPROVAL_MODE.BASIC, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: false, }; // PersonalDetails with the approver's information @@ -9686,7 +9680,6 @@ describe('OptionsListUtils', () => { owner: 'owner@test.com', role: 'user', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }; const testPersonalDetails = { @@ -9780,7 +9773,6 @@ describe('OptionsListUtils', () => { owner: 'owner@test.com', role: 'admin', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }; const testPersonalDetails = { @@ -9847,7 +9839,6 @@ describe('OptionsListUtils', () => { owner: 'owner@test.com', role: 'user', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }; await Onyx.merge(`${ONYXKEYS.COLLECTION.REPORT}${reportID}`, report); @@ -9893,7 +9884,6 @@ describe('OptionsListUtils', () => { owner: 'owner@test.com', role: 'user', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }; await Onyx.merge(`${ONYXKEYS.COLLECTION.REPORT}${reportID}`, report); @@ -9939,7 +9929,6 @@ describe('OptionsListUtils', () => { owner: 'owner@test.com', role: 'user', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }; await Onyx.merge(`${ONYXKEYS.COLLECTION.REPORT}${reportID}`, report); @@ -10070,7 +10059,6 @@ describe('OptionsListUtils', () => { owner: 'owner@test.com', role: 'user', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }; const testPersonalDetails = { @@ -10134,7 +10122,6 @@ describe('OptionsListUtils', () => { owner: 'owner@test.com', role: 'user', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }; await Onyx.merge(`${ONYXKEYS.COLLECTION.REPORT}${reportID}`, report); @@ -10168,7 +10155,6 @@ describe('OptionsListUtils', () => { owner: 'formatowner@test.com', role: 'admin', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }; const formatPersonalDetails = { @@ -10808,7 +10794,6 @@ describe('OptionsListUtils', () => { owner: 'owner@test.com', role: 'admin', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }; await Onyx.merge(`${ONYXKEYS.COLLECTION.REPORT}${reportID}`, report); diff --git a/tests/unit/PolicyUtilsTest.ts b/tests/unit/PolicyUtilsTest.ts index 7053b9fa065e..f68c557dca32 100644 --- a/tests/unit/PolicyUtilsTest.ts +++ b/tests/unit/PolicyUtilsTest.ts @@ -2917,7 +2917,6 @@ describe('PolicyUtils', () => { ...createRandomPolicy(1, CONST.POLICY.TYPE.CORPORATE), role: CONST.POLICY.ROLE.USER, pendingAction: null, - isPolicyExpenseChatEnabled: true, arePerDiemRatesEnabled: true, customUnits: { ABCDEF: perDiemCustomUnit, @@ -2927,7 +2926,6 @@ describe('PolicyUtils', () => { ...createRandomPolicy(2, CONST.POLICY.TYPE.TEAM), role: CONST.POLICY.ROLE.USER, pendingAction: null, - isPolicyExpenseChatEnabled: true, arePerDiemRatesEnabled: true, customUnits: { ABCDEF: perDiemCustomUnit, @@ -2946,7 +2944,6 @@ describe('PolicyUtils', () => { ...createRandomPolicy(1, CONST.POLICY.TYPE.CORPORATE), role: CONST.POLICY.ROLE.USER, pendingAction: null, - isPolicyExpenseChatEnabled: true, arePerDiemRatesEnabled: true, customUnits: { ABCDEF: perDiemCustomUnit, @@ -2956,7 +2953,6 @@ describe('PolicyUtils', () => { ...createRandomPolicy(2, CONST.POLICY.TYPE.CORPORATE), role: CONST.POLICY.ROLE.USER, pendingAction: null, - isPolicyExpenseChatEnabled: true, arePerDiemRatesEnabled: true, customUnits: { ABCDEF: { @@ -2969,7 +2965,6 @@ describe('PolicyUtils', () => { ...createRandomPolicy(3, CONST.POLICY.TYPE.TEAM), role: CONST.POLICY.ROLE.USER, pendingAction: null, - isPolicyExpenseChatEnabled: true, arePerDiemRatesEnabled: true, customUnits: { ABCDEF: perDiemCustomUnit, @@ -2987,7 +2982,6 @@ describe('PolicyUtils', () => { ...createRandomPolicy(1, CONST.POLICY.TYPE.CORPORATE), role: CONST.POLICY.ROLE.USER, pendingAction: null, - isPolicyExpenseChatEnabled: true, // Migrated member: flag never arrived, but the configured custom unit is present arePerDiemRatesEnabled: undefined, customUnits: { @@ -3005,7 +2999,6 @@ describe('PolicyUtils', () => { ...createRandomPolicy(1, CONST.POLICY.TYPE.CORPORATE), role: CONST.POLICY.ROLE.USER, pendingAction: null, - isPolicyExpenseChatEnabled: true, // Explicit off must be respected, so the lingering custom unit cannot re-enable it arePerDiemRatesEnabled: false, customUnits: { diff --git a/tests/unit/ProductMarketingWindowTest.tsx b/tests/unit/ProductMarketingWindowTest.tsx index ba4f8f9a6f35..940a027523fc 100644 --- a/tests/unit/ProductMarketingWindowTest.tsx +++ b/tests/unit/ProductMarketingWindowTest.tsx @@ -106,7 +106,6 @@ function buildAdminPolicy(policyID = POLICY_ID): Policy { role: CONST.POLICY.ROLE.ADMIN, owner: USER_EMAIL, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, employeeList: { [USER_EMAIL]: { email: USER_EMAIL, diff --git a/tests/unit/QuickActionMenuItemTest.tsx b/tests/unit/QuickActionMenuItemTest.tsx index 799cfd354a37..fe50107d1d8b 100644 --- a/tests/unit/QuickActionMenuItemTest.tsx +++ b/tests/unit/QuickActionMenuItemTest.tsx @@ -71,7 +71,6 @@ describe('QuickActionMenuItem', () => { const renderWithActivePolicy = async (policyType: ValueOf) => { const activePolicy = createRandomPolicy(Number(ACTIVE_POLICY_ID), policyType); - Reflect.deleteProperty(activePolicy as Record, 'isPolicyExpenseChatEnabled'); const policyExpenseChat = { ...createPolicyExpenseChat(Number(POLICY_CHAT_REPORT_ID)), @@ -114,14 +113,14 @@ describe('QuickActionMenuItem', () => { expect(mockTranslate).toHaveBeenCalledWith('quickAction.paySomeone', 'SPY_NAME'); }); - it('shows the workspace fallback quick action for a group policy even if the policy expense chat flag is absent', async () => { + it('shows the workspace fallback quick action for a group policy', async () => { const props = await renderWithActivePolicy(CONST.POLICY.TYPE.TEAM); expect(mockFABFocusableMenuItem).toHaveBeenCalled(); expect(props).toEqual(expect.objectContaining({isVisible: true})); }); - it('hides the workspace fallback quick action for a personal policy even if the policy expense chat flag is absent', async () => { + it('hides the workspace fallback quick action for a personal policy', async () => { const props = await renderWithActivePolicy(CONST.POLICY.TYPE.PERSONAL); expect(mockFABFocusableMenuItem).toHaveBeenCalled(); diff --git a/tests/unit/ReportPrimaryActionUtilsTest.ts b/tests/unit/ReportPrimaryActionUtilsTest.ts index fb00094d0d57..c4f800700d56 100644 --- a/tests/unit/ReportPrimaryActionUtilsTest.ts +++ b/tests/unit/ReportPrimaryActionUtilsTest.ts @@ -427,7 +427,6 @@ describe('getPrimaryAction', () => { name: 'Test Workspace', owner: OWNER_EMAIL, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.USER, type: CONST.POLICY.TYPE.CORPORATE, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -478,7 +477,6 @@ describe('getPrimaryAction', () => { name: 'Test Workspace', owner: CURRENT_USER_EMAIL, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.USER, type: CONST.POLICY.TYPE.CORPORATE, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, // Submit&Close @@ -530,7 +528,6 @@ describe('getPrimaryAction', () => { name: 'Test Workspace', owner: OTHER_WORKFLOW_APPROVER_EMAIL, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.USER, type: CONST.POLICY.TYPE.CORPORATE, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, diff --git a/tests/unit/ReportSecondaryActionUtilsTest.ts b/tests/unit/ReportSecondaryActionUtilsTest.ts index 3d274bb63b12..56ebd7b913e6 100644 --- a/tests/unit/ReportSecondaryActionUtilsTest.ts +++ b/tests/unit/ReportSecondaryActionUtilsTest.ts @@ -104,7 +104,6 @@ const createQBOPolicy = (role: Policy['role'], autoSyncEnabled: boolean, exporte type: CONST.POLICY.TYPE.TEAM, owner: ADMIN_EMAIL, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, connections: createQBOConnections(autoSyncEnabled, exporter), }); @@ -251,7 +250,6 @@ describe('getSecondaryAction', () => { reimbursementChoice: CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_MANUAL, reimburser: designatedPayerEmail, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, employeeList: { [ADMIN_EMAIL]: {role: CONST.POLICY.ROLE.ADMIN}, [designatedPayerEmail]: {role: CONST.POLICY.ROLE.ADMIN}, @@ -2861,7 +2859,6 @@ describe('getSecondaryAction', () => { id: POLICY_ID, type: CONST.POLICY.TYPE.TEAM, reimbursementChoice: CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_MANUAL, - isPolicyExpenseChatEnabled: true, employeeList: { [EMPLOYEE_EMAIL]: {email: EMPLOYEE_EMAIL, role: CONST.POLICY.ROLE.ADMIN}, [MANAGER_EMAIL]: {email: MANAGER_EMAIL, role: CONST.POLICY.ROLE.USER}, @@ -2907,7 +2904,6 @@ describe('getSecondaryAction', () => { id: POLICY_ID, type: CONST.POLICY.TYPE.TEAM, role: CONST.POLICY.ROLE.ADMIN, - isPolicyExpenseChatEnabled: true, employeeList: { [ADMIN_EMAIL]: {email: ADMIN_EMAIL, role: CONST.POLICY.ROLE.ADMIN}, [EMPLOYEE_EMAIL]: {email: EMPLOYEE_EMAIL, role: CONST.POLICY.ROLE.USER}, @@ -2950,7 +2946,6 @@ describe('getSecondaryAction', () => { const newPolicy = createMock({ id: POLICY_ID, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.USER, employeeList: { [MANAGER_EMAIL]: {email: MANAGER_EMAIL, role: CONST.POLICY.ROLE.USER}, @@ -3027,7 +3022,6 @@ describe('getSecondaryAction', () => { id: POLICY_ID, type: CONST.POLICY.TYPE.TEAM, approver: APPROVER_EMAIL, - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.USER, employeeList: { [APPROVER_EMAIL]: {email: APPROVER_EMAIL, role: CONST.POLICY.ROLE.USER}, @@ -3082,7 +3076,6 @@ describe('getSecondaryAction', () => { id: POLICY_ID, type: CONST.POLICY.TYPE.TEAM, role: CONST.POLICY.ROLE.ADMIN, - isPolicyExpenseChatEnabled: true, employeeList: { [ADMIN_EMAIL]: {email: ADMIN_EMAIL, role: CONST.POLICY.ROLE.ADMIN}, [EMPLOYEE_EMAIL]: {email: EMPLOYEE_EMAIL, role: CONST.POLICY.ROLE.USER}, @@ -3182,7 +3175,6 @@ describe('getSecondaryAction', () => { const policy = createMock({ id: POLICY_ID, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, employeeList: { [EMPLOYEE_EMAIL]: {email: EMPLOYEE_EMAIL, role: CONST.POLICY.ROLE.USER}, }, @@ -4627,7 +4619,6 @@ describe('getSecondaryTransactionThreadActions', () => { const policy = createMock({ id: POLICY_ID, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, employeeList: { [EMPLOYEE_EMAIL]: {email: EMPLOYEE_EMAIL, role: CONST.POLICY.ROLE.USER}, }, @@ -4668,7 +4659,6 @@ describe('getSecondaryTransactionThreadActions', () => { id: POLICY_ID, type: CONST.POLICY.TYPE.TEAM, reimbursementChoice: CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_MANUAL, - isPolicyExpenseChatEnabled: true, employeeList: { [EMPLOYEE_EMAIL]: {email: EMPLOYEE_EMAIL, role: CONST.POLICY.ROLE.ADMIN}, [MANAGER_EMAIL]: {email: MANAGER_EMAIL, role: CONST.POLICY.ROLE.USER}, @@ -4725,7 +4715,6 @@ describe('getSecondaryTransactionThreadActions', () => { const policy = createMock({ id: POLICY_ID, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: false, employeeList: { [EMPLOYEE_EMAIL]: {email: EMPLOYEE_EMAIL, role: CONST.POLICY.ROLE.USER}, [ADMIN_EMAIL]: {email: ADMIN_EMAIL, role: CONST.POLICY.ROLE.ADMIN}, @@ -4786,7 +4775,6 @@ describe('getSecondaryTransactionThreadActions', () => { const policy = createMock({ id: POLICY_ID, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, employeeList: { [EMPLOYEE_EMAIL]: {email: EMPLOYEE_EMAIL, role: CONST.POLICY.ROLE.USER}, [ADMIN_EMAIL]: {email: ADMIN_EMAIL, role: CONST.POLICY.ROLE.ADMIN}, @@ -4835,7 +4823,6 @@ describe('getSecondaryTransactionThreadActions', () => { const policy = createMock({ id: POLICY_ID, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, employeeList: { [ADMIN_EMAIL]: {email: ADMIN_EMAIL, role: CONST.POLICY.ROLE.ADMIN}, }, @@ -4880,7 +4867,6 @@ describe('getSecondaryTransactionThreadActions', () => { const policy = createMock({ id: POLICY_ID, type: CONST.POLICY.TYPE.PERSONAL, - isPolicyExpenseChatEnabled: true, employeeList: { [EMPLOYEE_EMAIL]: {email: EMPLOYEE_EMAIL, role: CONST.POLICY.ROLE.USER}, [ADMIN_EMAIL]: {email: ADMIN_EMAIL, role: CONST.POLICY.ROLE.ADMIN}, @@ -4927,7 +4913,6 @@ describe('getSecondaryTransactionThreadActions', () => { const policy = createMock({ id: POLICY_ID, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, autoReporting: true, autoReportingFrequency: CONST.POLICY.AUTO_REPORTING_FREQUENCIES.INSTANT, approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, diff --git a/tests/unit/ReportSubmitUtilsTest.ts b/tests/unit/ReportSubmitUtilsTest.ts index 590b5fb0cace..ebb139b94fad 100644 --- a/tests/unit/ReportSubmitUtilsTest.ts +++ b/tests/unit/ReportSubmitUtilsTest.ts @@ -27,7 +27,6 @@ const basePolicy: Policy = { outputCurrency: 'USD', type: 'team', owner: 'user@test.com', - isPolicyExpenseChatEnabled: true, harvesting: {enabled: false}, }; diff --git a/tests/unit/ReportUtilsTest.ts b/tests/unit/ReportUtilsTest.ts index f36ca6b25172..c9805ad5bcb9 100644 --- a/tests/unit/ReportUtilsTest.ts +++ b/tests/unit/ReportUtilsTest.ts @@ -544,7 +544,6 @@ const policy: Policy = { type: CONST.POLICY.TYPE.TEAM, owner: '', outputCurrency: '', - isPolicyExpenseChatEnabled: false, }; describe('ReportUtils', () => { @@ -4629,7 +4628,6 @@ describe('ReportUtils', () => { role: 'user', owner: '', outputCurrency: '', - isPolicyExpenseChatEnabled: false, } as const; const moneyRequestOptions = temporary_getMoneyRequestOptions( report, @@ -4690,7 +4688,6 @@ describe('ReportUtils', () => { role: 'user', owner: '', outputCurrency: '', - isPolicyExpenseChatEnabled: false, employeeList: { [currentUserEmail]: { email: currentUserEmail, @@ -4759,7 +4756,6 @@ describe('ReportUtils', () => { role: 'user', owner: currentUserEmail, outputCurrency: '', - isPolicyExpenseChatEnabled: false, }; Promise.all([ Onyx.merge(`${ONYXKEYS.COLLECTION.POLICY}${paidPolicy.id}`, paidPolicy), @@ -6033,7 +6029,6 @@ describe('ReportUtils', () => { type: CONST.POLICY.TYPE.CORPORATE, owner: '', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: false, employeeList: { 'lagertha2@vikings.net': { email: 'lagertha2@vikings.net', @@ -6296,7 +6291,6 @@ describe('ReportUtils', () => { type: CONST.POLICY.TYPE.CORPORATE, owner: '', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: false, employeeList: { 'lagertha2@vikings.net': { email: 'lagertha2@vikings.net', @@ -6987,7 +6981,6 @@ describe('ReportUtils', () => { role: CONST.POLICY.ROLE.USER, owner: `owner${ownerAccountID}@test.com`, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, approver: approverEmail, employeeList: { @@ -13886,7 +13879,6 @@ describe('ReportUtils', () => { role: CONST.POLICY.ROLE.ADMIN, owner: 'test@example.com', outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, autoReporting: true, autoReportingFrequency: CONST.POLICY.AUTO_REPORTING_FREQUENCIES.WEEKLY, harvesting: { @@ -16422,7 +16414,6 @@ describe('ReportUtils', () => { }, }, owner: currentUserEmail, - isPolicyExpenseChatEnabled: true, }; const chatReport: Report = { @@ -16510,7 +16501,6 @@ describe('ReportUtils', () => { }, }, owner: currentUserEmail, - isPolicyExpenseChatEnabled: true, }; const chatReport: Report = { @@ -16607,7 +16597,6 @@ describe('ReportUtils', () => { }, }, owner: currentUserEmail, - isPolicyExpenseChatEnabled: true, }; const chatReport: Report = { @@ -16717,7 +16706,6 @@ describe('ReportUtils', () => { }, }, owner: currentUserEmail, - isPolicyExpenseChatEnabled: true, }; const chatReport: Report = { @@ -16806,7 +16794,6 @@ describe('ReportUtils', () => { }, }, owner: currentUserEmail, - isPolicyExpenseChatEnabled: true, }; const chatReport: Report = { @@ -16893,7 +16880,6 @@ describe('ReportUtils', () => { }, }, owner: currentUserEmail, - isPolicyExpenseChatEnabled: true, }; const chatReport: Report = { @@ -16986,7 +16972,6 @@ describe('ReportUtils', () => { }, }, owner: currentUserEmail, - isPolicyExpenseChatEnabled: true, }; const chatReport: Report = { @@ -17132,7 +17117,6 @@ describe('ReportUtils', () => { }, }, owner: currentUserEmail, - isPolicyExpenseChatEnabled: true, }; const chatReport: Report = { @@ -17260,7 +17244,6 @@ describe('ReportUtils', () => { type: CONST.POLICY.TYPE.TEAM, owner: currentUserEmail, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, reimbursementChoice: CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_NO, }; @@ -17531,7 +17514,6 @@ describe('ReportUtils', () => { type: CONST.POLICY.TYPE.TEAM, owner: approverEmail, outputCurrency: CONST.CURRENCY.USD, - isPolicyExpenseChatEnabled: true, approver: approverEmail, approvalMode: CONST.POLICY.APPROVAL_MODE.BASIC, employeeList: { @@ -21105,7 +21087,6 @@ describe('ReportUtils', () => { type: CONST.POLICY.TYPE.CORPORATE, owner: 'test@test.com', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, taxRates: { taxes: { TAX_CODE_1: { diff --git a/tests/unit/Search/SearchUIUtilsTest.ts b/tests/unit/Search/SearchUIUtilsTest.ts index ddc5d8cde999..f9afa0af8e45 100644 --- a/tests/unit/Search/SearchUIUtilsTest.ts +++ b/tests/unit/Search/SearchUIUtilsTest.ts @@ -267,7 +267,6 @@ const policy: OnyxTypes.Policy = { id: 'Admin', name: 'Policy', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, approvalMode: 'ADVANCED', autoReimbursement: { limit: 0, @@ -8112,7 +8111,6 @@ describe('SearchUIUtils', () => { name: 'Test Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -8216,7 +8214,6 @@ describe('SearchUIUtils', () => { name: 'Test Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -8288,7 +8285,6 @@ describe('SearchUIUtils', () => { name: 'Test Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -8358,7 +8354,6 @@ describe('SearchUIUtils', () => { name: 'Test Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -8531,7 +8526,6 @@ describe('SearchUIUtils', () => { name: 'Personal Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: false, role: CONST.POLICY.ROLE.USER, type: CONST.POLICY.TYPE.PERSONAL, // personal policy, not team approvalMode: CONST.POLICY.APPROVAL_MODE.OPTIONAL, @@ -8562,7 +8556,6 @@ describe('SearchUIUtils', () => { name: 'Submit Workspace', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.USER, type: CONST.POLICY.TYPE.SUBMIT, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -8626,7 +8619,6 @@ describe('SearchUIUtils', () => { name: 'Submit Workspace', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.USER, type: CONST.POLICY.TYPE.SUBMIT, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -8669,7 +8661,6 @@ describe('SearchUIUtils', () => { name: 'Team Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.USER, // not admin type: CONST.POLICY.TYPE.TEAM, areCompanyCardsEnabled: false, @@ -8702,7 +8693,6 @@ describe('SearchUIUtils', () => { name: 'ACH Only Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -8746,7 +8736,6 @@ describe('SearchUIUtils', () => { name: 'Card Only Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -8782,7 +8771,6 @@ describe('SearchUIUtils', () => { name: 'Full Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -8837,7 +8825,6 @@ describe('SearchUIUtils', () => { name: 'ACH Only Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -8880,7 +8867,6 @@ describe('SearchUIUtils', () => { name: 'Test Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -8953,7 +8939,6 @@ describe('SearchUIUtils', () => { name: 'Test Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -9027,7 +9012,6 @@ describe('SearchUIUtils', () => { name: 'Test Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -9124,7 +9108,6 @@ describe('SearchUIUtils', () => { name: 'Test Policy', owner: adminEmail, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, approvalMode: CONST.POLICY.APPROVAL_MODE.ADVANCED, @@ -9526,7 +9509,6 @@ describe('SearchUIUtils', () => { }, type: 'corporate', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }, report_6523565988285061: { @@ -12485,7 +12467,6 @@ describe('SearchUIUtils', () => { role: CONST.POLICY.ROLE.USER, owner: 'owner@example.com', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, employeeList: {}, ...overrides, }); diff --git a/tests/unit/Search/handleActionButtonPressTest.ts b/tests/unit/Search/handleActionButtonPressTest.ts index 0376b64bd7fd..210ac00efb89 100644 --- a/tests/unit/Search/handleActionButtonPressTest.ts +++ b/tests/unit/Search/handleActionButtonPressTest.ts @@ -109,7 +109,6 @@ const mockReportItemWithHold = createMock({ owner: 'apb@apb.com', name: 'Policy', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }, reportAction: { reportActionID: '3042630993757922770', @@ -211,7 +210,6 @@ const mockReportItemWithHold = createMock({ owner: 'apb@apb.com', name: 'Policy', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }, reportAction: { reportActionID: '3042630993757922770', @@ -308,7 +306,6 @@ mockSnapshotData[policyKey] = createMock({ owner: 'apb@apb.com', ownerAccountID: mockReportItemWithHold.ownerAccountID, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }); mockSnapshotData[reportKey] = createMock({ reportID: mockReportItemWithHold.reportID, diff --git a/tests/unit/Search/yourSpendApplicabilityTest.ts b/tests/unit/Search/yourSpendApplicabilityTest.ts index fac6d00bded5..bcc3573caad3 100644 --- a/tests/unit/Search/yourSpendApplicabilityTest.ts +++ b/tests/unit/Search/yourSpendApplicabilityTest.ts @@ -17,7 +17,6 @@ function makePolicy(overrides: Partial = {}): Policy { role: 'admin', owner: 'test@example.com', ownerAccountID: 1, - isPolicyExpenseChatEnabled: true, outputCurrency: CONST.CURRENCY.USD, ...overrides, } as Policy; diff --git a/tests/unit/TodosUtilsTest.ts b/tests/unit/TodosUtilsTest.ts index 76af69086717..425533b4b8ea 100644 --- a/tests/unit/TodosUtilsTest.ts +++ b/tests/unit/TodosUtilsTest.ts @@ -42,7 +42,6 @@ const createMockPolicy = (policyID: string, overrides: Partial = {}): Po type: CONST.POLICY.TYPE.TEAM, owner: CURRENT_USER_EMAIL, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.BASIC, ...overrides, }); diff --git a/tests/unit/TransactionGroupListItemTest.tsx b/tests/unit/TransactionGroupListItemTest.tsx index 018465698a13..7240bc031381 100644 --- a/tests/unit/TransactionGroupListItemTest.tsx +++ b/tests/unit/TransactionGroupListItemTest.tsx @@ -137,7 +137,6 @@ const mockTransaction: TransactionListItemType = { owner: 'test@test.com', name: 'Policy', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, }, reportAction: { reportActionID: '2454187434077044186', diff --git a/tests/unit/ViolationUtilsTest.ts b/tests/unit/ViolationUtilsTest.ts index c9a4e8290737..8429995f05cb 100644 --- a/tests/unit/ViolationUtilsTest.ts +++ b/tests/unit/ViolationUtilsTest.ts @@ -3249,7 +3249,6 @@ describe('getViolations', () => { type: CONST.POLICY.TYPE.TEAM, role: CONST.POLICY.ROLE.ADMIN, owner: CARLOS_EMAIL, - isPolicyExpenseChatEnabled: false, autoReporting: true, autoReportingFrequency: CONST.POLICY.AUTO_REPORTING_FREQUENCIES.WEEKLY, outputCurrency: CONST.CURRENCY.USD, @@ -3308,7 +3307,6 @@ describe('getViolations', () => { type: CONST.POLICY.TYPE.TEAM, role: CONST.POLICY.ROLE.ADMIN, owner: CARLOS_EMAIL, - isPolicyExpenseChatEnabled: false, autoReporting: true, autoReportingFrequency: CONST.POLICY.AUTO_REPORTING_FREQUENCIES.INSTANT, outputCurrency: CONST.CURRENCY.USD, @@ -3367,7 +3365,6 @@ describe('getViolations', () => { type: CONST.POLICY.TYPE.TEAM, role: CONST.POLICY.ROLE.ADMIN, owner: CARLOS_EMAIL, - isPolicyExpenseChatEnabled: false, autoReporting: true, autoReportingFrequency: CONST.POLICY.AUTO_REPORTING_FREQUENCIES.MONTHLY, outputCurrency: CONST.CURRENCY.USD, diff --git a/tests/unit/WorkflowUtilsTest.ts b/tests/unit/WorkflowUtilsTest.ts index d4654850aa7b..b849651f4fa3 100644 --- a/tests/unit/WorkflowUtilsTest.ts +++ b/tests/unit/WorkflowUtilsTest.ts @@ -320,7 +320,6 @@ describe('WorkflowUtils', () => { type: 'team' as const, owner: 'owner@example.com', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, employeeList: employees, approver: defaultApprover, }); diff --git a/tests/unit/canEditFieldOfMoneyRequestTest.ts b/tests/unit/canEditFieldOfMoneyRequestTest.ts index 77ae864adc61..fbd5ec88a779 100644 --- a/tests/unit/canEditFieldOfMoneyRequestTest.ts +++ b/tests/unit/canEditFieldOfMoneyRequestTest.ts @@ -27,7 +27,6 @@ const policy: Policy = { type: CONST.POLICY.TYPE.TEAM, owner: '', outputCurrency: '', - isPolicyExpenseChatEnabled: false, }; describe('canEditFieldOfMoneyRequest', () => { @@ -680,7 +679,6 @@ describe('canEditFieldOfMoneyRequest', () => { id: PER_DIEM_POLICY_ID, role: CONST.POLICY.ROLE.ADMIN, arePerDiemRatesEnabled: true, - isPolicyExpenseChatEnabled: true, customUnits: { [PER_DIEM_CUSTOM_UNIT_ID]: { customUnitID: PER_DIEM_CUSTOM_UNIT_ID, diff --git a/tests/unit/hooks/useBrokenDirectCompanyCardFeedsForAdmin.test.ts b/tests/unit/hooks/useBrokenDirectCompanyCardFeedsForAdmin.test.ts index c18903637b21..98118d3a6aed 100644 --- a/tests/unit/hooks/useBrokenDirectCompanyCardFeedsForAdmin.test.ts +++ b/tests/unit/hooks/useBrokenDirectCompanyCardFeedsForAdmin.test.ts @@ -53,7 +53,6 @@ function createAdminPolicy(overrides: Partial & {id: string} = {id: POLI name: POLICY_NAME, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, policyAccountID: WORKSPACE_ACCOUNT_ID, ...overrides, } as Policy; diff --git a/tests/unit/hooks/useDefaultParticipants.test.ts b/tests/unit/hooks/useDefaultParticipants.test.ts index d53e31ec02f5..d08db5015173 100644 --- a/tests/unit/hooks/useDefaultParticipants.test.ts +++ b/tests/unit/hooks/useDefaultParticipants.test.ts @@ -23,7 +23,7 @@ const mockSelfDMReport: Report = createSelfDM(1, ACCOUNT_ID); const workspaceChat: Report = {...createPolicyExpenseChat(2), policyID: POLICY_ID, ownerAccountID: ACCOUNT_ID}; // Auto-reporting is on, so without the track-expense carve-out the default target resolves to the workspace chat. -const mockDefaultExpensePolicy: Policy = {...createRandomPolicy(2, CONST.POLICY.TYPE.TEAM), id: POLICY_ID, isPolicyExpenseChatEnabled: true, autoReporting: true}; +const mockDefaultExpensePolicy: Policy = {...createRandomPolicy(2, CONST.POLICY.TYPE.TEAM), id: POLICY_ID, autoReporting: true}; jest.mock('@hooks/useCurrentUserPersonalDetails', () => ({ __esModule: true, diff --git a/tests/unit/hooks/useSelectedTransactionsActions.test.ts b/tests/unit/hooks/useSelectedTransactionsActions.test.ts index 90123d762fbc..243c7c361edd 100644 --- a/tests/unit/hooks/useSelectedTransactionsActions.test.ts +++ b/tests/unit/hooks/useSelectedTransactionsActions.test.ts @@ -892,7 +892,6 @@ describe('useSelectedTransactionsActions', () => { }; const policy = { ...createRandomPolicy(1), - isPolicyExpenseChatEnabled: true, role: CONST.POLICY.ROLE.ADMIN, employeeList: { [CURRENT_USER_LOGIN]: {role: CONST.POLICY.ROLE.ADMIN}, diff --git a/tests/unit/hooks/useShowNotFoundPageInIOUStep.test.ts b/tests/unit/hooks/useShowNotFoundPageInIOUStep.test.ts index a5950d1107aa..0e14f4d2a2a2 100644 --- a/tests/unit/hooks/useShowNotFoundPageInIOUStep.test.ts +++ b/tests/unit/hooks/useShowNotFoundPageInIOUStep.test.ts @@ -70,7 +70,6 @@ describe('useShowNotFoundPageInIOUStep', () => { name: 'Corporate policy', owner: '', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: false, employeeList: { [currentUserEmail]: { email: currentUserEmail, diff --git a/tests/unit/inlineEditing/TransactionInlineEdit.test.ts b/tests/unit/inlineEditing/TransactionInlineEdit.test.ts index 9bad8efebf93..cb99d3b27550 100644 --- a/tests/unit/inlineEditing/TransactionInlineEdit.test.ts +++ b/tests/unit/inlineEditing/TransactionInlineEdit.test.ts @@ -81,7 +81,6 @@ describe('TransactionInlineEdit', () => { type: CONST.POLICY.TYPE.TEAM, owner: '', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: false, areCategoriesEnabled: true, }; diff --git a/tests/unit/pages/home/TimeSensitiveSection/UnlockBankAccountTest.tsx b/tests/unit/pages/home/TimeSensitiveSection/UnlockBankAccountTest.tsx index 8d08765bfccc..c5156d4fce24 100644 --- a/tests/unit/pages/home/TimeSensitiveSection/UnlockBankAccountTest.tsx +++ b/tests/unit/pages/home/TimeSensitiveSection/UnlockBankAccountTest.tsx @@ -101,7 +101,6 @@ describe('TimeSensitiveSection - UnlockBankAccount', () => { name: POLICY_NAME, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, achAccount: { bankAccountID: LOCKED_BANK_ACCOUNT_ID, accountNumber: 'XXXXXXXX1234', @@ -186,7 +185,6 @@ describe('TimeSensitiveSection - UnlockBankAccount', () => { name: POLICY_NAME, role: CONST.POLICY.ROLE.USER, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, achAccount: { bankAccountID: LOCKED_BANK_ACCOUNT_ID, accountNumber: 'XXXXXXXX1234', @@ -214,7 +212,6 @@ describe('TimeSensitiveSection - UnlockBankAccount', () => { name: 'Workspace One', role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, achAccount: { bankAccountID: LOCKED_BANK_ACCOUNT_ID, accountNumber: 'XXXXXXXX1234', @@ -230,7 +227,6 @@ describe('TimeSensitiveSection - UnlockBankAccount', () => { name: 'Workspace Two', role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, achAccount: { bankAccountID: SECOND_LOCKED_BANK_ACCOUNT_ID, accountNumber: 'XXXXXXXX5678', @@ -261,7 +257,6 @@ describe('TimeSensitiveSection - UnlockBankAccount', () => { name: 'Workspace One', role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, achAccount: { bankAccountID: LOCKED_BANK_ACCOUNT_ID, accountNumber: 'XXXXXXXX1234', @@ -277,7 +272,6 @@ describe('TimeSensitiveSection - UnlockBankAccount', () => { name: 'Workspace Two', role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, achAccount: { bankAccountID: LOCKED_BANK_ACCOUNT_ID, accountNumber: 'XXXXXXXX5678', @@ -310,7 +304,6 @@ describe('TimeSensitiveSection - UnlockBankAccount', () => { name: POLICY_NAME, role: CONST.POLICY.ROLE.ADMIN, type: CONST.POLICY.TYPE.TEAM, - isPolicyExpenseChatEnabled: true, achAccount: { bankAccountID: LOCKED_BANK_ACCOUNT_ID, accountNumber: 'XXXXXXXX1234', diff --git a/tests/unit/shouldUseDefaultExpensePolicyTest.ts b/tests/unit/shouldUseDefaultExpensePolicyTest.ts index 91d316cee363..7c64e95d995f 100644 --- a/tests/unit/shouldUseDefaultExpensePolicyTest.ts +++ b/tests/unit/shouldUseDefaultExpensePolicyTest.ts @@ -18,7 +18,6 @@ function makePaidGroupPolicy(overrides?: Partial): Policy { ...createRandomPolicy(Number(TEAM_POLICY_ID), CONST.POLICY.TYPE.TEAM), id: TEAM_POLICY_ID, ownerAccountID: OWNER_ACCOUNT_ID, - isPolicyExpenseChatEnabled: true, ...overrides, }; } @@ -50,7 +49,7 @@ describe('shouldUseDefaultExpensePolicy', () => { }); it('returns false when defaultExpensePolicy is not a group policy', () => { - const policy = makePaidGroupPolicy({type: CONST.POLICY.TYPE.PERSONAL, isPolicyExpenseChatEnabled: false}); + const policy = makePaidGroupPolicy({type: CONST.POLICY.TYPE.PERSONAL}); expect(shouldUseDefaultExpensePolicy(CONST.IOU.TYPE.CREATE, policy, undefined, undefined, undefined, CONST.DEFAULT_NUMBER_ID)).toBeFalsy(); }); diff --git a/tests/unit/useGetExpensifyCardFromReportActionTest.ts b/tests/unit/useGetExpensifyCardFromReportActionTest.ts index cdf5b6d2c324..3c4ccc037515 100644 --- a/tests/unit/useGetExpensifyCardFromReportActionTest.ts +++ b/tests/unit/useGetExpensifyCardFromReportActionTest.ts @@ -68,7 +68,6 @@ describe('useGetExpensifyCardFromReportAction', () => { isAttendeeTrackingEnabled: false, owner: '1', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: false, policyAccountID: 123, }; @@ -144,7 +143,6 @@ describe('useGetExpensifyCardFromReportAction', () => { isAttendeeTrackingEnabled: false, owner: '1', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: false, policyAccountID: 123, }); }); @@ -213,7 +211,6 @@ describe('useGetExpensifyCardFromReportAction', () => { isAttendeeTrackingEnabled: false, owner: '1', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: false, policyAccountID: 123, }); @@ -246,7 +243,6 @@ describe('useGetExpensifyCardFromReportAction', () => { isAttendeeTrackingEnabled: false, owner: '1', outputCurrency: 'USD', - isPolicyExpenseChatEnabled: false, policyAccountID: 123, }; mockUsePolicy.mockReturnValue(testPolicy); diff --git a/tests/unit/useTodoCountsTest.tsx b/tests/unit/useTodoCountsTest.tsx index b62ceb04feb5..cd4cc40c8ca2 100644 --- a/tests/unit/useTodoCountsTest.tsx +++ b/tests/unit/useTodoCountsTest.tsx @@ -46,7 +46,6 @@ const createMockPolicy = (policyID: string, overrides: Partial = {}): Po type: CONST.POLICY.TYPE.TEAM, owner: CURRENT_USER_EMAIL, outputCurrency: 'USD', - isPolicyExpenseChatEnabled: true, approvalMode: CONST.POLICY.APPROVAL_MODE.BASIC, ...overrides, }); diff --git a/tests/utils/LHNTestUtils.tsx b/tests/utils/LHNTestUtils.tsx index 1b53c8ca1a96..3afc9e5b41d5 100644 --- a/tests/utils/LHNTestUtils.tsx +++ b/tests/utils/LHNTestUtils.tsx @@ -249,7 +249,6 @@ function getFakePolicy(id = '1', name = 'Workspace-Test-001'): Policy { outputCurrency: 'BRL', avatarURL: '', employeeList: {}, - isPolicyExpenseChatEnabled: true, lastModified: '1697323926777105', autoReporting: true, autoReportingFrequency: 'immediate', diff --git a/tests/utils/collections/policies.ts b/tests/utils/collections/policies.ts index 4d2829ca3060..67867e3a9dc3 100644 --- a/tests/utils/collections/policies.ts +++ b/tests/utils/collections/policies.ts @@ -13,7 +13,6 @@ export default function createRandomPolicy(index: number, type?: ValueOf policyType !== CONST.POLICY.TYPE.SUBMIT)), autoReporting: randBoolean(), - isPolicyExpenseChatEnabled: randBoolean(), autoReportingFrequency: rand( Object.values(CONST.POLICY.AUTO_REPORTING_FREQUENCIES).filter( (frequency): frequency is Exclude, typeof CONST.POLICY.AUTO_REPORTING_FREQUENCIES.MANUAL> =>