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

Merge pull request #5835 from thomasmoelhave/getuid-sequence

Browse Source 
document and test ol.getUid sequence behavior
This commit is contained in:
Andreas Hocevar
2016-09-02 10:34:22 +02:00
committed by GitHub
parent 785639338b eddfc04ccc
commit e962b6f801
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
* with the same object as a parameter returns the same value. Adapted from
* goog.getUid.
* with the same object as a parameter returns the same value. Unique IDs are generated
* as a strictly increasing sequence. Adapted from goog.getUid.
*
* @param {Object} obj The object to get the unique ID for.
* @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));
});
});