Reject promise if the video throws an error (#9)

Reject promise if the video throws an error Co-authored-by: Ezekiel Victor <[email protected]>
evictor · Apr 2, 2021 · 69d3db9 · 69d3db9
1 parent cabf157
commit 69d3db9
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 3 deletions.
diff --git a/src/getBlobDuration.js b/src/getBlobDuration.js
@@ -6,7 +6,7 @@
 export default async function getBlobDuration(blob) {
   const tempVideoEl = document.createElement('video')
 
-  const durationP = new Promise(resolve =>
+  const durationP = new Promise((resolve, reject) => {
     tempVideoEl.addEventListener('loadedmetadata', () => {
       // Chrome bug: https://bugs.chromium.org/p/chromium/issues/detail?id=642012
       if(tempVideoEl.duration === Infinity) {
@@ -20,8 +20,9 @@ export default async function getBlobDuration(blob) {
       // Normal behavior
       else
         resolve(tempVideoEl.duration)
-    }),
-  )
+    })
+    tempVideoEl.onerror = (event) => reject(event.target.error)
+  })
 
   tempVideoEl.src = typeof blob === 'string' || blob instanceof String
     ? blob

diff --git a/test/getBlobDuration.test.js b/test/getBlobDuration.test.js
@@ -1,6 +1,11 @@
 import getBlobDuration from '../src/getBlobDuration'
 
 let dummyVideoEl, mockBlob
+const dummyErrorEventObject = {
+  target: {
+    error: 'Dummy error message'
+  }
+}
 
 beforeEach(() => {
   dummyVideoEl = jest.fn()
@@ -52,3 +57,12 @@ it('should execute Chrome bugfix duration retrieval as needed', async () => {
     resolve()
   })
 })
+
+it('should reject with the error object if an error occurs', async () => {
+  dummyVideoEl.duration = Infinity
+
+  // noinspection ES6MissingAwait
+  const durationP = getBlobDuration(mockBlob)
+  dummyVideoEl.onerror(dummyErrorEventObject)
+  await expect(durationP).rejects.toMatch(dummyErrorEventObject.target.error)
+})