From ce85e7faafa910731949ab5685f73f44fd0030d0 Mon Sep 17 00:00:00 2001 From: tyr Date: Sun, 29 Sep 2013 13:14:34 +0200 Subject: [PATCH] fix test case "does not delete tagged nodes" of iD.actions.Straighten The nodes.sort() statement caused some trouble in older browsers such as (FF 20, Chromium 25). Also, the straighten action should conserve node order. Thus, the sorting made the test weaker than it had to be. This just removes the unnecessary sorting. --- test/spec/actions/straighten.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/spec/actions/straighten.js b/test/spec/actions/straighten.js index 9278d3bc8..e58b69718 100644 --- a/test/spec/actions/straighten.js +++ b/test/spec/actions/straighten.js @@ -24,7 +24,7 @@ describe("iD.actions.Straighten", function () { graph = iD.actions.Straighten('-', projection)(graph); - expect(graph.entity('-').nodes.sort()).to.eql(['a', 'b', 'c']); + expect(graph.entity('-').nodes).to.eql(['a', 'b', 'c']); }); it("does not delete nodes connected to other ways", function() {