Plots.resize: reject old promises

plotly · Nov 29, 2019 · deb32ff · deb32ff
1 parent 5ef8a61
commit deb32ff
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/src/plots/plots.js b/src/plots/plots.js
@@ -81,6 +81,9 @@ plots.resize = function(gd) {
             reject(new Error('Resize must be passed a displayed plot div element.'));
         }
 
+        if(gd._rejectResize) gd._rejectResize();
+        gd._rejectResize = reject;
+
         if(gd._redrawTimer) clearTimeout(gd._redrawTimer);
 
         gd._redrawTimer = setTimeout(function() {

diff --git a/test/jasmine/tests/plots_test.js b/test/jasmine/tests/plots_test.js
@@ -412,6 +412,27 @@ describe('Test Plots', function() {
                     .then(done);
             });
         });
+
+        describe('returns Promises', function() {
+            afterEach(destroyGraphDiv);
+
+            it('should reject or resolve them all', function(done) {
+                gd = createGraphDiv();
+                var p = [];
+                Plotly.newPlot(gd, [{y: [5, 2, 5]}])
+                    .then(function() {
+                        // First call should get rejected
+                        p.push(Plotly.Plots.resize(gd).catch(function() {
+                            return Promise.resolve(true);
+                        }));
+                        // because we call the function again within 100ms
+                        p.push(Plotly.Plots.resize(gd));
+                        return Promise.all(p);
+                    })
+                    .catch(failTest)
+                    .then(done);
+            });
+        });
     });
 
     describe('Plots.purge', function() {