diff --git a/src/components/microlearning/VideoDetailPage.jsx b/src/components/microlearning/VideoDetailPage.jsx index 81a28152c8..15ed7610d2 100644 --- a/src/components/microlearning/VideoDetailPage.jsx +++ b/src/components/microlearning/VideoDetailPage.jsx @@ -87,19 +87,25 @@ const VideoDetailPage = () => {
-
-

- {videoData?.courseTitle} -

- - {videoData?.videoDuration && `(${videoData?.videoDuration} minutes)`} - -
+ { videoData?.courseTitle && ( +
+
+

+ {videoData?.courseTitle} +

+
+ + {videoData?.videoDuration && `(${videoData?.videoDuration} minutes)`} + +
+ )}

{videoData?.videoSummary}

+ {/* Skills that we are currently retrieving for videos are inaccurate, so we are temporarily + hiding this section. */} {videoData?.videoSkills?.length > 0 && ( -
+

{ expect(screen.getByTestId('video-title')).toHaveTextContent('Test Video'); expect(screen.getByText('(10:4 minutes)')).toBeInTheDocument(); expect(screen.getByText('This is a test video summary.')).toBeInTheDocument(); - expect(screen.getByText('Skill 1')).toBeInTheDocument(); - expect(screen.getByText('Skill 2')).toBeInTheDocument(); + // Skills that we are currently retrieving for videos are inaccurate, so we are + // temporarily hiding this section. + // expect(screen.getByText('Skill 1')).toBeInTheDocument(); + // expect(screen.getByText('Skill 2')).toBeInTheDocument(); expect(container.querySelector('.video-player-wrapper')).toBeTruthy(); }); it('renders a video detail page when course level type is Intermediate', () => { diff --git a/src/components/video/VideoPlayer.jsx b/src/components/video/VideoPlayer.jsx index 5c1dc92348..eaf62f299f 100644 --- a/src/components/video/VideoPlayer.jsx +++ b/src/components/video/VideoPlayer.jsx @@ -4,7 +4,7 @@ import VideoJS from './VideoJS'; const hlsExtension = '.m3u8'; const defaultOptions = { - autoplay: true, + autoplay: false, controls: true, responsive: true, fluid: true,