fix(connectGeoSearch): correctly dispose the connector (#2845)

* fix(connectGeoSearch): reset insideBoundingBox on dispose
* fix(connectGeoSearch): correctly reset the state on dispose

Author: samouss

src/connectors/geo-search/__tests__/connectGeoSearch-test.js

@@ -1268,6 +1268,63 @@ describe('connectGeoSearch - getConfiguration', () => {
 });
 
 describe('connectGeoSearch - dispose', () => {
+  it('expect reset insideBoundingBox', () => {
+    const render = jest.fn();
+    const unmount = jest.fn();
+
+    const customGeoSearch = connectGeoSearch(render, unmount);
+    const widget = customGeoSearch({
+      enableGeolocationWithIP: false,
+    });
+
+    const client = createFakeClient();
+    const helper = createFakeHelper(client);
+
+    helper
+      .setState(widget.getConfiguration(new SearchParameters()))
+      .setQueryParameter('insideBoundingBox', '10,12,12,14');
+
+    const expectation = {
+      insideBoundingBox: undefined,
+    };
+
+    const actual = widget.dispose({ state: helper.getState() });
+
+    expect(unmount).toHaveBeenCalled();
+    expect(actual).toMatchObject(expectation);
+  });
+
+  it('expect reset multiple parameters', () => {
+    const render = jest.fn();
+    const unmount = jest.fn();
+
+    const customGeoSearch = connectGeoSearch(render, unmount);
+    const widget = customGeoSearch({
+      radius: 100,
+      precision: 25,
+      position: {
+        lat: 10,
+        lng: 12,
+      },
+    });
+
+    const client = createFakeClient();
+    const helper = createFakeHelper(client);
+
+    helper.setState(widget.getConfiguration(new SearchParameters()));
+
+    const expectation = {
+      aroundRadius: undefined,
+      aroundPrecision: undefined,
+      aroundLatLng: undefined,
+    };
+
+    const actual = widget.dispose({ state: helper.getState() });
+
+    expect(unmount).toHaveBeenCalled();
+    expect(actual).toMatchObject(expectation);
+  });
+
   describe('aroundLatLngViaIP', () => {
     it('expect reset aroundLatLngViaIP', () => {
       const render = jest.fn();

src/connectors/geo-search/connectGeoSearch.js

@@ -313,21 +313,23 @@ const connectGeoSearch = (renderFn, unmountFn) => {
         let nextState = state;
 
         if (enableGeolocationWithIP && !position) {
-          nextState = state.setQueryParameter('aroundLatLngViaIP');
+          nextState = nextState.setQueryParameter('aroundLatLngViaIP');
         }
 
         if (position) {
-          nextState = state.setQueryParameter('aroundLatLng');
+          nextState = nextState.setQueryParameter('aroundLatLng');
         }
 
         if (radius) {
-          nextState = state.setQueryParameter('aroundRadius');
+          nextState = nextState.setQueryParameter('aroundRadius');
         }
 
         if (precision) {
-          nextState = state.setQueryParameter('aroundPrecision');
+          nextState = nextState.setQueryParameter('aroundPrecision');
         }
 
+        nextState = nextState.setQueryParameter('insideBoundingBox');
+
         return nextState;
       },
     };