ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Add new tests for FeatureOverlay

Browse Source 
In the process of creating these tests, a bug with setting this._style
was also fixed.
This commit is contained in:
Andreas Hocevar
2014-06-27 11:46:47 +02:00
parent 1551e6e7ff
commit 2171c92b5c
2 changed files with 35 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/ol/featureoverlay.js
View File

@@ -65,7 +65,7 @@ ol.FeatureOverlay = function(opt_options) {
* @private
* @type {ol.style.Style|Array.<ol.style.Style>|ol.feature.StyleFunction}
*/
this.style_ = null;
this.style_ = goog.isDef(options.style) ? options.style : null;
/**
* @private

34
test/spec/ol/featureoverlay.test.js Normal file
View File

@@ -0,0 +1,34 @@
goog.provide('ol.test.FeatureOverlay');
describe('ol.FeatureOverlay', function() {
describe('constructor', function() {
it('creates an new feature overlay', function() {
var featureOverlay = new ol.FeatureOverlay();
expect(featureOverlay).to.be.a(ol.FeatureOverlay);
});
it('takes features', function() {
var featureOverlay = new ol.FeatureOverlay({
features: [new ol.Feature(new ol.geom.Point([0, 0]))]
});
expect(featureOverlay.getFeatures().getLength()).to.be(1);
});
it('takes a style', function() {
var style = [new ol.style.Style()];
var featureOverlay = new ol.FeatureOverlay({
style: [new ol.style.Style()]
});
expect(featureOverlay.getStyle()).to.eql(style);
expect(featureOverlay.getStyleFunction()()).to.eql(style);
});
});
});
goog.require('ol.Feature');
goog.require('ol.FeatureOverlay');
goog.require('ol.geom.Point');
goog.require('ol.style.Style');