Fix: Reset retry count after successful preview (#488)

box · Nov 21, 2017 · a8f66ed · a8f66ed
1 parent eee1525
commit a8f66ed
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/src/lib/Preview.js b/src/lib/Preview.js
@@ -860,6 +860,10 @@ class Preview extends EventEmitter {
         // Once the viewer instance has been created, emit it so that clients can attach their events.
         // Viewer object will still be sent along the load event also.
         this.emit('viewer', this.viewer);
+
+        // Reset retry count after successful load so we don't go into the retry short circuit when the same file
+        // previewed again
+        this.retryCount = 0;
     }
 
     /**

diff --git a/src/lib/__tests__/Preview-test.js b/src/lib/__tests__/Preview-test.js
@@ -1349,6 +1349,11 @@ describe('lib/Preview', () => {
             preview.loadViewer();
             expect(stubs.emit).to.be.calledWith('viewer', stubs.viewer);
         });
+
+        it('should reset retry count', () => {
+            preview.loadViewer();
+            expect(preview.retryCount).to.equal(0);
+        });
     });
 
     describe('attachViewerListeners()', () => {