Rename hook to useDismissOnMoneyRequestReportRemoval

Author: TMisiukiewicz

Mobile-Expensify

@@ -11,7 +11,7 @@ import usePrevious from './usePrevious';
  * Skips dismissal during route changes — the new report's data may not be loaded yet,
  * so the absent `report` should not be interpreted as removal.
  */
-function useDismissOnReportRemoval(reportIDFromRoute: string | undefined) {
+function useDismissOnMoneyRequestReportRemoval(reportIDFromRoute: string | undefined) {
     const [report] = useOnyx(`${ONYXKEYS.COLLECTION.REPORT}${reportIDFromRoute}`);
     const prevReport = usePrevious(report);
     const prevReportIDFromRoute = usePrevious(reportIDFromRoute);
@@ -39,4 +39,4 @@ function useDismissOnReportRemoval(reportIDFromRoute: string | undefined) {
     }, [report, isFocused, prevReport, prevReportIDFromRoute, reportIDFromRoute]);
 }
 
-export default useDismissOnReportRemoval;
+export default useDismissOnMoneyRequestReportRemoval;

src/hooks/useDismissOnReportRemoval.ts …useDismissOnMoneyRequestReportRemoval.tssrc/hooks/useDismissOnReportRemoval.ts renamed to src/hooks/useDismissOnMoneyRequestReportRemoval.ts src/hooks/useDismissOnReportRemoval.ts renamed to src/hooks/useDismissOnMoneyRequestReportRemoval.ts

@@ -13,7 +13,7 @@ import useShowSuperWideRHPVersion from '@components/WideRHPContextProvider/useSh
 import WideRHPOverlayWrapper from '@components/WideRHPOverlayWrapper';
 import useActionListContextValue from '@hooks/useActionListContextValue';
 import useCurrentUserPersonalDetails from '@hooks/useCurrentUserPersonalDetails';
-import useDismissOnReportRemoval from '@hooks/useDismissOnReportRemoval';
+import useDismissOnMoneyRequestReportRemoval from '@hooks/useDismissOnMoneyRequestReportRemoval';
 import useDocumentTitle from '@hooks/useDocumentTitle';
 import useIsReportReadyToDisplay from '@hooks/useIsReportReadyToDisplay';
 import useNetwork from '@hooks/useNetwork';
@@ -88,7 +88,7 @@ function SearchMoneyRequestReportPage({route}: SearchMoneyRequestPageProps) {
     const {email: currentUserEmail, accountID: currentUserAccountID} = useCurrentUserPersonalDetails();
     const isFocused = useIsFocused();
 
-    useDismissOnReportRemoval(reportIDFromRoute);
+    useDismissOnMoneyRequestReportRemoval(reportIDFromRoute);
 
     useEffect(() => {
         // Update last visit time when the expense super wide RHP report is focused

src/pages/Search/SearchMoneyRequestReportPage.tsx

@@ -1,7 +1,7 @@
 import type * as ReactNavigation from '@react-navigation/native';
 import {renderHook} from '@testing-library/react-native';
 import Onyx from 'react-native-onyx';
-import useDismissOnReportRemoval from '@hooks/useDismissOnReportRemoval';
+import useDismissOnMoneyRequestReportRemoval from '@hooks/useDismissOnMoneyRequestReportRemoval';
 import Navigation from '@navigation/Navigation';
 import CONST from '@src/CONST';
 import ONYXKEYS from '@src/ONYXKEYS';
@@ -41,7 +41,7 @@ function buildChatReport(id: string): Report {
     } as Report;
 }
 
-describe('useDismissOnReportRemoval', () => {
+describe('useDismissOnMoneyRequestReportRemoval', () => {
     beforeAll(() => {
         Onyx.init({keys: ONYXKEYS});
     });
@@ -57,7 +57,7 @@ describe('useDismissOnReportRemoval', () => {
         await Onyx.set(`${ONYXKEYS.COLLECTION.REPORT}${REPORT_A_ID}`, buildMoneyRequestReport(REPORT_A_ID));
         await waitForBatchedUpdates();
 
-        const {rerender} = renderHook(({reportID}: {reportID: string}) => useDismissOnReportRemoval(reportID), {
+        const {rerender} = renderHook(({reportID}: {reportID: string}) => useDismissOnMoneyRequestReportRemoval(reportID), {
             initialProps: {reportID: REPORT_A_ID},
         });
 
@@ -69,7 +69,7 @@ describe('useDismissOnReportRemoval', () => {
     });
 
     it('does not dismiss the modal on first render even if the report is missing', async () => {
-        renderHook(({reportID}: {reportID: string}) => useDismissOnReportRemoval(reportID), {
+        renderHook(({reportID}: {reportID: string}) => useDismissOnMoneyRequestReportRemoval(reportID), {
             initialProps: {reportID: REPORT_A_ID},
         });
         await waitForBatchedUpdates();
@@ -82,7 +82,7 @@ describe('useDismissOnReportRemoval', () => {
         await Onyx.set(`${ONYXKEYS.COLLECTION.REPORT}${REPORT_A_ID}`, buildMoneyRequestReport(REPORT_A_ID));
         await waitForBatchedUpdates();
 
-        const {rerender} = renderHook(({reportID}: {reportID: string}) => useDismissOnReportRemoval(reportID), {
+        const {rerender} = renderHook(({reportID}: {reportID: string}) => useDismissOnMoneyRequestReportRemoval(reportID), {
             initialProps: {reportID: REPORT_A_ID},
         });
 
@@ -97,7 +97,7 @@ describe('useDismissOnReportRemoval', () => {
         await Onyx.set(`${ONYXKEYS.COLLECTION.REPORT}${REPORT_A_ID}`, buildChatReport(REPORT_A_ID));
         await waitForBatchedUpdates();
 
-        const {rerender} = renderHook(({reportID}: {reportID: string}) => useDismissOnReportRemoval(reportID), {
+        const {rerender} = renderHook(({reportID}: {reportID: string}) => useDismissOnMoneyRequestReportRemoval(reportID), {
             initialProps: {reportID: REPORT_A_ID},
         });
 
@@ -115,7 +115,7 @@ describe('useDismissOnReportRemoval', () => {
         await Onyx.set(`${ONYXKEYS.COLLECTION.REPORT}${REPORT_A_ID}`, buildMoneyRequestReport(REPORT_A_ID));
         await waitForBatchedUpdates();
 
-        const {rerender} = renderHook(({reportID}: {reportID: string}) => useDismissOnReportRemoval(reportID), {
+        const {rerender} = renderHook(({reportID}: {reportID: string}) => useDismissOnMoneyRequestReportRemoval(reportID), {
             initialProps: {reportID: REPORT_A_ID},
         });
 
@@ -132,7 +132,7 @@ describe('useDismissOnReportRemoval', () => {
         await Onyx.set(`${ONYXKEYS.COLLECTION.REPORT}${REPORT_A_ID}`, buildMoneyRequestReport(REPORT_A_ID, {stateNum: CONST.REPORT.STATE_NUM.OPEN}));
         await waitForBatchedUpdates();
 
-        const {rerender} = renderHook(({reportID}: {reportID: string}) => useDismissOnReportRemoval(reportID), {
+        const {rerender} = renderHook(({reportID}: {reportID: string}) => useDismissOnMoneyRequestReportRemoval(reportID), {
             initialProps: {reportID: REPORT_A_ID},
         });
 
@@ -161,7 +161,7 @@ describe('useDismissOnReportRemoval', () => {
         await Onyx.set(`${ONYXKEYS.COLLECTION.REPORT}${REPORT_A_ID}`, buildMoneyRequestReport(REPORT_A_ID, {stateNum: CONST.REPORT.STATE_NUM.OPEN}));
         await waitForBatchedUpdates();
 
-        const {rerender} = renderHook(({reportID}: {reportID: string}) => useDismissOnReportRemoval(reportID), {
+        const {rerender} = renderHook(({reportID}: {reportID: string}) => useDismissOnMoneyRequestReportRemoval(reportID), {
             initialProps: {reportID: REPORT_A_ID},
         });