UIQM-588 Updated tests

src/QuickMarcEditor/QuickMarcEditorContainer.js

@@ -95,7 +95,7 @@ const QuickMarcEditorContainer = ({
     && action !== QUICK_MARC_ACTIONS.CREATE;
 
   const showCallout = useShowCallout();
-  const { linksCount } = useAuthorityLinksCount({ id: externalId });
+  const { linksCount } = useAuthorityLinksCount({ id: marcType === MARC_TYPES.AUTHORITY && externalId });
 
   const closeEditor = useCallback((id) => {
     if (marcType === MARC_TYPES.HOLDINGS && action !== QUICK_MARC_ACTIONS.CREATE) {

src/QuickMarcEditor/QuickMarcEditorContainer.test.js

@@ -160,10 +160,6 @@ describe('Given Quick Marc Editor Container', () => {
 
   describe('when the marc type is authority', () => {
     it('should make a request to get the number of links', async () => {
-      useAuthorityLinksCount.mockClear().mockReturnValue({
-        fetchLinksCount: mockFetchLinksCount,
-      });
-
       await act(async () => {
         await renderQuickMarcEditorContainer({
           mutator,
@@ -174,7 +170,7 @@ describe('Given Quick Marc Editor Container', () => {
         });
       });
 
-      expect(mockFetchLinksCount).toHaveBeenCalledWith([match.params.externalId]);
+      expect(useAuthorityLinksCount).toHaveBeenCalledWith({ id: match.params.externalId });
     });
   });
 

src/queries/useAuthorityLinksCount/useAuthorityLinksCount.js

@@ -17,6 +17,7 @@ const useAuthorityLinksCount = ({ id, tenantId } = {}) => {
     () => ky.get('search/authorities', { searchParams }).json(),
     {
       keepPreviousData: true,
+      enabled: Boolean(id),
     },
   );
 

src/queries/useAuthorityLinksCount/useAuthorityLinksCount.test.js

@@ -4,16 +4,11 @@ import {
 } from 'react-query';
 import { renderHook } from '@folio/jest-config-stripes/testing-library/react';
 
-import '../../../test/jest/__mock__';
-
 import { useOkapiKy } from '@folio/stripes/core';
 
 import useAuthorityLinksCount from './useAuthorityLinksCount';
 
-jest.mock('react-router-dom', () => ({
-  ...jest.requireActual('react-router-dom'),
-  useLocation: jest.fn(),
-}));
+import '../../../test/jest/__mock__';
 
 const queryClient = new QueryClient();
 
@@ -23,22 +18,30 @@ const wrapper = ({ children }) => (
   </QueryClientProvider>
 );
 
-const mockPost = jest.fn().mockReturnValue({
-  json: jest.fn().mockResolvedValue({ links: [] }),
+const mockGet = jest.fn().mockReturnValue({
+  json: jest.fn().mockResolvedValue({ authorities: [{ numberOfTitles: 2 }] }),
 });
 
 describe('Given useAuthorityLinksCount', () => {
   beforeEach(() => {
     useOkapiKy.mockClear().mockReturnValue({
-      post: mockPost,
+      get: mockGet,
     });
   });
 
-  it('should fetch links count', async () => {
-    const { result } = renderHook(() => useAuthorityLinksCount(), { wrapper });
+  it('should fetch authorities', async () => {
+    renderHook(() => useAuthorityLinksCount({ id: 'fakeId' }), { wrapper });
+
+    expect(mockGet).toHaveBeenCalledWith('search/authorities', {
+      searchParams: {
+        query: '(id==fakeId and authRefType==("Authorized"))',
+      },
+    });
+  });
 
-    await result.current.fetchLinksCount(['fakeId']);
+  it('should return links count', async () => {
+    const { result } = renderHook(() => useAuthorityLinksCount({ id: 'fakeId' }), { wrapper });
 
-    expect(mockPost).toHaveBeenCalledWith('links/authorities/bulk/count', { json: { ids: ['fakeId'] } });
+    expect(result.current.linksCount).toEqual(2);
   });
 });