From 575e2b80df23950304d9da2d90dc326c3c6c400c Mon Sep 17 00:00:00 2001
From: Manuel Luypaert <luypaert@ebi.ac.uk>
Date: Thu, 5 Sep 2024 10:12:52 +0100
Subject: [PATCH 1/4] Added unit testing for AlignmentEntryList component

---
 .../__tests__/AlignmentEntryList.test.tsx     | 42 +++++++++++++++++++
 1 file changed, 42 insertions(+)
 create mode 100644 webui/src/app/components/client/__tests__/AlignmentEntryList.test.tsx

diff --git a/webui/src/app/components/client/__tests__/AlignmentEntryList.test.tsx b/webui/src/app/components/client/__tests__/AlignmentEntryList.test.tsx
new file mode 100644
index 00000000..d8052c5d
--- /dev/null
+++ b/webui/src/app/components/client/__tests__/AlignmentEntryList.test.tsx
@@ -0,0 +1,42 @@
+import { describe, expect, it } from '@jest/globals';
+
+import { render, fireEvent } from '@testing-library/react'
+import { AlignmentEntryList } from '../AlignmentEntryList/AlignmentEntryList'
+
+jest.mock('https://raw.githubusercontent.com/alliance-genome/agr_ui/main/src/lib/utils.js',
+  () => {
+      return {
+          getSpecies: jest.fn(() => {}),
+          getSingleGenomeLocation: jest.fn(() => {})
+      }
+  },
+  {virtual: true}
+)
+
+describe('AlignmentEntryList', () => {
+  it('renders one input record by default', () => {
+    const result = render(
+      <AlignmentEntryList agrjBrowseDataRelease='7.3.0' dispatchInputPayloadPart={jest.fn()} />
+    )
+
+    const inputGroups = result.container.querySelectorAll('div.p-inputgroup')
+    expect(inputGroups).toHaveLength(1)  // Expect exactly one input group to be found
+  })
+
+  it('renders a functional add-record button', () => {
+    const result = render(
+      <AlignmentEntryList agrjBrowseDataRelease='7.3.0' dispatchInputPayloadPart={jest.fn()} />
+    )
+
+    const addRecordBtn = result.container.querySelector('button#add-record')
+    expect(addRecordBtn).not.toBe(null)  // Expect add-record button to be found
+    expect(addRecordBtn).toBeEnabled()
+
+    //Click the button
+    fireEvent.click(addRecordBtn!)
+
+    //Check one new entry-record was added (two total)
+    const inputGroups = result.container.querySelectorAll('div.p-inputgroup')
+    expect(inputGroups).toHaveLength(2)  // Expect exactly two input groups to be found
+  })
+})

From 6ddc30237b0e74c8a15f8d149699922342678743 Mon Sep 17 00:00:00 2001
From: Manuel Luypaert <luypaert@ebi.ac.uk>
Date: Thu, 5 Sep 2024 10:38:52 +0100
Subject: [PATCH 2/4] Added unit test for webUI record removal

---
 .../AlignmentEntryList/AlignmentEntryList.tsx |  2 +-
 .../__tests__/AlignmentEntryList.test.tsx     | 19 +++++++++++++++++++
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/webui/src/app/components/client/AlignmentEntryList/AlignmentEntryList.tsx b/webui/src/app/components/client/AlignmentEntryList/AlignmentEntryList.tsx
index 9e9202c4..ac801256 100644
--- a/webui/src/app/components/client/AlignmentEntryList/AlignmentEntryList.tsx
+++ b/webui/src/app/components/client/AlignmentEntryList/AlignmentEntryList.tsx
@@ -88,7 +88,7 @@ export const AlignmentEntryList: FunctionComponent<AlignmentEntryListProps> = (p
             <tbody>
                 {Array.from(alignmentEntries.values()).map((listEntry) => (
                     <tr key={listEntry.props.index}>
-                        <td><Button text icon="pi pi-trash" onClick={() => removeAlignmentEntry(listEntry.props.index)} /></td>
+                        <td><Button text id="remove-record" icon="pi pi-trash" onClick={() => removeAlignmentEntry(listEntry.props.index)} /></td>
                         <td>< AlignmentEntry {...listEntry.props} /></td>
                     </tr>))
                 }
diff --git a/webui/src/app/components/client/__tests__/AlignmentEntryList.test.tsx b/webui/src/app/components/client/__tests__/AlignmentEntryList.test.tsx
index d8052c5d..22870ffd 100644
--- a/webui/src/app/components/client/__tests__/AlignmentEntryList.test.tsx
+++ b/webui/src/app/components/client/__tests__/AlignmentEntryList.test.tsx
@@ -23,6 +23,25 @@ describe('AlignmentEntryList', () => {
     expect(inputGroups).toHaveLength(1)  // Expect exactly one input group to be found
   })
 
+  it('renders a functional button to remove individual records', () => {
+    const result = render(
+      <AlignmentEntryList agrjBrowseDataRelease='7.3.0' dispatchInputPayloadPart={jest.fn()} />
+    )
+
+    const inputGroupsBefore = result.container.querySelectorAll('div.p-inputgroup')
+    expect(inputGroupsBefore).toHaveLength(1)  // Expect exactly one input group to be found
+
+    const removeRecordBtn = result.container.querySelector('button#remove-record')
+    expect(removeRecordBtn).not.toBeNull()  // Expect remove-record button to be found
+    expect(removeRecordBtn).toBeEnabled()
+
+    fireEvent.click(removeRecordBtn!)
+
+    //Check one entry-record was removed (zero left)
+    const inputGroupsAfter = result.container.querySelectorAll('div.p-inputgroup')
+    expect(inputGroupsAfter).toHaveLength(0)  // Expect exactly one input group to be found
+  })
+
   it('renders a functional add-record button', () => {
     const result = render(
       <AlignmentEntryList agrjBrowseDataRelease='7.3.0' dispatchInputPayloadPart={jest.fn()} />

From ce91af8b3cc5607a9c7f20792abe1d0e9791dd74 Mon Sep 17 00:00:00 2001
From: Manuel Luypaert <luypaert@ebi.ac.uk>
Date: Thu, 5 Sep 2024 10:39:19 +0100
Subject: [PATCH 3/4] Minor cleanup

---
 .../app/components/client/__tests__/AlignmentEntryList.test.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/webui/src/app/components/client/__tests__/AlignmentEntryList.test.tsx b/webui/src/app/components/client/__tests__/AlignmentEntryList.test.tsx
index 22870ffd..73ee0e58 100644
--- a/webui/src/app/components/client/__tests__/AlignmentEntryList.test.tsx
+++ b/webui/src/app/components/client/__tests__/AlignmentEntryList.test.tsx
@@ -48,7 +48,7 @@ describe('AlignmentEntryList', () => {
     )
 
     const addRecordBtn = result.container.querySelector('button#add-record')
-    expect(addRecordBtn).not.toBe(null)  // Expect add-record button to be found
+    expect(addRecordBtn).not.toBeNull()  // Expect add-record button to be found
     expect(addRecordBtn).toBeEnabled()
 
     //Click the button

From b5fd7bf7af79c141e00e18fb81eece349bc69778 Mon Sep 17 00:00:00 2001
From: Manuel Luypaert <luypaert@ebi.ac.uk>
Date: Thu, 5 Sep 2024 10:39:46 +0100
Subject: [PATCH 4/4] Added JobSubmitForm unit testing

---
 .../client/__tests__/JobSubmitForm.test.tsx   | 35 +++++++++++++++++++
 1 file changed, 35 insertions(+)
 create mode 100644 webui/src/app/components/client/__tests__/JobSubmitForm.test.tsx

diff --git a/webui/src/app/components/client/__tests__/JobSubmitForm.test.tsx b/webui/src/app/components/client/__tests__/JobSubmitForm.test.tsx
new file mode 100644
index 00000000..58d67d10
--- /dev/null
+++ b/webui/src/app/components/client/__tests__/JobSubmitForm.test.tsx
@@ -0,0 +1,35 @@
+import { describe, expect, it } from '@jest/globals';
+
+import { render } from '@testing-library/react'
+import { JobSubmitForm } from '../JobSubmitForm/JobSubmitForm'
+
+jest.mock('https://raw.githubusercontent.com/alliance-genome/agr_ui/main/src/lib/utils.js',
+  () => {
+      return {
+          getSpecies: jest.fn(() => {}),
+          getSingleGenomeLocation: jest.fn(() => {})
+      }
+  },
+  {virtual: true}
+)
+
+describe('AlignmentEntry', () => {
+  it('renders a data-entry form', () => {
+    const result = render(
+      <JobSubmitForm agrjBrowseDataRelease='7.3.0' />
+    )
+
+    const inputGroup = result.container.querySelector('div.p-inputgroup')
+    expect(inputGroup).not.toBeNull()  // Expect (at least one) input group to be found
+  })
+
+  it('renders a submit button that is disabled by default', () => {
+    const result = render(
+      <JobSubmitForm agrjBrowseDataRelease='7.3.0' />
+    )
+ 
+    const submitBtn = result.container.querySelector('button[aria-label="Submit"]')
+    expect(submitBtn).not.toBeNull()  // Expect submit button to be found
+    expect(submitBtn).toBeDisabled()
+  })
+})