@osimatic/helpers-js 1.5.15 → 1.5.16

package/open_street_map.js

@@ -30,16 +30,16 @@ class OpenStreetMap {
 
 	static createMap(mapContainer, options={}) {
 		mapContainer = toEl(mapContainer);
-		if (!mapContainer.length) {
+		if (!mapContainer) {
 			return null;
 		}
 
-		const container = L.DomUtil.get(mapContainer[0]);
+		const container = L.DomUtil.get(mapContainer);
 		if (container != null) {
 			container._leaflet_id = null;
 		}
 
-		const map = L.map(mapContainer[0], options);
+		const map = L.map(mapContainer, options);
 
 		L.tileLayer('https://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png', {
 			attribution: '&copy; <a href="https://www.openstreetmap.org/copyright">OpenStreetMap</a>'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@osimatic/helpers-js",
-  "version": "1.5.15",
+  "version": "1.5.16",
   "main": "main.js",
   "scripts": {
     "test": "jest",

package/tests/open_street_map.test.js

@@ -215,6 +215,100 @@ describe('OpenStreetMap', () => {
 		});
 	});
 
+	describe('createMap', () => {
+		let mockMap;
+		let mockTileLayer;
+		let domUtilGetSpy;
+		let mapSpy;
+		let tileLayerSpy;
+
+		beforeEach(() => {
+			mockMap = {
+				setView: jest.fn(),
+				getZoom: jest.fn(() => 6),
+				setZoom: jest.fn(),
+				fitBounds: jest.fn(),
+				invalidateSize: jest.fn(),
+			};
+			mockTileLayer = { addTo: jest.fn() };
+			domUtilGetSpy = jest.spyOn(L.DomUtil, 'get').mockReturnValue(null);
+			mapSpy = jest.spyOn(L, 'map').mockReturnValue(mockMap);
+			tileLayerSpy = jest.spyOn(L, 'tileLayer').mockReturnValue(mockTileLayer);
+		});
+
+		afterEach(() => {
+			domUtilGetSpy.mockRestore();
+			mapSpy.mockRestore();
+			tileLayerSpy.mockRestore();
+		});
+
+		test('should return null when mapContainer is null', () => {
+			const result = OpenStreetMap.createMap(null);
+
+			expect(result).toBeNull();
+			expect(mapSpy).not.toHaveBeenCalled();
+		});
+
+		test('should return null when mapContainer is undefined', () => {
+			const result = OpenStreetMap.createMap(undefined);
+
+			expect(result).toBeNull();
+			expect(mapSpy).not.toHaveBeenCalled();
+		});
+
+		test('should create and return a Leaflet map for a valid DOM element', () => {
+			const div = document.createElement('div');
+			document.body.appendChild(div);
+
+			const result = OpenStreetMap.createMap(div);
+
+			expect(mapSpy).toHaveBeenCalledWith(div, {});
+			expect(tileLayerSpy).toHaveBeenCalled();
+			expect(mockTileLayer.addTo).toHaveBeenCalledWith(mockMap);
+			expect(result).toBe(mockMap);
+
+			document.body.removeChild(div);
+		});
+
+		test('should reset _leaflet_id when container already has one', () => {
+			const div = document.createElement('div');
+			document.body.appendChild(div);
+			const existingContainer = { _leaflet_id: 42 };
+			domUtilGetSpy.mockReturnValue(existingContainer);
+
+			OpenStreetMap.createMap(div);
+
+			expect(existingContainer._leaflet_id).toBeNull();
+
+			document.body.removeChild(div);
+		});
+
+		test('should pass options to L.map', () => {
+			const div = document.createElement('div');
+			document.body.appendChild(div);
+			const options = { zoomControl: false };
+
+			OpenStreetMap.createMap(div, options);
+
+			expect(mapSpy).toHaveBeenCalledWith(div, options);
+
+			document.body.removeChild(div);
+		});
+
+		test('should center on France by default', () => {
+			const div = document.createElement('div');
+			document.body.appendChild(div);
+			const fitBoundsSpy = jest.spyOn(L, 'latLngBounds').mockReturnValue({ _bounds: [] });
+
+			OpenStreetMap.createMap(div);
+
+			expect(mockMap.fitBounds).toHaveBeenCalled();
+
+			fitBoundsSpy.mockRestore();
+			document.body.removeChild(div);
+		});
+	});
+
 	describe('centerOnFrance', () => {
 		test('should call setView with France coordinates', () => {
 			const mockMap = {