From a8b5ce5ae8e450bd02d2ab674de9f2caa1b13ed0 Mon Sep 17 00:00:00 2001
From: Piotr Gawron <piotr.gawron@uni.lu>
Date: Wed, 31 Oct 2018 09:30:07 +0100
Subject: [PATCH] problem with getBounds method

---
 .../canvas/OpenLayers/OpenLayerPolyline.js    |  2 +-
 .../OpenLayers/OpenLayersPolyline-test.js     | 44 +++++++++++++++++++
 2 files changed, 45 insertions(+), 1 deletion(-)
 create mode 100644 frontend-js/src/test/js/map/canvas/OpenLayers/OpenLayersPolyline-test.js

diff --git a/frontend-js/src/main/js/map/canvas/OpenLayers/OpenLayerPolyline.js b/frontend-js/src/main/js/map/canvas/OpenLayers/OpenLayerPolyline.js
index e87a516537..6ee77ee559 100644
--- a/frontend-js/src/main/js/map/canvas/OpenLayers/OpenLayerPolyline.js
+++ b/frontend-js/src/main/js/map/canvas/OpenLayers/OpenLayerPolyline.js
@@ -83,7 +83,7 @@ OpenLayerPolyline.prototype.isShown = function () {
 
 OpenLayerPolyline.prototype.getBounds = function () {
   var self = this;
-  var extent = self.getOpenLayersRectangle().getGeometry().getExtent();
+  var extent = self.getOpenLayersPolyline().getGeometry().getExtent();
 
   var projection1 = [extent[0], extent[1]];
   var p1 = self.getMap().fromProjectionToPoint(projection1);
diff --git a/frontend-js/src/test/js/map/canvas/OpenLayers/OpenLayersPolyline-test.js b/frontend-js/src/test/js/map/canvas/OpenLayers/OpenLayersPolyline-test.js
new file mode 100644
index 0000000000..f6107f6b54
--- /dev/null
+++ b/frontend-js/src/test/js/map/canvas/OpenLayers/OpenLayersPolyline-test.js
@@ -0,0 +1,44 @@
+"use strict";
+require("../../../mocha-config");
+
+// noinspection JSUnusedLocalSymbols
+var logger = require('../../../logger');
+
+var OpenLayerCanvas = require('../../../../../main/js/map/canvas/OpenLayers/OpenLayerCanvas');
+var OpenLayerPolyline = require('../../../../../main/js/map/canvas/OpenLayers/OpenLayerPolyline');
+var Bounds = require('../../../../../main/js/map/canvas/Bounds');
+var Point = require('../../../../../main/js/map/canvas/Point');
+
+var SelectionContextMenu = require('../../../../../main/js/gui/SelectionContextMenu');
+
+var chai = require('chai');
+var assert = chai.assert;
+
+describe('OpenLayerPolyline', function () {
+  var testOptions = {
+    center: new Point(0, 0),
+    tileSize: 256,
+    width: 300,
+    height: 600,
+    zoom: 3,
+    minZoom: 2,
+    maxZoom: 10,
+    backgroundOverlays: [{
+      id: 1,
+      name: "overlay",
+      directory: "overlay_dir"
+    }]
+  };
+  it("getBounds", function () {
+    var canvas = new OpenLayerCanvas(testDiv, testOptions);
+    var path = [new Point(0, 0), new Point(10, 5)];
+
+    var polyline = canvas.createPolyline({
+      strokeWeight: 1,
+      strokeColor: "#00ff00",
+      path: path
+    });
+    polyline.show();
+    assert.ok(polyline.getBounds());
+  });
+});
-- 
GitLab