fix: find my video will throw error when there is no video (#46)

video-trading · May 22, 2023 · cfac1ee · cfac1ee · vercel · May 22, 2023
1 parent 3f1dc5b
commit cfac1ee
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/video/video.service.spec.ts b/src/video/video.service.spec.ts
@@ -454,6 +454,14 @@ describe('VideoService', () => {
     expect(videos.metadata.total).toBe(1);
   });
 
+  it('Should be able to find my videos when there is no video', async () => {
+    const videos = await service.findMyVideos(userId, 1, 2);
+    expect(videos.items).toHaveLength(0);
+    expect(videos.metadata.total).toBe(0);
+    expect(videos.metadata.page).toBe(1);
+    expect(videos.metadata.totalPages).toBe(0);
+  });
+
   it('Should be able to find my videos', async () => {
     const video: CreateVideoDto = {
       title: 'Test Video',

diff --git a/src/video/video.service.ts b/src/video/video.service.ts
@@ -530,7 +530,7 @@ export class VideoService {
       metadata: getPaginationMetaData(
         page,
         per,
-        (totalResult[0] as any)?.total,
+        (totalResult[0] as any)?.total ?? 0,
       ),
     };
   }