UIQM-740 Don't show warn/fail error toasts when there are no warns/fa…

…ils. (#766)

* UIQM-740 Don't show warn/fail error toasts when there are no warns/fails.

* UIQM-740 update test case names

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 * [UIQM-716](https://issues.folio.org/browse/UIQM-716) *BREAKING* Consolidate routes based on MARC type for bib and authority records to avoid page refresh after redirecting from the create page to the edit one.
 * [UIQM-730](https://issues.folio.org/browse/UIQM-730) Create/Edit/Derive MARC record - Retain focus when MARC record validation rules error display. Show validation issues toasts.
+* [UIQM-740](https://issues.folio.org/browse/UIQM-740) Don't show warn/fail error toasts when there are no warns/fails.
 
 ## [9.0.1](https://github.com/folio-org/ui-quick-marc/tree/v9.0.1) (2024-11-22)
 

src/QuickMarcEditor/QuickMarcEditor.js

@@ -232,11 +232,16 @@ const QuickMarcEditor = ({
     const failCount = allIssuesArray.filter(issue => issue.severity === SEVERITY.ERROR).length;
     const warnCount = allIssuesArray.length - failCount;
 
+    if (!failCount && !warnCount) {
+      return;
+    }
+
     const values = {
       warnCount,
       failCount,
       breakingLine: <br />,
     };
+
     let messageId = null;
 
     if (failCount && warnCount) {

src/QuickMarcEditor/QuickMarcEditor.test.js

@@ -895,6 +895,30 @@ describe('Given QuickMarcEditor', () => {
     });
   });
 
+  describe('when saving form without validation warnings or errors', () => {
+    beforeEach(async () => {
+      mockValidate.mockClear().mockResolvedValue({});
+
+      const {
+        getByText,
+        getByTestId,
+      } = renderQuickMarcEditor();
+
+      const contentField = getByTestId('content-field-3');
+
+      fireEvent.change(contentField, { target: { value: '' } });
+      await fireEvent.click(getByText('stripes-acq-components.FormFooter.save'));
+    });
+
+    it('should not show a toast notification about validation warning and error', async () => {
+      await waitFor(() => {
+        expect(mockShowCallout).not.toHaveBeenCalledWith(expect.objectContaining({
+          messageId: expect.stringContaining('ui-quick-marc.record.save.error'),
+        }));
+      });
+    });
+  });
+
   describe('when marc record is of type HOLDINGS', () => {
     describe('when action is create', () => {
       it('should not show "Save & keep editing" button', () => {