From fd58aeaec843abc224eefbd47d7cd16425205708 Mon Sep 17 00:00:00 2001
From: Cheng Shi <cks5520psu@gmail.com>
Date: Tue, 25 Feb 2025 10:47:59 -0500
Subject: [PATCH] fix: some adjustments, remove contactFactory & isCollection

---
 .../src/lib/components/alert/Alert.tsx        |  3 +--
 src/sections/collection/Collection.tsx        |  9 ++++----
 src/sections/collection/CollectionFactory.tsx |  3 +++
 src/sections/dataset/Dataset.tsx              |  4 ++++
 src/sections/dataset/DatasetFactory.tsx       |  5 +++-
 .../DatasetActionButtons.tsx                  |  9 ++++----
 src/sections/shared/contact/ContactButton.tsx | 15 ++++++------
 .../shared/contact/ContactFactory.tsx         |  8 -------
 src/stories/collection/Collection.stories.tsx |  9 ++++++++
 src/stories/dataset/Dataset.stories.tsx       | 12 ++++++++++
 .../DatasetActionButtons.stories.tsx          |  4 ++++
 .../shared/contact/ContactMockRepository.ts   | 13 +++++++++++
 .../contact-modal/ContactModal.stories.tsx    |  2 +-
 .../sections/collection/Collection.spec.tsx   | 23 ++++++++++++++++---
 .../sections/contact/ContactButton.spec.tsx   | 12 +++++-----
 ...c.tsx => useSendFeedbacktoOwners.spec.tsx} |  4 ++--
 .../sections/dataset/Dataset.spec.tsx         | 21 +++++++++++++++--
 17 files changed, 116 insertions(+), 40 deletions(-)
 delete mode 100644 src/sections/shared/contact/ContactFactory.tsx
 create mode 100644 src/stories/shared/contact/ContactMockRepository.ts
 rename src/stories/shared/{ => contact}/contact-modal/ContactModal.stories.tsx (94%)
 rename tests/component/sections/contact/{useSubmitContact.spec.tsx => useSendFeedbacktoOwners.spec.tsx} (96%)

diff --git a/packages/design-system/src/lib/components/alert/Alert.tsx b/packages/design-system/src/lib/components/alert/Alert.tsx
index 5e8e12382..8fc1d2049 100644
--- a/packages/design-system/src/lib/components/alert/Alert.tsx
+++ b/packages/design-system/src/lib/components/alert/Alert.tsx
@@ -9,10 +9,9 @@ interface AlertProps {
   dismissible?: boolean
   customHeading?: string
   children?: ReactNode
-  onClose?: () => void
 }
 
-function Alert({ variant, dismissible = true, customHeading, children, onClose }: AlertProps) {
+function Alert({ variant, dismissible = true, customHeading, children }: AlertProps) {
   interface AlertHeadings {
     [key: string]: string
   }
diff --git a/src/sections/collection/Collection.tsx b/src/sections/collection/Collection.tsx
index 1069b8384..66267937e 100644
--- a/src/sections/collection/Collection.tsx
+++ b/src/sections/collection/Collection.tsx
@@ -18,7 +18,7 @@ import { ContactButton } from '@/sections/shared/contact/ContactButton'
 import { EditCollectionDropdown } from './edit-collection-dropdown/EditCollectionDropdown'
 import { FeaturedItems } from './featured-items/FeaturedItems'
 import styles from './Collection.module.scss'
-import { ContactRepositoryFactory } from '@/sections/shared/contact/ContactFactory'
+import { ContactRepository } from '@/contact/domain/repositories/ContactRepository'
 
 interface CollectionProps {
   collectionRepository: CollectionRepository
@@ -28,6 +28,7 @@ interface CollectionProps {
   edited?: boolean
   collectionQueryParams: UseCollectionQueryParamsReturnType
   infiniteScrollEnabled?: boolean
+  contactRepository: ContactRepository
 }
 
 export function Collection({
@@ -36,11 +37,11 @@ export function Collection({
   created,
   published,
   edited,
-  collectionQueryParams
+  collectionQueryParams,
+  contactRepository
 }: CollectionProps) {
   useScrollTop()
   const { t } = useTranslation('collection')
-  const contactRepository = ContactRepositoryFactory.create()
   const { collection, isLoading: isLoadingCollection } = useCollection(
     collectionRepository,
     collectionIdFromParams,
@@ -98,7 +99,7 @@ export function Collection({
               <div className={styles['right-content']}>
                 <ContactButton
                   toContactName={collection.name}
-                  isCollection={true}
+                  contactObjectType={'collection'}
                   id={collection.id}
                   contactRepository={contactRepository}
                 />
diff --git a/src/sections/collection/CollectionFactory.tsx b/src/sections/collection/CollectionFactory.tsx
index eaf28195b..2403308e4 100644
--- a/src/sections/collection/CollectionFactory.tsx
+++ b/src/sections/collection/CollectionFactory.tsx
@@ -1,11 +1,13 @@
 import { ReactElement } from 'react'
 import { useLocation, useParams } from 'react-router-dom'
 import { CollectionJSDataverseRepository } from '../../collection/infrastructure/repositories/CollectionJSDataverseRepository'
+import { ContactJSDataverseRepository } from '@/contact/infrastructure/ContactJSDataverseRepository'
 import { Collection } from './Collection'
 import { INFINITE_SCROLL_ENABLED } from './config'
 import { useGetCollectionQueryParams } from './useGetCollectionQueryParams'
 
 const collectionRepository = new CollectionJSDataverseRepository()
+const contactRepository = new ContactJSDataverseRepository()
 export class CollectionFactory {
   static create(): ReactElement {
     return <CollectionWithSearchParams />
@@ -32,6 +34,7 @@ function CollectionWithSearchParams() {
       published={published}
       edited={edited}
       infiniteScrollEnabled={INFINITE_SCROLL_ENABLED}
+      contactRepository={contactRepository}
     />
   )
 }
diff --git a/src/sections/dataset/Dataset.tsx b/src/sections/dataset/Dataset.tsx
index eee832876..7f33edfe8 100644
--- a/src/sections/dataset/Dataset.tsx
+++ b/src/sections/dataset/Dataset.tsx
@@ -27,12 +27,14 @@ import { QueryParamKey, Route } from '../Route.enum'
 import { MetadataBlockInfoRepository } from '../../metadata-block-info/domain/repositories/MetadataBlockInfoRepository'
 import { CollectionRepository } from '../../collection/domain/repositories/CollectionRepository'
 import { DatasetTerms } from '@/sections/dataset/dataset-terms/DatasetTerms'
+import { ContactRepository } from '@/contact/domain/repositories/ContactRepository'
 
 interface DatasetProps {
   datasetRepository: DatasetRepository
   fileRepository: FileRepository
   metadataBlockInfoRepository: MetadataBlockInfoRepository
   collectionRepository: CollectionRepository
+  contactRepository: ContactRepository
   created?: boolean
   metadataUpdated?: boolean
   filesTabInfiniteScrollEnabled?: boolean
@@ -45,6 +47,7 @@ export function Dataset({
   fileRepository,
   metadataBlockInfoRepository,
   collectionRepository,
+  contactRepository,
   created,
   metadataUpdated,
   filesTabInfiniteScrollEnabled,
@@ -129,6 +132,7 @@ export function Dataset({
                     datasetRepository={datasetRepository}
                     collectionRepository={collectionRepository}
                     dataset={dataset}
+                    contactRepository={contactRepository}
                   />
                 </Col>
               </Row>
diff --git a/src/sections/dataset/DatasetFactory.tsx b/src/sections/dataset/DatasetFactory.tsx
index 4caa3e5e0..f418c79bb 100644
--- a/src/sections/dataset/DatasetFactory.tsx
+++ b/src/sections/dataset/DatasetFactory.tsx
@@ -16,12 +16,13 @@ import { AlertProvider } from '../alerts/AlertProvider'
 import { searchParamVersionToDomainVersion } from '../../router'
 import { FILES_TAB_INFINITE_SCROLL_ENABLED } from './config'
 import { CollectionJSDataverseRepository } from '@/collection/infrastructure/repositories/CollectionJSDataverseRepository'
+import { ContactJSDataverseRepository } from '@/contact/infrastructure/ContactJSDataverseRepository'
 
 const collectionRepository = new CollectionJSDataverseRepository()
 const datasetRepository = new DatasetJSDataverseRepository()
 const fileRepository = new FileJSDataverseRepository()
 const metadataBlockInfoRepository = new MetadataBlockInfoJSDataverseRepository()
-
+const contactRepository = new ContactJSDataverseRepository()
 const settingRepository = new SettingJSDataverseRepository()
 
 export class DatasetFactory {
@@ -73,6 +74,7 @@ function DatasetWithSearchParams() {
           datasetRepository={datasetRepository}
           fileRepository={fileRepository}
           metadataBlockInfoRepository={metadataBlockInfoRepository}
+          contactRepository={contactRepository}
           filesTabInfiniteScrollEnabled={FILES_TAB_INFINITE_SCROLL_ENABLED}
           tab={tab}
         />
@@ -90,6 +92,7 @@ function DatasetWithSearchParams() {
         datasetRepository={datasetRepository}
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
+        contactRepository={contactRepository}
         created={created}
         publishInProgress={publishInProgress}
         metadataUpdated={metadataUpdated}
diff --git a/src/sections/dataset/dataset-action-buttons/DatasetActionButtons.tsx b/src/sections/dataset/dataset-action-buttons/DatasetActionButtons.tsx
index a415de81c..487b8a639 100644
--- a/src/sections/dataset/dataset-action-buttons/DatasetActionButtons.tsx
+++ b/src/sections/dataset/dataset-action-buttons/DatasetActionButtons.tsx
@@ -11,21 +11,22 @@ import { LinkDatasetButton } from './link-dataset-button/LinkDatasetButton'
 import { ShareDatasetButton } from './share-dataset-button/ShareDatasetButton'
 import styles from './DatasetActionButtons.module.scss'
 import { ContactButton } from '@/sections/shared/contact/ContactButton'
-import { ContactRepositoryFactory } from '@/sections/shared/contact/ContactFactory'
+import { ContactRepository } from '@/contact/domain/repositories/ContactRepository'
 
 interface DatasetActionButtonsProps {
   dataset: Dataset
   datasetRepository: DatasetRepository
   collectionRepository: CollectionRepository
+  contactRepository: ContactRepository
 }
 
 export function DatasetActionButtons({
   dataset,
   datasetRepository,
-  collectionRepository
+  collectionRepository,
+  contactRepository
 }: DatasetActionButtonsProps) {
   const { t } = useTranslation('dataset')
-  const contactRepository = ContactRepositoryFactory.create()
 
   return (
     <ButtonGroup aria-label={t('datasetActionButtons.title')} vertical className={styles.group}>
@@ -46,8 +47,8 @@ export function DatasetActionButtons({
       <LinkDatasetButton dataset={dataset} />
       <ButtonGroup className={styles['contact-owner-and-share-group']}>
         <ContactButton
-          isCollection={false}
           toContactName={dataset.metadataBlocks[0].fields.title}
+          contactObjectType="dataset"
           id={dataset.persistentId}
           contactRepository={contactRepository}
         />
diff --git a/src/sections/shared/contact/ContactButton.tsx b/src/sections/shared/contact/ContactButton.tsx
index 364334a91..396825131 100644
--- a/src/sections/shared/contact/ContactButton.tsx
+++ b/src/sections/shared/contact/ContactButton.tsx
@@ -7,14 +7,16 @@ import { ContactRepository } from '@/contact/domain/repositories/ContactReposito
 
 interface ContactButtonProps {
   toContactName: string
-  isCollection: boolean
+  contactObjectType: ContactObjectType
   id: string | number
   contactRepository: ContactRepository
 }
 
+type ContactObjectType = 'collection' | 'dataset'
+
 export const ContactButton = ({
   toContactName,
-  isCollection,
+  contactObjectType,
   id,
   contactRepository
 }: ContactButtonProps) => {
@@ -25,11 +27,9 @@ export const ContactButton = ({
 
   return (
     <>
-      {isCollection ? (
+      {contactObjectType == 'collection' && (
         <>
-          <Tooltip
-            overlay={isCollection ? t('contact.contactCollection') : t('contact.contactDataset')}
-            placement="top">
+          <Tooltip overlay={t('contact.contactCollection')} placement="top">
             <Button
               variant="link"
               onClick={openContactModal}
@@ -47,7 +47,8 @@ export const ContactButton = ({
             contactRepository={contactRepository}
           />
         </>
-      ) : (
+      )}
+      {contactObjectType == 'dataset' && (
         <>
           <Button variant="secondary" onClick={openContactModal} size="sm">
             {t('contact.title.dataset')}
diff --git a/src/sections/shared/contact/ContactFactory.tsx b/src/sections/shared/contact/ContactFactory.tsx
deleted file mode 100644
index e1bc74da9..000000000
--- a/src/sections/shared/contact/ContactFactory.tsx
+++ /dev/null
@@ -1,8 +0,0 @@
-import { ContactRepository } from '@/contact/domain/repositories/ContactRepository'
-import { ContactJSDataverseRepository } from '@/contact/infrastructure/ContactJSDataverseRepository'
-
-export class ContactRepositoryFactory {
-  static create(): ContactRepository {
-    return new ContactJSDataverseRepository()
-  }
-}
diff --git a/src/stories/collection/Collection.stories.tsx b/src/stories/collection/Collection.stories.tsx
index aa102f8ba..99a78211b 100644
--- a/src/stories/collection/Collection.stories.tsx
+++ b/src/stories/collection/Collection.stories.tsx
@@ -8,6 +8,7 @@ import { CollectionLoadingMockRepository } from './CollectionLoadingMockReposito
 import { UnpublishedCollectionMockRepository } from '@/stories/collection/UnpublishedCollectionMockRepository'
 import { CollectionFeaturedItemMother } from '@tests/component/collection/domain/models/CollectionFeaturedItemMother'
 import { FakerHelper } from '@tests/component/shared/FakerHelper'
+import { ContactMockRepository } from '../shared/contact/ContactMockRepository'
 
 const meta: Meta<typeof Collection> = {
   title: 'Pages/Collection',
@@ -26,6 +27,7 @@ export const Default: Story = {
   render: () => (
     <Collection
       collectionRepository={new CollectionMockRepository()}
+      contactRepository={new ContactMockRepository()}
       collectionIdFromParams="collection"
       created={false}
       published={false}
@@ -43,6 +45,7 @@ export const Loading: Story = {
     <Collection
       collectionIdFromParams="collection"
       collectionRepository={new CollectionLoadingMockRepository()}
+      contactRepository={new ContactMockRepository()}
       created={false}
       published={false}
       collectionQueryParams={{ pageQuery: 1, searchQuery: undefined, typesQuery: undefined }}
@@ -56,6 +59,7 @@ export const LoggedIn: Story = {
     <Collection
       collectionIdFromParams="collection"
       collectionRepository={new CollectionMockRepository()}
+      contactRepository={new ContactMockRepository()}
       created={false}
       published={false}
       collectionQueryParams={{ pageQuery: 1, searchQuery: undefined, typesQuery: undefined }}
@@ -68,6 +72,7 @@ export const Unpublished: Story = {
     <Collection
       collectionIdFromParams="collection"
       collectionRepository={new UnpublishedCollectionMockRepository()}
+      contactRepository={new ContactMockRepository()}
       created={false}
       published={false}
       collectionQueryParams={{ pageQuery: 1, searchQuery: undefined, typesQuery: undefined }}
@@ -80,6 +85,7 @@ export const Created: Story = {
   render: () => (
     <Collection
       collectionRepository={new CollectionMockRepository()}
+      contactRepository={new ContactMockRepository()}
       collectionIdFromParams="collection"
       created={true}
       published={false}
@@ -92,6 +98,7 @@ export const Published: Story = {
   render: () => (
     <Collection
       collectionRepository={new CollectionMockRepository()}
+      contactRepository={new ContactMockRepository()}
       collectionIdFromParams="collection"
       created={false}
       published={true}
@@ -105,6 +112,7 @@ export const Edited: Story = {
   render: () => (
     <Collection
       collectionRepository={new CollectionMockRepository()}
+      contactRepository={new ContactMockRepository()}
       collectionIdFromParams="collection"
       created={false}
       published={false}
@@ -128,6 +136,7 @@ export const WithFeaturedItems: Story = {
   render: () => (
     <Collection
       collectionRepository={collectionRepositoryWithFeaturedItems}
+      contactRepository={new ContactMockRepository()}
       collectionIdFromParams="collection"
       created={false}
       published={false}
diff --git a/src/stories/dataset/Dataset.stories.tsx b/src/stories/dataset/Dataset.stories.tsx
index 503953428..6aec80e63 100644
--- a/src/stories/dataset/Dataset.stories.tsx
+++ b/src/stories/dataset/Dataset.stories.tsx
@@ -18,6 +18,7 @@ import { WithNotImplementedModal } from '../WithNotImplementedModal'
 import { MetadataBlockInfoMockRepository } from '../shared-mock-repositories/metadata-block-info/MetadataBlockInfoMockRepository'
 import { DatasetMockRepository } from './DatasetMockRepository'
 import { CollectionMockRepository } from '@/stories/collection/CollectionMockRepository'
+import { ContactMockRepository } from '../shared/contact/ContactMockRepository'
 
 const meta: Meta<typeof Dataset> = {
   title: 'Pages/Dataset',
@@ -40,6 +41,7 @@ export const Default: Story = {
       datasetRepository={new DatasetMockRepository()}
       fileRepository={new FileMockRepository()}
       metadataBlockInfoRepository={new MetadataBlockInfoMockRepository()}
+      contactRepository={new ContactMockRepository()}
       filesTabInfiniteScrollEnabled
     />
   )
@@ -52,6 +54,7 @@ export const WithNormalPagination: Story = {
       datasetRepository={new DatasetMockRepository()}
       fileRepository={new FileMockRepository()}
       metadataBlockInfoRepository={new MetadataBlockInfoMockRepository()}
+      contactRepository={new ContactMockRepository()}
     />
   )
 }
@@ -64,6 +67,7 @@ export const Created: Story = {
       datasetRepository={new DatasetMockRepository()}
       fileRepository={new FileMockRepository()}
       metadataBlockInfoRepository={new MetadataBlockInfoMockRepository()}
+      contactRepository={new ContactMockRepository()}
       created={true}
       filesTabInfiniteScrollEnabled
     />
@@ -78,6 +82,7 @@ export const MetadataUpdated: Story = {
       datasetRepository={new DatasetMockRepository()}
       fileRepository={new FileMockRepository()}
       metadataBlockInfoRepository={new MetadataBlockInfoMockRepository()}
+      contactRepository={new ContactMockRepository()}
       metadataUpdated={true}
     />
   )
@@ -90,6 +95,7 @@ export const DraftWithAllDatasetPermissions: Story = {
       datasetRepository={new DatasetMockRepository()}
       fileRepository={new FileMockRepository()}
       metadataBlockInfoRepository={new MetadataBlockInfoMockRepository()}
+      contactRepository={new ContactMockRepository()}
       filesTabInfiniteScrollEnabled
     />
   )
@@ -102,6 +108,7 @@ export const Deaccessioned: Story = {
       datasetRepository={new DatasetMockRepository()}
       fileRepository={new FileMockRepository()}
       metadataBlockInfoRepository={new MetadataBlockInfoMockRepository()}
+      contactRepository={new ContactMockRepository()}
       filesTabInfiniteScrollEnabled
     />
   )
@@ -114,6 +121,7 @@ export const LoggedInAsOwner: Story = {
       datasetRepository={new DatasetMockRepository()}
       fileRepository={new FileMockRepository()}
       metadataBlockInfoRepository={new MetadataBlockInfoMockRepository()}
+      contactRepository={new ContactMockRepository()}
       filesTabInfiniteScrollEnabled
     />
   )
@@ -127,6 +135,7 @@ export const Loading: Story = {
       datasetRepository={new DatasetMockRepository()}
       fileRepository={new FileMockRepository()}
       metadataBlockInfoRepository={new MetadataBlockInfoMockRepository()}
+      contactRepository={new ContactMockRepository()}
       filesTabInfiniteScrollEnabled
     />
   )
@@ -140,6 +149,7 @@ export const DatasetNotFound: Story = {
       datasetRepository={new DatasetMockRepository()}
       fileRepository={new FileMockRepository()}
       metadataBlockInfoRepository={new MetadataBlockInfoMockRepository()}
+      contactRepository={new ContactMockRepository()}
       filesTabInfiniteScrollEnabled
     />
   )
@@ -153,6 +163,7 @@ export const DatasetAnonymizedView: Story = {
       datasetRepository={new DatasetMockRepository()}
       fileRepository={new FileMockRepository()}
       metadataBlockInfoRepository={new MetadataBlockInfoMockRepository()}
+      contactRepository={new ContactMockRepository()}
       filesTabInfiniteScrollEnabled
     />
   )
@@ -166,6 +177,7 @@ export const DatasetWithNoFiles: Story = {
       datasetRepository={new DatasetMockRepository()}
       fileRepository={new FileMockNoDataRepository()}
       metadataBlockInfoRepository={new MetadataBlockInfoMockRepository()}
+      contactRepository={new ContactMockRepository()}
       filesTabInfiniteScrollEnabled
     />
   )
diff --git a/src/stories/dataset/dataset-action-buttons/DatasetActionButtons.stories.tsx b/src/stories/dataset/dataset-action-buttons/DatasetActionButtons.stories.tsx
index ab14eaf48..8d20e4a55 100644
--- a/src/stories/dataset/dataset-action-buttons/DatasetActionButtons.stories.tsx
+++ b/src/stories/dataset/dataset-action-buttons/DatasetActionButtons.stories.tsx
@@ -11,6 +11,7 @@ import {
 import { WithLoggedInUser } from '../../WithLoggedInUser'
 import { DatasetMockRepository } from '../DatasetMockRepository'
 import { CollectionMockRepository } from '@/stories/collection/CollectionMockRepository'
+import { ContactMockRepository } from '@/stories/shared/contact/ContactMockRepository'
 
 const meta: Meta<typeof DatasetActionButtons> = {
   title: 'Sections/Dataset Page/DatasetActionButtons',
@@ -41,6 +42,7 @@ export const WithPublishPermissions: Story = {
       })}
       datasetRepository={new DatasetMockRepository()}
       collectionRepository={new CollectionMockRepository()}
+      contactRepository={new ContactMockRepository()}
     />
   )
 }
@@ -60,6 +62,7 @@ export const WithNoDatasetPermissions: Story = {
       })}
       datasetRepository={new DatasetMockRepository()}
       collectionRepository={new CollectionMockRepository()}
+      contactRepository={new ContactMockRepository()}
     />
   )
 }
@@ -83,6 +86,7 @@ export const WithUpdateAndNoPublishDatasetPermissions: Story = {
       })}
       datasetRepository={new DatasetMockRepository()}
       collectionRepository={new CollectionMockRepository()}
+      contactRepository={new ContactMockRepository()}
     />
   )
 }
diff --git a/src/stories/shared/contact/ContactMockRepository.ts b/src/stories/shared/contact/ContactMockRepository.ts
new file mode 100644
index 000000000..f8dcf8ad3
--- /dev/null
+++ b/src/stories/shared/contact/ContactMockRepository.ts
@@ -0,0 +1,13 @@
+import { Contact } from '@/contact/domain/models/Contact'
+import { ContactRepository } from '@/contact/domain/repositories/ContactRepository'
+import { FeedbackDTO } from '@/contact/domain/useCases/FeedbackDTO'
+
+export class ContactMockRepository implements ContactRepository {
+  sendFeedbacktoOwners(_feedbackDTO: FeedbackDTO): Promise<Contact[]> {
+    return new Promise<Contact[]>((resolve) => {
+      setTimeout(() => {
+        resolve([])
+      }, 1_000)
+    })
+  }
+}
diff --git a/src/stories/shared/contact-modal/ContactModal.stories.tsx b/src/stories/shared/contact/contact-modal/ContactModal.stories.tsx
similarity index 94%
rename from src/stories/shared/contact-modal/ContactModal.stories.tsx
rename to src/stories/shared/contact/contact-modal/ContactModal.stories.tsx
index f08e6f03d..e323854ae 100644
--- a/src/stories/shared/contact-modal/ContactModal.stories.tsx
+++ b/src/stories/shared/contact/contact-modal/ContactModal.stories.tsx
@@ -1,5 +1,5 @@
 import { Meta, StoryObj } from '@storybook/react'
-import { WithI18next } from '../../WithI18next'
+import { WithI18next } from '../../../WithI18next'
 import { ContactModal } from '@/sections/shared/contact/contact-modal/contact-modal'
 import { ContactRepository } from '@/contact/domain/repositories/ContactRepository'
 
diff --git a/tests/component/sections/collection/Collection.spec.tsx b/tests/component/sections/collection/Collection.spec.tsx
index 156f804ca..0a44c3afd 100644
--- a/tests/component/sections/collection/Collection.spec.tsx
+++ b/tests/component/sections/collection/Collection.spec.tsx
@@ -4,9 +4,10 @@ import { CollectionMother } from '@tests/component/collection/domain/models/Coll
 import { CollectionItemsMother } from '@tests/component/collection/domain/models/CollectionItemsMother'
 import { CollectionItemSubset } from '@/collection/domain/models/CollectionItemSubset'
 import { CollectionFeaturedItemMother } from '@tests/component/collection/domain/models/CollectionFeaturedItemMother'
-import { ContactJSDataverseRepository } from '@/contact/infrastructure/ContactJSDataverseRepository'
+import { ContactRepository } from '@/contact/domain/repositories/ContactRepository'
 
 const collectionRepository = {} as CollectionRepository
+const contactRepository = {} as ContactRepository
 const collection = CollectionMother.create({ name: 'Collection Name' })
 const userPermissionsMock = CollectionMother.createUserPermissions()
 
@@ -37,6 +38,7 @@ describe('Collection page', () => {
     cy.customMount(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
@@ -62,6 +64,7 @@ describe('Collection page', () => {
     cy.customMount(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
@@ -76,6 +79,7 @@ describe('Collection page', () => {
     cy.customMount(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
@@ -90,6 +94,7 @@ describe('Collection page', () => {
     cy.customMount(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
@@ -103,6 +108,7 @@ describe('Collection page', () => {
     cy.customMount(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
@@ -116,6 +122,7 @@ describe('Collection page', () => {
     cy.mountAuthenticated(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
@@ -134,6 +141,7 @@ describe('Collection page', () => {
     cy.customMount(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created
         published={false}
@@ -147,6 +155,7 @@ describe('Collection page', () => {
     cy.customMount(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={true}
@@ -168,6 +177,7 @@ describe('Collection page', () => {
     cy.mountAuthenticated(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
@@ -187,6 +197,7 @@ describe('Collection page', () => {
     cy.mountAuthenticated(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
@@ -208,6 +219,7 @@ describe('Collection page', () => {
     cy.mountAuthenticated(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
@@ -232,6 +244,7 @@ describe('Collection page', () => {
     cy.customMount(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
@@ -248,6 +261,7 @@ describe('Collection page', () => {
     cy.customMount(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
@@ -264,6 +278,7 @@ describe('Collection page', () => {
     cy.customMount(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
@@ -285,13 +300,14 @@ describe('Collection page', () => {
     cy.customMount(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
         collectionQueryParams={{ pageQuery: 1 }}
       />
     )
-    cy.stub(ContactJSDataverseRepository.prototype, 'sendFeedbacktoOwners').resolves([])
+    cy.stub(contactRepository, 'sendFeedbacktoOwners').resolves([])
 
     cy.findByRole('button', { name: /Contact/i })
       .should('exist')
@@ -304,13 +320,14 @@ describe('Collection page', () => {
     cy.customMount(
       <Collection
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         collectionIdFromParams="collection"
         created={false}
         published={false}
         collectionQueryParams={{ pageQuery: 1 }}
       />
     )
-    cy.stub(ContactJSDataverseRepository.prototype, 'sendFeedbacktoOwners').resolves([])
+    cy.stub(contactRepository, 'sendFeedbacktoOwners').resolves([])
 
     cy.findByRole('button', { name: /Contact/i })
       .should('exist')
diff --git a/tests/component/sections/contact/ContactButton.spec.tsx b/tests/component/sections/contact/ContactButton.spec.tsx
index a1dc08863..d80a330ec 100644
--- a/tests/component/sections/contact/ContactButton.spec.tsx
+++ b/tests/component/sections/contact/ContactButton.spec.tsx
@@ -14,7 +14,7 @@ describe('ContactButton', () => {
     cy.customMount(
       <ContactButton
         toContactName="Test Dataset"
-        isCollection={true}
+        contactObjectType="collection"
         id="root"
         contactRepository={contactRepository}
       />
@@ -24,11 +24,11 @@ describe('ContactButton', () => {
       .click()
   })
 
-  it('shows correct contact title based on isCollection prop', () => {
+  it('shows correct contact title based on different contact object', () => {
     cy.customMount(
       <ContactButton
         toContactName="Test Dataset"
-        isCollection={true}
+        contactObjectType="collection"
         id="root"
         contactRepository={contactRepository}
       />
@@ -40,7 +40,7 @@ describe('ContactButton', () => {
     cy.customMount(
       <ContactButton
         toContactName="Test Dataset"
-        isCollection={false}
+        contactObjectType="dataset"
         id="root"
         contactRepository={contactRepository}
       />
@@ -54,7 +54,7 @@ describe('ContactButton', () => {
     cy.customMount(
       <ContactButton
         toContactName="Test Dataset"
-        isCollection={false}
+        contactObjectType="dataset"
         id="1"
         contactRepository={contactRepository}
       />
@@ -92,7 +92,7 @@ describe('ContactButton Error', () => {
     cy.customMount(
       <ContactButton
         toContactName="Test Dataset"
-        isCollection={true}
+        contactObjectType="collection"
         id="root"
         contactRepository={contactRepository}
       />
diff --git a/tests/component/sections/contact/useSubmitContact.spec.tsx b/tests/component/sections/contact/useSendFeedbacktoOwners.spec.tsx
similarity index 96%
rename from tests/component/sections/contact/useSubmitContact.spec.tsx
rename to tests/component/sections/contact/useSendFeedbacktoOwners.spec.tsx
index 468d9d3e8..0ef303e44 100644
--- a/tests/component/sections/contact/useSubmitContact.spec.tsx
+++ b/tests/component/sections/contact/useSendFeedbacktoOwners.spec.tsx
@@ -4,11 +4,11 @@ import {
   SubmissionStatus
 } from '@/sections/shared/form/ContactForm/useSendFeedbacktoOwners'
 import { ContactRepository } from '@/contact/domain/repositories/ContactRepository'
-import { feedbackDTO } from '@/contact/domain/useCases/feedbackDTO'
+import { FeedbackDTO } from '@/contact/domain/useCases/FeedbackDTO'
 
 const contactRepository: ContactRepository = {} as ContactRepository
 
-const mockFormData: feedbackDTO = {
+const mockFormData: FeedbackDTO = {
   subject: 'Test',
   body: 'Hello',
   fromEmail: 'test@dataverse.com'
diff --git a/tests/component/sections/dataset/Dataset.spec.tsx b/tests/component/sections/dataset/Dataset.spec.tsx
index 366641187..304bde623 100644
--- a/tests/component/sections/dataset/Dataset.spec.tsx
+++ b/tests/component/sections/dataset/Dataset.spec.tsx
@@ -17,13 +17,14 @@ import { AlertProvider } from '../../../../src/sections/alerts/AlertProvider'
 import { MetadataBlockInfoRepository } from '../../../../src/metadata-block-info/domain/repositories/MetadataBlockInfoRepository'
 import { MetadataBlockInfoMother } from '../../metadata-block-info/domain/models/MetadataBlockInfoMother'
 import { CollectionRepository } from '@/collection/domain/repositories/CollectionRepository'
-import { ContactJSDataverseRepository } from '@/contact/infrastructure/ContactJSDataverseRepository'
+import { ContactRepository } from '@/contact/domain/repositories/ContactRepository'
 
 const setAnonymizedView = () => {}
 const fileRepository: FileRepository = {} as FileRepository
 const datasetRepository: DatasetRepository = {} as DatasetRepository
 const metadataBlockInfoRepository: MetadataBlockInfoRepository = {} as MetadataBlockInfoRepository
 const collectionRepository: CollectionRepository = {} as CollectionRepository
+const contactRepository = {} as ContactRepository
 
 const TOTAL_FILES_COUNT = 200
 const allFiles = FilePreviewMother.createMany(TOTAL_FILES_COUNT)
@@ -97,6 +98,7 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
       />,
       testDataset
     )
@@ -114,6 +116,7 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
       />,
       emptyDataset
     )
@@ -130,6 +133,7 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
       />,
       dataset
     )
@@ -147,6 +151,7 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
       />,
       dataset
     )
@@ -163,6 +168,7 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
       />,
       testDataset
     )
@@ -180,6 +186,7 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
       />,
       testDataset
     )
@@ -200,6 +207,7 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
       />,
       testDataset
     )
@@ -222,6 +230,7 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
       />,
       testDataset
     )
@@ -244,6 +253,7 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         tab={'metadata'}
       />,
       testDataset
@@ -267,6 +277,7 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
       />,
       testDatasetAnonymized
     )
@@ -285,6 +296,7 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
       />,
       testDataset
     )
@@ -302,6 +314,7 @@ describe('Dataset', () => {
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         filesTabInfiniteScrollEnabled={true}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
       />,
       testDataset
     )
@@ -321,6 +334,7 @@ describe('Dataset', () => {
           metadataBlockInfoRepository={metadataBlockInfoRepository}
           collectionRepository={collectionRepository}
           created={true}
+          contactRepository={contactRepository}
         />
       </AlertProvider>,
       testDataset
@@ -341,6 +355,7 @@ describe('Dataset', () => {
           metadataBlockInfoRepository={metadataBlockInfoRepository}
           collectionRepository={collectionRepository}
           metadataUpdated={true}
+          contactRepository={contactRepository}
         />
       </AlertProvider>,
       testDataset
@@ -359,11 +374,12 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         metadataUpdated={true}
       />,
       testDataset
     )
-    cy.stub(ContactJSDataverseRepository.prototype, 'sendFeedbacktoOwners').resolves([])
+    cy.stub(contactRepository, 'sendFeedbacktoOwners').resolves([])
 
     cy.findByRole('button', { name: /Contact Owner/i })
       .should('exist')
@@ -395,6 +411,7 @@ describe('Dataset', () => {
         fileRepository={fileRepository}
         metadataBlockInfoRepository={metadataBlockInfoRepository}
         collectionRepository={collectionRepository}
+        contactRepository={contactRepository}
         metadataUpdated={true}
       />,
       testDataset