From e1cf49eaffd6e7d58f898c997a3c1ed5a312bcad Mon Sep 17 00:00:00 2001 From: Bryan Housel Date: Wed, 28 Feb 2018 13:42:26 -0500 Subject: [PATCH] Remove the "excludes coincident highways" check These should not be a thing, and if they are, I'm ok with them being treated as 2 separate ways. We can add code back in later if it turns out to be a widespread issue in OSM. --- test/spec/osm/intersection.js | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/test/spec/osm/intersection.js b/test/spec/osm/intersection.js index 9f287e328..880b281a5 100644 --- a/test/spec/osm/intersection.js +++ b/test/spec/osm/intersection.js @@ -22,17 +22,6 @@ describe('iD.osmIntersection', function() { expect(result.map(function(i) { return i.id; })).to.eql(['=']); }); -//TODO? - // it('excludes coincident highways', function() { - // var graph = iD.coreGraph([ - // iD.osmNode({id: 'u'}), - // iD.osmNode({id: '*'}), - // iD.osmWay({id: '=', nodes: ['u', '*'], tags: {highway: 'residential'}}), - // iD.osmWay({id: '-', nodes: ['u', '*'], tags: {highway: 'residential'}}) - // ]); - // expect(iD.osmIntersection(graph, '*').ways).to.eql([]); - // }); - it('includes line highways', function() { var graph = iD.coreGraph([ iD.osmNode({id: 'u'}),