From b2d2f44f43b9605ffab244693cf0c5ed97e6829f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Maximilian=20Kr=C3=B6g?= <maxi_kroeg@web.de>
Date: Sun, 31 Jul 2022 18:25:07 +0200
Subject: [PATCH 1/2] No need to wait for postrender in Map test

---
 test/browser/spec/ol/Map.test.js | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/test/browser/spec/ol/Map.test.js b/test/browser/spec/ol/Map.test.js
index a6ca16fcd0..5bf76232f5 100644
--- a/test/browser/spec/ol/Map.test.js
+++ b/test/browser/spec/ol/Map.test.js
@@ -1070,29 +1070,24 @@ describe('ol/Map', function () {
       expect(spy.callCount).to.be(0);
     });
 
-    it('calls renderFrame_ and results in an postrender event', function (done) {
+    it('calls renderFrame_ and results in a postrender event', function (done) {
       const spy = sinon.spy(map, 'renderFrame_');
       map.render();
       map.once('postrender', function (event) {
         expect(event).to.be.a(MapEvent);
         expect(typeof spy.firstCall.args[0]).to.be('number');
         spy.restore();
-        const frameState = event.frameState;
-        expect(frameState).not.to.be(null);
+        expect(event.frameState).not.to.be(null);
         done();
       });
     });
 
-    it('uses the same render frame for subsequent calls', function (done) {
+    it('uses the same render frame for subsequent calls', function () {
       map.render();
       const id1 = map.animationDelayKey_;
-      let id2 = null;
-      map.once('postrender', function () {
-        expect(id2).to.be(id1);
-        done();
-      });
       map.render();
-      id2 = map.animationDelayKey_;
+      const id2 = map.animationDelayKey_;
+      expect(id1).to.be(id2);
     });
 
     it('creates a new render frame after renderSync()', function (done) {

From e3987dfdbf42d8670919fe97fb2f39a0caa2f9ed Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Maximilian=20Kr=C3=B6g?= <maxi_kroeg@web.de>
Date: Sun, 31 Jul 2022 18:28:09 +0200
Subject: [PATCH 2/2] Fix Map render test

The value set to id2 was never read, postrender fires in renderSync
---
 test/browser/spec/ol/Map.test.js | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/test/browser/spec/ol/Map.test.js b/test/browser/spec/ol/Map.test.js
index 5bf76232f5..356f3bf6f7 100644
--- a/test/browser/spec/ol/Map.test.js
+++ b/test/browser/spec/ol/Map.test.js
@@ -1090,16 +1090,12 @@ describe('ol/Map', function () {
       expect(id1).to.be(id2);
     });
 
-    it('creates a new render frame after renderSync()', function (done) {
-      let id2 = null;
+    it('creates a new render frame after renderSync()', function () {
       map.render();
-      const id1 = map.animationDelayKey_;
-      map.once('postrender', function () {
-        expect(id2).to.not.be(id1);
-        done();
-      });
+      expect(map.animationDelayKey_).to.not.be(undefined);
+
       map.renderSync();
-      id2 = map.animationDelayKey_;
+      expect(map.animationDelayKey_).to.be(undefined);
     });
 
     it('results in an postrender event (for zero height map)', function (done) {