UIORGS-390 rename hook useBankingInformation to useBankingInformation…

…Settings: to not confuse domens
folio-org · Nov 9, 2023 · 7b6029e · 7b6029e
1 parent e2d4e81
commit 7b6029e
Show file tree

Hide file tree

Showing 9 changed files with 19 additions and 19 deletions.
diff --git a/src/Settings/BankingInformationSettings/BankingInformationSettings.js b/src/Settings/BankingInformationSettings/BankingInformationSettings.js
@@ -5,7 +5,7 @@ import { Loading } from '@folio/stripes/components';
 import { useOkapiKy } from '@folio/stripes/core';
 
 import { SETTINGS_API } from '../../common/constants';
-import { useBankingInformation } from '../../common/hooks';
+import { useBankingInformationSettings } from '../../common/hooks';
 import BankingInformationSettingsForm from './BankingInformationSettingsForm';
 
 const BankingInformationSettings = () => {
@@ -14,7 +14,7 @@ const BankingInformationSettings = () => {
     bankingInformation,
     isLoading,
     refetch,
-  } = useBankingInformation();
+  } = useBankingInformationSettings();
   const ky = useOkapiKy();
   const sendCallout = useShowCallout();
 

diff --git a/src/Settings/BankingInformationSettings/BankingInformationSettings.test.js b/src/Settings/BankingInformationSettings/BankingInformationSettings.test.js
@@ -8,7 +8,7 @@ import {
 import user from '@folio/jest-config-stripes/testing-library/user-event';
 import { useOkapiKy } from '@folio/stripes/core';
 
-import { useBankingInformation } from '../../common/hooks';
+import { useBankingInformationSettings } from '../../common/hooks';
 import BankingInformationSettings from './BankingInformationSettings';
 
 const mockRefetch = jest.fn();
@@ -19,7 +19,7 @@ jest.mock('@folio/stripes/components', () => ({
 }));
 
 jest.mock('../hooks', () => ({
-  useBankingInformation: jest.fn(() => ({
+  useBankingInformationSettings: jest.fn(() => ({
     isLoading: false,
     enabled: false,
     refetch: mockRefetch,
@@ -43,7 +43,7 @@ describe('BankingInformationSettings component', () => {
   });
 
   it('should render Loading component', () => {
-    useBankingInformation.mockReturnValue({
+    useBankingInformationSettings.mockReturnValue({
       isLoading: true,
       enabled: false,
     });
@@ -54,7 +54,7 @@ describe('BankingInformationSettings component', () => {
   });
 
   it('should save banking options', async () => {
-    useBankingInformation.mockClear().mockReturnValue({
+    useBankingInformationSettings.mockClear().mockReturnValue({
       isLoading: false,
       enabled: true,
       refetch: mockRefetch,

diff --git a/src/Settings/SettingsPage.js b/src/Settings/SettingsPage.js
@@ -3,7 +3,7 @@ import { FormattedMessage } from 'react-intl';
 
 import { Settings } from '@folio/stripes/smart-components';
 
-import { useBankingInformation } from '../common/hooks';
+import { useBankingInformationSettings } from '../common/hooks';
 import { CategorySettings } from './CategorySettings';
 import { TypeSettings } from './TypeSettings';
 import { BankingAccountTypeSettings } from './BankingAccountTypeSettings';
@@ -38,7 +38,7 @@ const bankingAccountTypesPage = {
 };
 
 const SettingsPage = (props) => {
-  const { enabled } = useBankingInformation();
+  const { enabled } = useBankingInformationSettings();
 
   const settingsPages = useMemo(() => (enabled ? pages.concat(bankingAccountTypesPage) : pages), [enabled]);
 

diff --git a/src/Settings/SettingsPage.test.js b/src/Settings/SettingsPage.test.js
@@ -3,7 +3,7 @@ import { MemoryRouter } from 'react-router-dom';
 
 import { screen, render } from '@folio/jest-config-stripes/testing-library/react';
 
-import { useBankingInformation } from '../common/hooks';
+import { useBankingInformationSettings } from '../common/hooks';
 import SettingsPage from './SettingsPage';
 
 jest.mock('@folio/stripes/core');
@@ -14,7 +14,7 @@ jest.mock('@folio/stripes/components', () => ({
   Loading: () => <div>Loading</div>,
 }));
 jest.mock('./hooks', () => ({
-  useBankingInformation: jest.fn(() => ({
+  useBankingInformationSettings: jest.fn(() => ({
     isLoading: false,
     enabled: false,
   })),
@@ -52,7 +52,7 @@ describe('SettingsPage', () => {
   });
 
   it('should return banking account types link', async () => {
-    useBankingInformation.mockReturnValue({
+    useBankingInformationSettings.mockReturnValue({
       isLoading: false,
       enabled: true,
     });
@@ -62,8 +62,8 @@ describe('SettingsPage', () => {
     expect(screen.getByText('ui-organizations.settings.bankingAccountTypes')).toBeInTheDocument();
   });
 
-  it('should display loading on fetching useBankingInformation', async () => {
-    useBankingInformation.mockReturnValue({
+  it('should display loading on fetching useBankingInformationSettings', async () => {
+    useBankingInformationSettings.mockReturnValue({
       isLoading: true,
       enabled: false,
     });

diff --git a/src/common/hooks/index.js b/src/common/hooks/index.js
@@ -1,5 +1,5 @@
 export * from './useAcqMethods';
-export * from './useBankingInformation';
+export * from './useBankingInformationSettings';
 export * from './useIntegrationConfig';
 export * from './useIntegrationConfigMutation';
 export * from './useLinkedAgreements';

diff --git a/src/common/hooks/useBankingInformation/index.js b/src/common/hooks/useBankingInformation/index.js
diff --git a/src/common/hooks/useBankingInformationSettings/index.js b/src/common/hooks/useBankingInformationSettings/index.js
@@ -0,0 +1 @@
+export { useBankingInformationSettings } from './useBankingInformationSettings';
diff --git a/...nkingInformation/useBankingInformation.js → ...Settings/useBankingInformationSettings.js b/...nkingInformation/useBankingInformation.js → ...Settings/useBankingInformationSettings.js
@@ -13,7 +13,7 @@ import {
 
 const DEFAULT_DATA = {};
 
-export const useBankingInformation = () => {
+export const useBankingInformationSettings = () => {
   const ky = useOkapiKy();
   const [namespace] = useNamespace({ key: 'banking-information-settings' });
 

diff --git a/...Information/useBankingInformation.test.js → ...ngs/useBankingInformationSettings.test.js b/...Information/useBankingInformation.test.js → ...ngs/useBankingInformationSettings.test.js
@@ -10,7 +10,7 @@ import {
 import { useOkapiKy } from '@folio/stripes/core';
 
 import { BANKING_INFORMATION_ENABLED_KEY } from '../../constants';
-import { useBankingInformation } from './useBankingInformation';
+import { useBankingInformationSettings } from './useBankingInformationSettings';
 
 const queryClient = new QueryClient();
 
@@ -28,7 +28,7 @@ const wrapper = ({ children }) => (
   </QueryClientProvider>
 );
 
-describe('useBankingInformation', () => {
+describe('useBankingInformationSettings', () => {
   beforeEach(() => {
     useOkapiKy
       .mockClear()
@@ -40,7 +40,7 @@ describe('useBankingInformation', () => {
   });
 
   it('should fetch banking information settings', async () => {
-    const { result } = renderHook(() => useBankingInformation(), { wrapper });
+    const { result } = renderHook(() => useBankingInformationSettings(), { wrapper });
 
     await waitFor(() => expect(result.current.isLoading).toBeFalsy());
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		export { useBankingInformationSettings } from './useBankingInformationSettings';