Demonstrating that the layer.destroyFeatures() issue is already fixed (see #2649).
git-svn-id: http://svn.openlayers.org/trunk/openlayers@10651 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf
This commit is contained in:
+14
-5
@@ -412,12 +412,12 @@
|
|||||||
}
|
}
|
||||||
|
|
||||||
function test_Layer_Vector_destroyFeatures (t) {
|
function test_Layer_Vector_destroyFeatures (t) {
|
||||||
t.plan(5);
|
t.plan(8);
|
||||||
layer = new OpenLayers.Layer.Vector(name);
|
var layer = new OpenLayers.Layer.Vector(name);
|
||||||
var map = new OpenLayers.Map('map');
|
var map = new OpenLayers.Map('map');
|
||||||
map.addLayer(layer);
|
map.addLayer(layer);
|
||||||
var features = [];
|
var features = [], i;
|
||||||
for (var i = 0; i < 5; i++) {
|
for (i = 0; i < 5; i++) {
|
||||||
features.push(new OpenLayers.Feature.Vector(
|
features.push(new OpenLayers.Feature.Vector(
|
||||||
new OpenLayers.Geometry.Point(0,0)));
|
new OpenLayers.Geometry.Point(0,0)));
|
||||||
}
|
}
|
||||||
@@ -427,8 +427,15 @@
|
|||||||
layer.destroyFeatures();
|
layer.destroyFeatures();
|
||||||
t.eq(layer.features.length, 0, "destroyFeatures triggers removal");
|
t.eq(layer.features.length, 0, "destroyFeatures triggers removal");
|
||||||
t.eq(layer.selectedFeatures, [], "Destroy features removes selected features");
|
t.eq(layer.selectedFeatures, [], "Destroy features removes selected features");
|
||||||
|
var allDestroyed = true;
|
||||||
|
for (i = 0; i < 5; i++) {
|
||||||
|
if(features[i].geometry) {
|
||||||
|
allDestroyed = false;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
t.ok(allDestroyed, "destroyFeatures actually destroys features");
|
||||||
features = [];
|
features = [];
|
||||||
for (var i = 0; i < 5; i++) {
|
for (i = 0; i < 5; i++) {
|
||||||
features.push(new OpenLayers.Feature.Vector(
|
features.push(new OpenLayers.Feature.Vector(
|
||||||
new OpenLayers.Geometry.Point(0,0)));
|
new OpenLayers.Geometry.Point(0,0)));
|
||||||
}
|
}
|
||||||
@@ -438,6 +445,8 @@
|
|||||||
layer.destroyFeatures([features[0], features[1]]);
|
layer.destroyFeatures([features[0], features[1]]);
|
||||||
t.eq(layer.features.length, 3, "destroyFeatures removes appropriate features");
|
t.eq(layer.features.length, 3, "destroyFeatures removes appropriate features");
|
||||||
t.eq(layer.selectedFeatures, [], "destroyFeatures removes appropriate selected features");
|
t.eq(layer.selectedFeatures, [], "destroyFeatures removes appropriate selected features");
|
||||||
|
t.eq(features[0].geometry, null, "destroyFeatures destroys feature 0");
|
||||||
|
t.eq(features[1].geometry, null, "destroyFeatures destroys feature 1");
|
||||||
}
|
}
|
||||||
|
|
||||||
function test_Layer_Vector_destroy (t) {
|
function test_Layer_Vector_destroy (t) {
|
||||||
|
|||||||
Reference in New Issue
Block a user