761-fix: Filter out private videos on mentorship page (#762)

* fix: 761 - to filter out private videos * refactor: 761 - add test case for private videos
rolling-scopes · Jan 26, 2025 · e091d61 · e091d61
1 parent a52c93e
commit e091d61
Show file tree

Hide file tree

Showing 3 changed files with 49 additions and 2 deletions.
diff --git a/src/shared/ui/video-playlist-with-player/constants.ts b/src/shared/ui/video-playlist-with-player/constants.ts
@@ -0,0 +1,4 @@
+export const videoPrivacyStatus = {
+  private: 'private',
+  public: 'public',
+} as const;
diff --git a/src/shared/ui/video-playlist-with-player/video-playlist-with-player.test.tsx b/src/shared/ui/video-playlist-with-player/video-playlist-with-player.test.tsx
@@ -41,6 +41,7 @@ describe('VideoPlaylistWithPlayer Component', () => {
             title: video.title,
             thumbnails: { medium: { url: video.thumbnail } },
           },
+          status: { privacyStatus: 'public' },
         })),
       }),
     } as Response);
@@ -65,6 +66,7 @@ describe('VideoPlaylistWithPlayer Component', () => {
             title: video.title,
             thumbnails: { medium: { url: video.thumbnail } },
           },
+          status: { privacyStatus: 'public' },
         })),
       }),
     } as Response);
@@ -112,6 +114,7 @@ describe('VideoPlaylistWithPlayer Component', () => {
             title: video.title,
             thumbnails: { medium: { url: video.thumbnail } },
           },
+          status: { privacyStatus: 'public' },
         })),
       }),
     } as Response);
@@ -141,4 +144,38 @@ describe('VideoPlaylistWithPlayer Component', () => {
       expect(playlistContainer).toHaveStyle({ maxHeight: '300px' });
     });
   });
+
+  it('should filter out private videos', async () => {
+    const publicVideos = MOCKED_VIDEOS.map((video) => ({
+      snippet: {
+        resourceId: { videoId: video.id },
+        title: video.title,
+        thumbnails: { medium: { url: video.thumbnail } },
+      },
+      status: { privacyStatus: 'public' },
+    }));
+    const privateVideos = MOCKED_VIDEOS.map((video) => ({
+      snippet: {
+        resourceId: { videoId: video.id },
+        title: video.title,
+        thumbnails: { medium: { url: video.thumbnail } },
+      },
+      status: { privacyStatus: 'private' },
+    }));
+
+    fetchMock.mockResolvedValue({
+      ok: true,
+      json: async () => ({ items: [...publicVideos, ...privateVideos] }),
+    } as Response);
+
+    render(<VideoPlaylistWithPlayer playlistId="testPlaylist" apiKey="testApiKey" />);
+
+    await waitFor(() => {
+      expect(screen.getByTestId('videos-container')).toBeVisible();
+    });
+
+    const videosContainer = screen.getByTestId('videos-container');
+
+    expect(videosContainer.children).toHaveLength(3);
+  });
 });
diff --git a/src/shared/ui/video-playlist-with-player/video-playlist-with-player.tsx b/src/shared/ui/video-playlist-with-player/video-playlist-with-player.tsx
@@ -3,6 +3,7 @@
 import React, { useEffect, useRef, useState } from 'react';
 import classNames from 'classnames/bind';
 
+import { videoPrivacyStatus } from './constants';
 import { Playlist } from './playlist';
 import type { Video } from '@/shared/types';
 import { VideoPlayer } from '@/shared/ui/video-player';
@@ -29,7 +30,7 @@ export const VideoPlaylistWithPlayer = ({ playlistId, apiKey }: VideoPlaylistWit
       setIsLoading(true);
 
       try {
-        const apiUrl = `https://www.googleapis.com/youtube/v3/playlistItems?part=snippet&maxResults=50&playlistId=${playlistId}&key=${apiKey}`;
+        const apiUrl = `https://www.googleapis.com/youtube/v3/playlistItems?part=snippet,status&maxResults=50&playlistId=${playlistId}&key=${apiKey}`;
         const response = await fetch(apiUrl);
 
         if (!response.ok) {
@@ -39,7 +40,12 @@ export const VideoPlaylistWithPlayer = ({ playlistId, apiKey }: VideoPlaylistWit
         }
 
         const data = await response.json();
-        const videoItems: Video[] = data.items.map(
+        const publicVideos = data.items.filter(
+          (item: GoogleApiYouTubePlaylistItemResource) =>
+            item.status.privacyStatus === videoPrivacyStatus.public,
+        );
+
+        const videoItems: Video[] = publicVideos.map(
           (item: GoogleApiYouTubePlaylistItemResource) => ({
             id: item.snippet.resourceId.videoId,
             title: item.snippet.title,