Merge pull request #12259 from tschaub/clearer-failure
Make it clearer what fails in webgl point layer test
This commit is contained in:
@@ -335,10 +335,6 @@ describe('ol.renderer.webgl.PointsLayer', function () {
|
|||||||
},
|
},
|
||||||
baseFrameState
|
baseFrameState
|
||||||
);
|
);
|
||||||
let found;
|
|
||||||
const cb = function (feature) {
|
|
||||||
found = feature;
|
|
||||||
};
|
|
||||||
|
|
||||||
renderer.prepareFrame(frameState);
|
renderer.prepareFrame(frameState);
|
||||||
renderer.worker_.addEventListener('message', function () {
|
renderer.worker_.addEventListener('message', function () {
|
||||||
@@ -349,9 +345,23 @@ describe('ol.renderer.webgl.PointsLayer', function () {
|
|||||||
renderer.renderFrame(frameState);
|
renderer.renderFrame(frameState);
|
||||||
|
|
||||||
function checkHit(x, y, expected) {
|
function checkHit(x, y, expected) {
|
||||||
found = null;
|
let called = false;
|
||||||
renderer.forEachFeatureAtCoordinate([x, y], frameState, 0, cb, null);
|
renderer.forEachFeatureAtCoordinate(
|
||||||
expect(found).to.be(expected);
|
[x, y],
|
||||||
|
frameState,
|
||||||
|
0,
|
||||||
|
function (feature) {
|
||||||
|
expect(feature).to.be(expected);
|
||||||
|
called = true;
|
||||||
|
},
|
||||||
|
null
|
||||||
|
);
|
||||||
|
|
||||||
|
if (expected) {
|
||||||
|
expect(called).to.be(true);
|
||||||
|
} else {
|
||||||
|
expect(called).to.be(false);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
checkHit(0, 0, feature);
|
checkHit(0, 0, feature);
|
||||||
|
|||||||
Reference in New Issue
Block a user