BugFix : Fixed Improper Handling for Non-Numeric Values in Allotted hours section of Create Action item modal

src/screens/OrganizationActionItems/ItemModal.spec.tsx

@@ -778,6 +778,48 @@ describe('Testing ItemModal', () => {
     });
   });
 
+  it('should not allow letters or negative values in allotted hours', async () => {
+    renderItemModal(link1, itemProps[0]);
+
+    const allottedHours = screen.getByLabelText(t.allottedHours);
+    expect(allottedHours).toBeInTheDocument();
+
+    // Test letter input
+    fireEvent.change(allottedHours, { target: { value: 'abc' } });
+    await waitFor(() => {
+      expect(allottedHours).toHaveValue('');
+    });
+
+    // Test negative value
+    fireEvent.change(allottedHours, { target: { value: '-5' } });
+    await waitFor(() => {
+      expect(allottedHours).toHaveValue('');
+    });
+
+    // Test valid positive number
+    fireEvent.change(allottedHours, { target: { value: '5' } });
+    await waitFor(() => {
+      expect(allottedHours).toHaveValue('5');
+    });
+  });
+
+  it('validates allottedHours edge cases', async () => {
+    renderItemModal(link1, itemProps[0]);
+    const allottedHours = screen.getByLabelText(t.allottedHours);
+
+    // Test invalid string
+    fireEvent.change(allottedHours, { target: { value: 'invalid' } });
+    expect(allottedHours).toHaveValue('');
+
+    // Test NaN
+    fireEvent.change(allottedHours, { target: { value: NaN } });
+    expect(allottedHours).toHaveValue('');
+
+    // Test negative number
+    fireEvent.change(allottedHours, { target: { value: -5 } });
+    expect(allottedHours).toHaveValue('');
+  });
+
   it('should fail to Create Action Item', async () => {
     renderItemModal(link2, itemProps[0]);
     // Click Submit
@@ -802,6 +844,17 @@ describe('Testing ItemModal', () => {
     });
   });
 
+  it('handles null date change', async () => {
+    renderItemModal(link1, itemProps[0]);
+
+    const dateInput = screen.getByLabelText(t.dueDate);
+    fireEvent.change(dateInput, { target: { value: null } });
+
+    await waitFor(() => {
+      expect(dateInput).toHaveValue('');
+    });
+  });
+
   it('should fail to Update Action Item', async () => {
     renderItemModal(link2, itemProps[2]);
     expect(screen.getAllByText(t.updateActionItem)).toHaveLength(2);

src/screens/OrganizationActionItems/ItemModal.tsx

@@ -226,6 +226,20 @@
     field: keyof InterfaceFormStateType,
     value: string | number | boolean | Date | undefined | null,
   ): void => {
+    // Special handling for allottedHours
+    if (field === 'allottedHours') {
+      // If the value is not a valid number or is negative, set to null
+      const numValue = typeof value === 'string' ? Number(value) : value;
+      if (
+        typeof numValue !== 'number' ||
+        Number.isNaN(numValue) ||
+        numValue < 0
+      ) {
+        setFormState((prevState) => ({ ...prevState, [field]: null }));
+        return;
+      }
+    }
+
     setFormState((prevState) => ({ ...prevState, [field]: value }));
   };
 
@@ -574,8 +588,6 @@
                   className={styles.noOutline}
                   value={dayjs(dueDate)}
                   onChange={(date: Dayjs | null): void => {
-                    // Added istanbul ignore else, which will ignore else condition, we are not using else condition here
-                    /* istanbul ignore else -- @preserve */
                     if (date) handleFormChange('dueDate', date.toDate());
                   }}
                 />