Live location sharing: remove geoUri logs (#8465)

* remove geoUri logs Signed-off-by: Kerry Archibald <[email protected]> * Update src/components/views/location/Map.tsx Co-authored-by: Michael Telatynski <[email protected]> Co-authored-by: Michael Telatynski <[email protected]>
matrix-org · May 3, 2022 · 3b1e715 · 3b1e715
1 parent f280fab
commit 3b1e715
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/components/views/location/Map.tsx b/src/components/views/location/Map.tsx
@@ -51,8 +51,8 @@ const useMapWithStyle = ({ id, centerGeoUri, onError, interactive, bounds }) =>
             try {
                 const coords = parseGeoUri(centerGeoUri);
                 map.setCenter({ lon: coords.longitude, lat: coords.latitude });
-            } catch (error) {
-                logger.error('Could not set map center', centerGeoUri);
+            } catch (_error) {
+                logger.error('Could not set map center');
             }
         }
     }, [map, centerGeoUri]);
@@ -65,8 +65,8 @@ const useMapWithStyle = ({ id, centerGeoUri, onError, interactive, bounds }) =>
                     [bounds.east, bounds.north],
                 );
                 map.fitBounds(lngLatBounds, { padding: 100, maxZoom: 15 });
-            } catch (error) {
-                logger.error('Invalid map bounds', error);
+            } catch (_error) {
+                logger.error('Invalid map bounds');
             }
         }
     }, [map, bounds]);

diff --git a/test/components/views/location/Map-test.tsx b/test/components/views/location/Map-test.tsx
@@ -101,7 +101,7 @@ describe('<Map />', () => {
             const logSpy = jest.spyOn(logger, 'error').mockImplementation();
             getComponent({ centerGeoUri: '123 Sesame Street' });
             expect(mockMap.setCenter).not.toHaveBeenCalled();
-            expect(logSpy).toHaveBeenCalledWith('Could not set map center', '123 Sesame Street');
+            expect(logSpy).toHaveBeenCalledWith('Could not set map center');
         });
 
         it('updates map center when centerGeoUri prop changes', () => {
@@ -133,7 +133,7 @@ describe('<Map />', () => {
             const bounds = { north: 'a', south: 'b', east: 42, west: 41 };
             getComponent({ bounds });
             expect(mockMap.fitBounds).not.toHaveBeenCalled();
-            expect(logSpy).toHaveBeenCalledWith('Invalid map bounds', new Error('Invalid LngLat object: (41, NaN)'));
+            expect(logSpy).toHaveBeenCalledWith('Invalid map bounds');
         });
 
         it('updates map bounds when bounds prop changes', () => {