fix specs for polygon holes

ilblog · Dec 18, 2013 · 33e23a6 · 33e23a6
1 parent dc5151b
commit 33e23a6
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/debug/vector/vector2.html b/debug/vector/vector2.html
@@ -26,7 +26,7 @@
 
 		var path = L.polygon([
 			[route, [[50.5, 30.5], [50.5, 40], [40, 40]]],
-			[[20, 0], [20, 40], [0, 40]]
+			[[[20, 0], [20, 40], [0, 40]]]
 		], {renderer: canvas}).addTo(map).bindPopup('Hello Canvas');
 
 		var poly = L.polyline([

diff --git a/spec/suites/layer/vector/PolygonSpec.js b/spec/suites/layer/vector/PolygonSpec.js
@@ -36,8 +36,11 @@ describe('Polygon', function () {
 
 			var polygon = new L.Polygon(originalLatLngs);
 
-			//getLatLngs() returns only external ring
-			expect(polygon.getLatLngs()).to.eql([L.latLng([0, 10]), L.latLng([10, 10]), L.latLng([10, 0])]);
+			// getLatLngs() returns both rings
+			expect(polygon.getLatLngs()).to.eql([
+				[L.latLng([0, 10]), L.latLng([10, 10]), L.latLng([10, 0])],
+				[L.latLng([2, 3]), L.latLng([2, 4]), L.latLng([3, 4])]
+			]);
 		});
 	});
 
@@ -68,8 +71,10 @@ describe('Polygon', function () {
 			var polygon = new L.Polygon([]);
 			polygon.setLatLngs(latLngs);
 
-			//getLatLngs() returns only external ring
-			expect(polygon.getLatLngs()).to.eql([L.latLng([0, 10]), L.latLng([10, 10]), L.latLng([10, 0])]);
+			expect(polygon.getLatLngs()).to.eql([
+				[L.latLng([0, 10]), L.latLng([10, 10]), L.latLng([10, 0])],
+				[L.latLng([2, 3]), L.latLng([2, 4]), L.latLng([3, 4])]
+			]);
 		});
 	});