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

Document and test ol.getUid sequence behavior

Browse Source
This commit is contained in:
Thomas Moelhave
2016-09-02 09:09:24 +02:00
parent 617dca34f5
commit eddfc04ccc
2 changed files with 25 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/ol/index.js
View File

@@ -270,8 +270,8 @@ ol.nullFunction = function() {};
/** /**
* Gets a unique ID for an object. This mutates the object so that further calls * Gets a unique ID for an object. This mutates the object so that further calls
* with the same object as a parameter returns the same value. Adapted from * with the same object as a parameter returns the same value. Unique IDs are generated
* goog.getUid. * as a strictly increasing sequence. Adapted from goog.getUid.
* *
* @param {Object} obj The object to get the unique ID for. * @param {Object} obj The object to get the unique ID for.
* @return {number} The unique ID for the object. * @return {number} The unique ID for the object.

23
test/spec/ol/index.test.js Normal file
View File

@@ -0,0 +1,23 @@
goog.provide('ol.test');
goog.require('ol');
describe('getUid()', function() {
it('is constant once generated', function() {
var a = {};
expect(ol.getUid(a)).to.be(ol.getUid(a));
});
it('generates a strictly increasing sequence', function() {
var a = {} , b = {}, c = {};
ol.getUid(a);
ol.getUid(c);
ol.getUid(b);
//uid order should be a < c < b
expect(ol.getUid(a)).to.be.lessThan(ol.getUid(c));
expect(ol.getUid(c)).to.be.lessThan(ol.getUid(b));
expect(ol.getUid(a)).to.be.lessThan(ol.getUid(b));
});
});