Merge pull request #13835 from ahocevar/source-state

Recover from incorrect rebase after removal of string enums

src/ol/renderer/Layer.js

@@ -195,7 +195,7 @@ class LayerRenderer extends Observable {
    */
   renderIfReadyAndVisible() {
     const layer = this.getLayer();
-    if (layer && layer.getVisible() && 'ready') {
+    if (layer && layer.getVisible() && layer.getSourceState() === 'ready') {
       layer.changed();
     }
   }

test/browser/spec/ol/renderer/layer.test.js

@@ -7,15 +7,32 @@ import View from '../../../../../src/ol/View.js';
 import XYZ from '../../../../../src/ol/source/XYZ.js';
 import {fromKey} from '../../../../../src/ol/tilecoord.js';
 
-describe('ol.renderer.Layer', function () {
+describe('ol/renderer/Layer', function () {
-  let renderer;
+  let layer, renderer;
   const eventType = 'change';
 
   beforeEach(function () {
-    const layer = new Layer({});
+    layer = new Layer({});
     renderer = new LayerRenderer(layer);
   });
 
+  describe('#renderIfReadyAndVisible', function () {
+    it('updates revision when data is ready and layer is visible', function () {
+      layer.setVisible(true);
+      let state;
+      layer.getSourceState = function () {
+        return state;
+      };
+      const revision = layer.getRevision();
+      state = 'foo';
+      renderer.renderIfReadyAndVisible();
+      expect(layer.getRevision()).to.be(revision);
+      state = 'ready';
+      renderer.renderIfReadyAndVisible();
+      expect(layer.getRevision()).to.be(revision + 1);
+    });
+  });
+
   describe('#loadImage', function () {
     let image;
     let imageLoadFunction;