@masterportal/masterportalapi 2.15.2 → 2.17.0

package/CHANGELOG.md

@@ -3,25 +3,54 @@
 
  The [Semantic Versioning](https://semver.org/spec/v2.0.0.html) is used.
 
-## Unreleased - in development
-###
-Added
-###
-Changed
-###
-Deprecated
-### 
-Removed
-### 
-Fixed
+## 2.17.0 - 2023-04-27
+### Added
+- Added functionality for mapbox style raster sources for vector tile layer.
+- Added possibility to use zoomToFeatureId in vectorStyle.
+
+### Changed
+- ol map default interactions and keyboardEventTarget can now be set by config parameters.
+- The following packages have been updated:
+    - dependencies:
+        - xml2js: 0.4.23 to 0.5.0
+
+### Fix
+- Fix vectorStyle for circlesegments.
+
+---
+
+## 2.16.0 - 2023-03-28
+### Added
+- Added support for `npm` version ^9.1.3.
+
+### Changed
+- The following packages have been updated:
+    - dependencies:
+        - core-js: 3.27.1 to 3.29.1
+        - ol: 7.2.2 to 7.3.0
+        - proj4: 2.8.1 to 2.9.0
+    - devDependencies:
+        - @babel/core: 7.20.12 to 7.21.3
+        - @babel/plugin-transform-modules-commonjs: 7.20.11 to 7.21.2
+        - eslint: 8.31.0 to 8.36.0
+        - jest: 29.3.1 to 29.5.0
+        - jest-environment-jsdom: 29.3.1 to 29.5.0
+        - jsdoc: 4.0.0 to 4.0.2
+        - sass: 1.57.1 to 1.59.3
+
+### Fixed
+- Issue Masterportal.#870: Features with `null` style were shown with default style. However, the default style is only supposed to be used for fallbacks; `null` style features must explicitly be invisible. This has been resolved.
+- Style of legend is now always not clustered.
+
 ---
 
 ## 2.15.2 - 2023-03-14
-###
-Changed
+### Changed
 - Update parameters for getGeometryTypeFromService.getGeometryTypeFromWFS and getGeometryTypeFromService.getGeometryTypeFromOAF to pass wfsImgPath.
+
 ### Fixed
 - Fix imports in createLegendInfo.js
+
 ---
 
 ## 2.15.1 - 2023-03-09

package/example/index.js

@@ -262,7 +262,7 @@ fetch(vtStyleUrl)
     .then(style => {
         const vtLayer = window.mpapi.map.addLayer("6001");
 
-        mpapi.vectorTile.setStyle(vtLayer, style);
+        mpapi.vectorTile.setStyle(vtLayer, style, vtStyleUrl);
     });
 //*/
 

package/package.json

@@ -1,63 +1,64 @@
-{
-  "name": "@masterportal/masterportalapi",
-  "author": "Implementierungspartnerschaft Masterportal <info@masterportal.org> (https://www.masterportal.org)",
-  "version": "2.15.2",
-  "license": "MIT",
-  "description": "Basic functions of the Masterportal as api",
-  "repository": {
-    "type": "git",
-    "url": "git+https://bitbucket.org/geowerkstatt-hamburg/masterportalapi.git"
-  },
-  "main": "src/index.js",
-  "scripts": {
-    "test": "jest .test.js --config ./jest.config.js --collectCoverage",
-    "test:trace": "node --trace-warnings node_modules/jest/bin/jest.js ./test/layer/wfs.test.js --config ./jest.config.js",
-    "test:watch": "jest .test.js --config ./jest.config.js --watch",
-    "example": "parcel example/index.html",
-    "generate-jsdoc": "jsdoc ./src -r -d ./docs/ -R ./README.md -c jsdoc.json",
-    "prePushHook": "eslint --max-warnings 0 \"./**/*.js\" && npm run test",
-    "prepare": "husky install"
-  },
-  "dependencies": {
-    "core-js": "^3.27.1",
-    "ol": "7.2.2",
-    "olcs": "^2.13.1",
-    "proj4": "^2.8.1",
-    "xml2js": "^0.4.23"
-  },
-  "devDependencies": {
-    "@babel/core": "^7.20.12",
-    "@babel/plugin-transform-modules-commonjs": "^7.20.11",
-    "@parcel/transformer-sass": "^2.6.0",
-    "buffer": "^6.0.3",
-    "canvas": "^2.11.0",
-    "eslint": "^8.31.0",
-    "events": "^3.3.0",
-    "husky": "^8.0.3",
-    "jest": "^29.3.1",
-    "jest-canvas-mock": "^2.4.0",
-    "jest-environment-jsdom": "^29.3.1",
-    "jsdoc": "^4.0.0",
-    "parcel": "^2.6.0",
-    "process": "^0.11.10",
-    "sass": "^1.57.1",
-    "stream-browserify": "^3.0.0",
-    "timers-browserify": "^2.0.12",
-    "use-resize-observer": "^9.1.0"
-  },
-  "peerDependencies": {
-    "cesium": "^1.95.0"
-  },
-  "engines": {
-    "node": "^16.13.2 || ^18.12.0",
-    "npm": "^8.1.2"
-  },
-  "homepage": "https://bitbucket.org/geowerkstatt-hamburg/masterportalapi#readme",
-  "directories": {
-    "example": "example",
-    "test": "test"
-  },
-  "bugs": {
-    "url": "https://bitbucket.org/geowerkstatt-hamburg/masterportalapi/issues"
-  }
-}
+{
+  "name": "@masterportal/masterportalapi",
+  "author": "Implementierungspartnerschaft Masterportal <info@masterportal.org> (https://www.masterportal.org)",
+  "version": "2.17.0",
+  "license": "MIT",
+  "description": "Basic functions of the Masterportal as api",
+  "repository": {
+    "type": "git",
+    "url": "git+https://bitbucket.org/geowerkstatt-hamburg/masterportalapi.git"
+  },
+  "main": "src/index.js",
+  "scripts": {
+    "test": "jest .test.js --config ./jest.config.js --collectCoverage",
+    "test:trace": "node --trace-warnings node_modules/jest/bin/jest.js ./test/layer/wfs.test.js --config ./jest.config.js",
+    "test:watch": "jest .test.js --config ./jest.config.js --watch",
+    "example": "parcel example/index.html",
+    "generate-jsdoc": "jsdoc ./src -r -d ./docs/ -R ./README.md -c jsdoc.json",
+    "prePushHook": "eslint --max-warnings 0 \"./**/*.js\" && npm run test",
+    "prepare": "husky install"
+  },
+  "dependencies": {
+    "core-js": "^3.29.1",
+    "ol": "7.3.0",
+    "olcs": "^2.13.1",
+    "proj4": "^2.9.0",
+    "xml2js": "^0.5.0"
+  },
+  "devDependencies": {
+    "@babel/core": "^7.21.3",
+    "@babel/plugin-transform-modules-commonjs": "^7.21.2",
+    "@parcel/transformer-sass": "^2.6.0",
+    "buffer": "^6.0.3",
+    "canvas": "^2.11.0",
+    "eslint": "^8.36.0",
+    "events": "^3.3.0",
+    "husky": "^8.0.3",
+    "jest": "^29.5.0",
+    "jest-canvas-mock": "^2.4.0",
+    "jest-environment-jsdom": "^29.5.0",
+    "jsdoc": "^4.0.2",
+    "parcel": "^2.6.0",
+    "process": "^0.11.10",
+    "sass": "^1.59.3",
+    "stream-browserify": "^3.0.0",
+    "timers-browserify": "^2.0.12",
+    "use-resize-observer": "^9.1.0"
+  },
+  "peerDependencies": {
+    "cesium": "^1.95.0"
+  },
+  "engines": {
+    "node": "^16.13.2 || ^18.12.0",
+    "npm": "^8.1.2 || ^9.3.1"
+  },
+  "homepage": "https://bitbucket.org/geowerkstatt-hamburg/masterportalapi#readme",
+  "directories": {
+    "doc": "docs",
+    "example": "example",
+    "test": "test"
+  },
+  "bugs": {
+    "url": "https://bitbucket.org/geowerkstatt-hamburg/masterportalapi/issues"
+  }
+}

package/src/layer/vectorTile.js

@@ -4,7 +4,7 @@ import VectorTileSource from "ol/source/VectorTile";
 import OpenLayersTileGrid from "ol/tilegrid/TileGrid";
 import {extentFromProjection} from "ol/tilegrid";
 
-import {stylefunction} from "ol-mapbox-style";
+import {stylefunction, apply} from "ol-mapbox-style";
 
 import defaults from "../defaults";
 
@@ -161,13 +161,25 @@ export function createLayer (rawLayer = {}, {layerParams = {}, options = {}} = {
  * @param {ol.layer.VectorTile} layer - layer to style
  * @param {object} glStyle - MapBox style definition
  * @param {options} [styleParams={}] - Additional options for style specification
+ * @param {string} url - MapBox style url
  * @returns {void}
  */
-export function setStyle (layer, glStyle, {options = {}} = {}) {
-    stylefunction(layer, glStyle,
-        options.sourceId ? options.sourceId : Object.keys(glStyle.sources)[0],
-        options.resolutions,
-        options.spriteData,
-        options.spriteImageUrl,
-        options.getFonts);
+export function setStyle (layer, glStyle, {options = {}} = {}, url) {
+    Object.keys(glStyle.sources).forEach((key) => {
+        const glStyleSourceId = key;
+
+        if (glStyle.sources[key].type !== "vector" && glStyle.sources[key].type !== "geojson") {
+            const olMap = layer.get("map");
+
+            apply(olMap, url);
+        }
+        else {
+            stylefunction(layer, glStyle,
+                options.sourceId ? options.sourceId : glStyleSourceId,
+                options.resolutions,
+                options.spriteData,
+                options.spriteImageUrl,
+                options.getFonts);
+        }
+    });
 }

package/src/maps/ol/olMap.js

@@ -150,18 +150,23 @@ export function createMap (config = defaults, {mapParams, callback, errorCallbac
     crs.registerProjections(config.namedProjections);
     setBackgroundImage(config);
     setGazetteerUrl(config.gazetteerUrl);
-    const map = new Map(Object.assign({
-        target: config.target || defaults.target,
-        interactions: olDefaultInteractions({dragPan: false, altShiftDragRotate: false, pinchRotate: false}).extend([
-            new DragPan({
-                condition: function (event) {
-                    return (!event.originalEvent.pointerType || event.originalEvent.pointerType === "mouse") || (config.twoFingerPan && this.getPointerCount() === 2) || !config.twoFingerPan;
-                }
-            })
-        ]),
-        controls: [],
-        view: createMapView(config)
-    }, mapParams));
+    const selectedInteractions = config.mapInteractions?.interactionModes ? config.mapInteractions?.interactionModes : {dragPan: false, altShiftDragRotate: false, pinchRotate: false},
+        map = new Map(Object.assign({
+            target: config.target || defaults.target,
+            interactions: olDefaultInteractions(selectedInteractions).extend([
+                new DragPan({
+                    condition: function (event) {
+                        if (event.originalEvent.shiftKey) {
+                            return false;
+                        }
+                        return (!event.originalEvent.pointerType || event.originalEvent.pointerType === "mouse") || (config.twoFingerPan && this.getPointerCount() === 2) || !config.twoFingerPan;
+                    }
+                })
+            ]),
+            controls: [],
+            view: createMapView(config),
+            keyboardEventTarget: config.mapInteractions?.keyboardEventTarget ? config.mapInteractions?.keyboardEventTarget : false
+        }, mapParams));
 
     map.set("mapMode", "2D");
     map.set("id", `map2D_${mapIdCounter++}`);

package/src/vectorStyle/createStyle.js

@@ -32,7 +32,8 @@ function getSimpleGeometryStyle (geometryType, feature, rule, isClustered, wfsIm
     const style = rule?.style,
         legendValue = style ? style.legendValue : null,
         alreadyExisitingStyle = uniqueStyles.find(element => element.featureStyle === style && element.geometryType === geometryType);
-    let styleObject;
+    let styleObject,
+        styleObjectForLegend = null;
 
     feature.legendValue = legendValue ? legendValue : null;
 
@@ -41,12 +42,21 @@ function getSimpleGeometryStyle (geometryType, feature, rule, isClustered, wfsIm
     }
     else if (geometryType === "Point") {
         styleObject = new PointStyle(feature, style, isClustered);
+        if (isClustered) {
+            styleObjectForLegend = new PointStyle(feature, style, false);
+        }
     }
     else if (geometryType === "LineString") {
         styleObject = new LineStringStyle(feature, style, isClustered);
+        if (isClustered) {
+            styleObjectForLegend = new LineStringStyle(feature, style, false);
+        }
     }
     else if (geometryType === "Polygon") {
         styleObject = new PolygonStyle(feature, style, isClustered);
+        if (isClustered) {
+            styleObjectForLegend = new PolygonStyle(feature, style, false);
+        }
     }
     else if (geometryType === "Cesium") {
         styleObject = new CesiumStyle(rule);
@@ -63,9 +73,14 @@ function getSimpleGeometryStyle (geometryType, feature, rule, isClustered, wfsIm
         console.warn("Geometry type not implemented: " + geometryType + " default style ist used for feature " + feature);
         return new Style();
     }
+
+    if (styleObjectForLegend !== null) {
+        styleObjectForLegend.initialize(feature, style, false, wfsImgPathFromConfig);
+        styleObjectForLegend.legendValue = legendValue;
+    }
     uniqueStyles.push({featureStyle: style, featureStyleObject: styleObject, geometryType});
     styleObject.initialize(feature, style, isClustered, wfsImgPathFromConfig);
-    styleObject.addLegendInfo(geometryType, styleObject, rule);
+    styleObject.addLegendInfo(geometryType, styleObjectForLegend === null ? styleObject : styleObjectForLegend, rule);
     styleObject.legendValue = legendValue;
     return styleObject;
 }

package/src/vectorStyle/styleList.js

@@ -18,7 +18,8 @@ let styleList,
     highlightFeaturesPolygonStyleId,
     highlightFeaturesLineStyleId,
     styleConf,
-    featureViaUrlLayers;
+    featureViaUrlLayers,
+    zoomToFeatureId;
 
 /**
  * Gathers the styleIds of the layers.
@@ -196,9 +197,10 @@ function parseStyles (data) {
         getStyleIdForHighlightFeaturesPolygon(),
         getStyleIdForHighlightFeaturesLine(),
         getStyleIdsFromTools(),
-        getFeatureViaURLStyles());
+        getFeatureViaURLStyles(),
+        zoomToFeatureId);
 
-    styleIds = styleIds.reduce((acc, val) => acc.concat(val), []);
+    styleIds = styleIds.reduce((acc, val) => acc.concat(val), []).filter(item => item);
     filteredData = dataWithDefaultValue.filter(styleObject => styleIds.includes(styleObject.styleId));
 
     return filteredData;
@@ -223,6 +225,7 @@ function initializeStyleList (styleGetters, Config, layers, tools, callback) {
     highlightFeaturesPointStyleId = styleGetters.highlightFeaturesPointStyleId;
     highlightFeaturesPolygonStyleId = styleGetters.highlightFeaturesPolygonStyleId;
     highlightFeaturesLineStyleId = styleGetters.highlightFeaturesLineStyleId;
+    zoomToFeatureId = styleGetters.zoomToFeatureId;
 
     styleConf = Config.styleConf;
     featureViaUrlLayers = Config.featureViaURL?.layers;

package/src/vectorStyle/styles/point/stylePoint.js

@@ -46,6 +46,10 @@ class PointStyle extends StyleClass {
      * @returns {ol/style} - The created style.
      */
     getStyle () {
+        if (this.nullStyle) {
+            return null;
+        }
+
         const isClustered = this.isClustered;
         let type = this.attributes.type.toLowerCase();
 

package/src/vectorStyle/styles/point/stylePointNominal.js

@@ -220,11 +220,14 @@ export function createNominalPointStyle (attributes, feature, defaultImageName)
     if (styleScalingShape === "CIRCLESEGMENTS") {
         svgPath = createNominalCircleSegments(workingFeature, attributes);
         style = createSVGStyle(svgPath, 5);
+        style.type = "CIRCLESEGMENTS";
+        style.scalingAttribute = workingFeature.get(attributes.scalingAttribute);
     }
 
     // create style from svg and image
     if (imageName !== defaultImageName) {
         imageStyle = createIconStyle(attributes, feature);
+        imageStyle.type = "imageStyle";
         style = [style, imageStyle];
     }
 

package/src/vectorStyle/styles/polygon/stylePolygon.js

@@ -36,6 +36,10 @@ class PolygonStyle extends StyleClass {
      * @returns {ol/style} - The created style.
      */
     getStyle () {
+        if (this.nullStyle) {
+            return null;
+        }
+
         const polygonFillColor = this.attributes.polygonFillColor,
             polygonFillHatch = this.attributes.polygonFillHatch;
 

package/src/vectorStyle/styles/style.js

@@ -14,6 +14,7 @@ class StyleClass {
         this.attributes = {};
         this.legendInfos = [];
         this.styleMultiGeomOnlyWithRule = false;
+        this.nullStyle = false;
     }
 
     /**
@@ -43,12 +44,16 @@ class StyleClass {
         // check if styles object is defined
         // if not, use defaults instead
         if (styles) {
+            this.nullStyle = false;
             Object.keys(styles).forEach(element => {
                 const value = styles[element];
 
                 this.attributes[element] = value;
             });
         }
+        else {
+            this.nullStyle = true;
+        }
     }
 
     /**

package/src/vectorStyle/styles/styleLine.js

@@ -34,6 +34,10 @@ class LineStringStyle extends StyleClass {
      * @returns {ol/style/Style} - The created style.
      */
     getStyle () {
+        if (this.nullStyle) {
+            return null;
+        }
+
         return new Style({
             stroke: new Stroke({
                 lineCap: this.attributes.lineStrokeCap,

package/src/vectorStyle/styles/styleText.js

@@ -36,6 +36,10 @@ class TextStyle extends StyleClass {
     * @returns {ol/style} - The created style.
     */
     getStyle () {
+        if (this.nullStyle) {
+            return null;
+        }
+
         const isClustered = this.isClustered,
             feature = this.feature,
             labelField = this.attributes.labelField;

package/test/vectorStyle/createStyle.test.js

@@ -242,23 +242,69 @@ describe("createStyle", () => {
 });
 
 describe("getSimpleGeometryStyle", function () {
-    it("should return ol point style", function () {
-        expect(createStyle.getSimpleGeometryStyle("Point", jsonObjects[0], rules[0], wfsImgPathFromConfig)).toBeInstanceOf(PointStyle);
-        expect(typeof createStyle.getSimpleGeometryStyle("Point", jsonObjects[0], rules[0], wfsImgPathFromConfig).getStyle().getImage().getStroke().getColor()).toBe("object");
-        expect(createStyle.getSimpleGeometryStyle("Point", jsonObjects[0], rules[0], wfsImgPathFromConfig).getStyle().getImage().getStroke().getColor()).toEqual([0, 0, 0, 1]);
+    let isClustered = false;
+
+    it("should return ol point style - not clustered", function () {
+        const styleObject = createStyle.getSimpleGeometryStyle("Point", jsonObjects[0], rules[0], isClustered, wfsImgPathFromConfig);
+
+        expect(styleObject).toBeInstanceOf(PointStyle);
+        expect(typeof styleObject.getStyle().getImage().getStroke().getColor()).toBe("object");
+        expect(styleObject.getStyle().getImage().getStroke().getColor()).toEqual([255, 0, 0, 1]);
+        expect(styleObject.getLegendInfos()[0].styleObject.circleStrokeColor).toEqual([255, 0, 0, 1]);
+    });
+    it("should return ol linestring style - not clustered", function () {
+        const styleObject = createStyle.getSimpleGeometryStyle("LineString", jsonObjects[1], rules[1], isClustered, wfsImgPathFromConfig);
+
+        expect(styleObject).toBeInstanceOf(LineStringStyle);
+        expect(typeof styleObject.getStyle().getStroke().getColor()).toBe("object");
+        expect(styleObject.getStyle().getStroke().getColor()).toEqual([255, 0, 0, 1]);
+        expect(styleObject.getLegendInfos()[0].styleObject.lineStrokeColor).toEqual([255, 0, 0, 1]);
+    });
+    it("should return ol polygon style - not clustered", function () {
+        const styleObject = createStyle.getSimpleGeometryStyle("Polygon", jsonObjects[2], rules[2], isClustered, wfsImgPathFromConfig);
+
+        expect(styleObject).toBeInstanceOf(PolygonStyle);
+        expect(typeof styleObject.getStyle().getStroke().getColor()).toBe("object");
+        expect(styleObject.getStyle().getStroke().getColor()).toEqual([255, 0, 0, 255]);
+        expect(styleObject.getStyle().getFill().getColor()).toEqual([10, 200, 100, 0.5]);
+        expect(styleObject.getLegendInfos()[0].styleObject.polygonStrokeColor).toEqual([255, 0, 0, 255]);
+        expect(styleObject.getLegendInfos()[0].styleObject.polygonFillColor).toEqual([10, 200, 100, 0.5]);
     });
-    it("should return ol linestring style", function () {
-        expect(createStyle.getSimpleGeometryStyle("LineString", jsonObjects[1], rules[1], wfsImgPathFromConfig)).toBeInstanceOf(LineStringStyle);
-        expect(typeof createStyle.getSimpleGeometryStyle("LineString", jsonObjects[1], rules[1], wfsImgPathFromConfig).getStyle().getStroke().getColor()).toBe("object");
-        expect(createStyle.getSimpleGeometryStyle("LineString", jsonObjects[1], rules[1], wfsImgPathFromConfig).getStyle().getStroke().getColor()).toEqual([255, 0, 0, 1]);
+    it("should return ol default style - not clustered", function () {
+        expect(createStyle.getSimpleGeometryStyle("not exist", jsonObjects[0], null, isClustered, wfsImgPathFromConfig)).toBeInstanceOf(Style);
     });
-    it("should return ol polygon style", function () {
-        expect(createStyle.getSimpleGeometryStyle("Polygon", jsonObjects[2], rules[2], wfsImgPathFromConfig)).toBeInstanceOf(PolygonStyle);
-        expect(typeof createStyle.getSimpleGeometryStyle("Polygon", jsonObjects[2], rules[2], wfsImgPathFromConfig).getStyle().getStroke().getColor()).toBe("object");
-        expect(createStyle.getSimpleGeometryStyle("Polygon", jsonObjects[2], rules[2], wfsImgPathFromConfig).getStyle().getStroke().getColor()).toEqual([255, 0, 0, 255]);
+
+    it("should return ol point style - clustered", function () {
+        isClustered = true;
+        const styleObject = createStyle.getSimpleGeometryStyle("Point", jsonObjects[0], rules[0], isClustered, wfsImgPathFromConfig);
+
+        expect(styleObject).toBeInstanceOf(PointStyle);
+        expect(typeof styleObject).toBe("object");
+        expect(styleObject.getStyle().getImage().getStroke().getColor()).toEqual([0, 0, 0, 1]);
+        expect(styleObject.getLegendInfos()[0].styleObject.circleStrokeColor).toEqual([255, 0, 0, 1]);
+    });
+    it("should return ol linestring style - clustered", function () {
+        isClustered = true;
+        const styleObject = createStyle.getSimpleGeometryStyle("LineString", jsonObjects[1], rules[1], isClustered, wfsImgPathFromConfig);
+
+        expect(styleObject).toBeInstanceOf(LineStringStyle);
+        expect(typeof styleObject.getStyle().getStroke().getColor()).toBe("object");
+        expect(styleObject.getStyle().getStroke().getColor()).toEqual([255, 0, 0, 1]);
+        expect(styleObject.getLegendInfos()[0].styleObject.lineStrokeColor).toEqual([255, 0, 0, 1]);
+    });
+    it("should return ol polygon style - clustered", function () {
+        isClustered = true;
+        const styleObject = createStyle.getSimpleGeometryStyle("Polygon", jsonObjects[2], rules[2], isClustered, wfsImgPathFromConfig);
+
+        expect(styleObject).toBeInstanceOf(PolygonStyle);
+        expect(typeof styleObject.getStyle().getStroke().getColor()).toBe("object");
+        expect(styleObject.getStyle().getStroke().getColor()).toEqual([255, 0, 0, 255]);
+        expect(styleObject.getLegendInfos()[0].styleObject.polygonStrokeColor).toEqual([255, 0, 0, 255]);
+        expect(styleObject.getLegendInfos()[0].styleObject.polygonFillColor).toEqual([10, 200, 100, 0.5]);
     });
-    it("should return ol default style", function () {
-        expect(createStyle.getSimpleGeometryStyle("not exist", jsonObjects[0], null, wfsImgPathFromConfig)).toBeInstanceOf(Style);
+    it("should return ol default style - clustered", function () {
+        isClustered = true;
+        expect(createStyle.getSimpleGeometryStyle("not exist", jsonObjects[0], null, isClustered, wfsImgPathFromConfig)).toBeInstanceOf(Style);
     });
 });
 
@@ -328,8 +374,7 @@ describe("getGeometryStyle", function () {
         expect(Array.isArray(createStyle.getGeometryStyle(jsonObjects[0], rules, wfsImgPathFromConfig).getStyle().getImage().getStroke().getColor())).toBe(true);
         expect(createStyle.getGeometryStyle(jsonObjects[0], rules, wfsImgPathFromConfig).getStyle().getImage().getStroke().getColor()).toEqual([0, 0, 0, 1]);
     });
-    it("should return default style if no rule is found", function () {
-        expect(Array.isArray(createStyle.getGeometryStyle(jsonObjects[0], [], wfsImgPathFromConfig).getStyle().getImage().getStroke().getColor())).toBe(true);
-        expect(createStyle.getGeometryStyle(jsonObjects[0], [], wfsImgPathFromConfig).getStyle().getImage().getStroke().getColor()).toEqual([0, 0, 0, 1]);
+    it("should return null as style if no rule is found", function () {
+        expect(createStyle.getGeometryStyle(jsonObjects[0], [], wfsImgPathFromConfig).getStyle()).toBe(null);
     });
 });