CalvinBackup/iD
1
0
Fork 0
mirror of https://github.com/FoggedLens/iD.git synced 2026-05-25 01:24:05 +02:00
Code Issues Packages Projects Releases Wiki Activity

Add pending tests for #508

Browse Source 
Update DeleteWay tests to use Graph#update. Eventually we
should do this for all action tests.
This commit is contained in:
John Firebaugh
2013-01-25 16:33:17 -05:00
parent 3b7b15a12f
commit 98073b0d26
1 changed files with 28 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
test/spec/actions/delete_way.js
+28 -5
View File
@@ -2,7 +2,7 @@ describe("iD.actions.DeleteWay", function () {
it("removes the way from the graph", function () {
var way = iD.Way(),
action = iD.actions.DeleteWay(way.id),
graph = action(iD.Graph([way]));
graph = iD.Graph([way]).update(action);
expect(graph.entity(way.id)).to.be.undefined;
});
@@ -10,7 +10,7 @@ describe("iD.actions.DeleteWay", function () {
var way = iD.Way(),
relation = iD.Relation({members: [{ id: way.id }]}),
action = iD.actions.DeleteWay(way.id),
graph = action(iD.Graph([way, relation]));
graph = iD.Graph([way, relation]).update(action);
expect(_.pluck(graph.entity(relation.id).members, 'id')).not.to.contain(way.id);
});
@@ -18,7 +18,7 @@ describe("iD.actions.DeleteWay", function () {
var node = iD.Node(),
way = iD.Way({nodes: [node.id]}),
action = iD.actions.DeleteWay(way.id),
graph = action(iD.Graph([node, way]));
graph = iD.Graph([node, way]).update(action);
expect(graph.entity(node.id)).to.be.undefined;
});
@@ -27,15 +27,38 @@ describe("iD.actions.DeleteWay", function () {
way1 = iD.Way({nodes: [node.id]}),
way2 = iD.Way({nodes: [node.id]}),
action = iD.actions.DeleteWay(way1.id),
graph = action(iD.Graph([node, way1, way2]));
graph = iD.Graph([node, way1, way2]).update(action);
expect(graph.entity(node.id)).not.to.be.undefined;
});
// See #508
xit("deletes multiple member nodes", function () {
var a = iD.Node(),
b = iD.Node(),
way = iD.Way({nodes: [a.id, b.id]}),
action = iD.actions.DeleteWay(way.id),
graph = iD.Graph([a, b, way]).update(action);
expect(graph.entity(a.id)).to.be.undefined;
expect(graph.entity(b.id)).to.be.undefined;
});
xit("deletes a circular way's start/end node", function () {
var a = iD.Node(),
b = iD.Node(),
c = iD.Node(),
way = iD.Way({nodes: [a.id, b.id, c.id, a.id]}),
action = iD.actions.DeleteWay(way.id),
graph = iD.Graph([a, b, way]).update(action);
expect(graph.entity(a.id)).to.be.undefined;
expect(graph.entity(b.id)).to.be.undefined;
expect(graph.entity(c.id)).to.be.undefined;
});
it("does not delete member nodes with interesting tags", function () {
var node = iD.Node({tags: {highway: 'traffic_signals'}}),
way = iD.Way({nodes: [node.id]}),
action = iD.actions.DeleteWay(way.id),
graph = action(iD.Graph([node, way]));
graph = iD.Graph([node, way]).update(action);
expect(graph.entity(node.id)).not.to.be.undefined;
});
});