From 94fcc049ce003a40a5f7d7e4ca14d09f9a380e07 Mon Sep 17 00:00:00 2001 From: Michael Jurkoic Date: Sun, 10 Jan 2021 10:10:09 -0800 Subject: [PATCH] Improved renderer clone tests Removed test that asserted the cloned renderer would not reference the same object as the original. Replaced tests that stringified the renderer function with tests that compare the original with the clone by reference. --- test/spec/ol/style/style.test.js | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/test/spec/ol/style/style.test.js b/test/spec/ol/style/style.test.js index 5733002e1b..210c0bb577 100644 --- a/test/spec/ol/style/style.test.js +++ b/test/spec/ol/style/style.test.js @@ -68,9 +68,7 @@ describe('ol.style.Style', function () { expect(original.getImage().getRadius()).to.eql( clone.getImage().getRadius() ); - expect(original.getRenderer().toString()).to.eql( - clone.getRenderer().toString() - ); + expect(original.getRenderer()).to.eql(clone.getRenderer()); expect(original.getStroke().getColor()).to.eql( clone.getStroke().getColor() ); @@ -102,7 +100,6 @@ describe('ol.style.Style', function () { expect(original.getGeometry()).not.to.be(clone.getGeometry()); expect(original.getFill()).not.to.be(clone.getFill()); expect(original.getImage()).not.to.be(clone.getImage()); - expect(original.getRenderer()).not.to.be(clone.getRenderer()); expect(original.getStroke()).not.to.be(clone.getStroke()); expect(original.getText()).not.to.be(clone.getText()); @@ -123,9 +120,7 @@ describe('ol.style.Style', function () { expect(original.getImage().getScale()).not.to.eql( clone.getImage().getScale() ); - expect(original.getRenderer().toString()).not.to.eql( - clone.getRenderer().toString() - ); + expect(original.getRenderer()).not.to.eql(clone.getRenderer()); expect(original.getStroke().getColor()).not.to.eql( clone.getStroke().getColor() );