Merge pull request #67 from folio-org/UIREQMED-44-fix

UIREQMED-44: Persist filters when changing between actions

Author: Dmitriy-Litvinenko

src/components/NavigationMenu/NavigationMenu.js

@@ -57,7 +57,7 @@ export const handleChangeMenu = (event, location, history) => {
   if (pathname === getMediatedRequestsActivitiesUrl()) {
     destination.search = location.state;
   } else {
-    destination.state = location.search;
+    destination.state = location.search ? location.search : destination.state;
   }
 
   history.push(destination);

src/components/NavigationMenu/NavigationMenu.test.js

@@ -133,7 +133,7 @@ describe('NavigationMenu', () => {
       }));
     });
 
-    it('should handle correct when pathname not equal mediated requests activities', () => {
+    it('should handle correct when pathname not equal mediated requests activities with empty search', () => {
       const event = {
         target: {
           value: getConfirmItemArrivalUrl(),
@@ -155,5 +155,28 @@ describe('NavigationMenu', () => {
         pathname: getConfirmItemArrivalUrl(),
       }));
     });
+
+    it('should handle correct when pathname not equal mediated requests activities with not empty search', () => {
+      const event = {
+        target: {
+          value: getConfirmItemArrivalUrl(),
+        },
+      };
+      const location = {
+        pathname: getMediatedRequestsActivitiesUrl(),
+        state: '',
+        search: '?query=Test',
+      };
+      const history = {
+        location,
+        push,
+      };
+
+      handleChangeMenu(event, location, history);
+
+      expect(push).toHaveBeenCalledWith(expect.objectContaining({
+        pathname: getConfirmItemArrivalUrl(),
+      }));
+    });
   });
 });