replace "noop" in orcidlink testing with "noOp" in common [URO-209]

kbase · Jun 20, 2024 · 1aba3e3 · 1aba3e3
1 parent 0c65156
commit 1aba3e3
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 23 deletions.
diff --git a/src/features/orcidlink/HomeLinked/ManageTab.test.tsx b/src/features/orcidlink/HomeLinked/ManageTab.test.tsx
@@ -2,8 +2,8 @@ import { screen } from '@testing-library/dom';
 import { render, waitFor } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { MemoryRouter } from 'react-router-dom';
+import { noOp } from '../../common';
 import { LINK_RECORD_1, PROFILE_1 } from '../test/data';
-import { noop } from '../test/mocks';
 import ManageTab from './ManageTab';
 
 describe('The ManageTab component', () => {
@@ -18,8 +18,8 @@ describe('The ManageTab component', () => {
           linkRecord={linkRecord}
           profile={profile}
           orcidSiteURL={orcidSiteURL}
-          removeLink={noop}
-          toggleShowInProfile={noop}
+          removeLink={noOp}
+          toggleShowInProfile={noOp}
         />
       </MemoryRouter>
     );
@@ -45,8 +45,8 @@ describe('The ManageTab component', () => {
           linkRecord={linkRecord}
           profile={profile}
           orcidSiteURL={orcidSiteURL}
-          removeLink={noop}
-          toggleShowInProfile={noop}
+          removeLink={noOp}
+          toggleShowInProfile={noOp}
         />
       </MemoryRouter>
     );
@@ -95,7 +95,7 @@ describe('The ManageTab component', () => {
           profile={profile}
           orcidSiteURL={orcidSiteURL}
           removeLink={removeLink}
-          toggleShowInProfile={noop}
+          toggleShowInProfile={noOp}
         />
       </MemoryRouter>
     );
@@ -137,8 +137,8 @@ describe('The ManageTab component', () => {
           linkRecord={linkRecord}
           profile={profile}
           orcidSiteURL={orcidSiteURL}
-          removeLink={noop}
-          toggleShowInProfile={noop}
+          removeLink={noOp}
+          toggleShowInProfile={noOp}
         />
       </MemoryRouter>
     );
@@ -183,7 +183,7 @@ describe('The ManageTab component', () => {
           linkRecord={linkRecord}
           profile={profile}
           orcidSiteURL={orcidSiteURL}
-          removeLink={noop}
+          removeLink={noOp}
           toggleShowInProfile={toggleShowInProfile}
         />
       </MemoryRouter>

diff --git a/src/features/orcidlink/HomeLinked/view.test.tsx b/src/features/orcidlink/HomeLinked/view.test.tsx
@@ -1,8 +1,8 @@
 import { render, screen, waitFor } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { MemoryRouter } from 'react-router-dom';
+import { noOp } from '../../common';
 import { LINK_RECORD_1, PROFILE_1, SERVICE_INFO_1 } from '../test/data';
-import { noop } from '../test/mocks';
 import HomeLinked from './view';
 
 describe('The HomeLinked Component', () => {
@@ -24,8 +24,8 @@ describe('The HomeLinked Component', () => {
         info={info}
         linkRecord={linkRecord}
         profile={profile}
-        removeLink={noop}
-        toggleShowInProfile={noop}
+        removeLink={noOp}
+        toggleShowInProfile={noOp}
       />
     );
 
@@ -48,8 +48,8 @@ describe('The HomeLinked Component', () => {
           info={info}
           linkRecord={linkRecord}
           profile={profile}
-          removeLink={noop}
-          toggleShowInProfile={noop}
+          removeLink={noOp}
+          toggleShowInProfile={noOp}
         />
       </MemoryRouter>
     );
@@ -100,7 +100,7 @@ describe('The HomeLinked Component', () => {
           info={info}
           linkRecord={linkRecord}
           profile={profile}
-          removeLink={noop}
+          removeLink={noOp}
           toggleShowInProfile={toggleShowInProfile}
         />
       </MemoryRouter>

diff --git a/src/features/orcidlink/test/mocks.ts b/src/features/orcidlink/test/mocks.ts
@@ -160,11 +160,3 @@ export function setupMockRegularUserWithError() {
     }
   );
 }
-
-/**
- * This is an empty function that serves as a placeholder for props that expect
- * one, but in tests in which these props are not tested, so they need not do anything.
- */
-export function noop() {
-  // do nothing
-}