fix: made changes to cater updated video detail api response (#1126)

Co-authored-by: Maham Akif <[email protected]>
openedx · Jul 22, 2024 · 7947a71 · 7947a71
1 parent 7620f17
commit 7947a71
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 33 deletions.
diff --git a/src/components/microlearning/data/utils.js b/src/components/microlearning/data/utils.js
@@ -10,7 +10,7 @@ export const formatDuration = (durationInSeconds) => {
   return `${minutes}:${seconds}`;
 };
 
-export const formatSkills = (skills) => skills?.['list-item'].map(skill => ({
+export const formatSkills = (skills) => skills?.map(skill => ({
   name: skill?.name,
   description: skill?.description,
   category: skill?.category?.name,
@@ -20,7 +20,7 @@ export const formatSkills = (skills) => skills?.['list-item'].map(skill => ({
 export const transformVideoData = (data) => ({
   videoUrl: data?.json_metadata?.download_link,
   courseTitle: data?.parent_content_metadata?.title,
-  videoSummary: data?.summary_transcripts?.['list-item'],
+  videoSummary: data?.summary_transcripts?.[0],
   transcriptUrls: data?.json_metadata?.transcript_urls,
   videoSkills: formatSkills(data?.skills),
   videoDuration: formatDuration(data?.json_metadata?.duration),

diff --git a/src/components/microlearning/data/utils.test.js b/src/components/microlearning/data/utils.test.js
@@ -1,22 +1,20 @@
 import { formatDuration, formatSkills, transformVideoData } from './utils';
 
 describe('Microlearning utils tests', () => {
-  const mockSkills = {
-    'list-item': [
-      {
-        name: 'Skill 1',
-        description: 'Description 1',
-        category: { name: 'Category 1' },
-        subcategory: { name: 'Subcategory 1' },
-      },
-      {
-        name: 'Skill 2',
-        description: 'Description 2',
-        category: { name: 'Category 2' },
-        subcategory: { name: 'Subcategory 2' },
-      },
-    ],
-  };
+  const mockSkills = [
+    {
+      name: 'Skill 1',
+      description: 'Description 1',
+      category: { name: 'Category 1' },
+      subcategory: { name: 'Subcategory 1' },
+    },
+    {
+      name: 'Skill 2',
+      description: 'Description 2',
+      category: { name: 'Category 2' },
+      subcategory: { name: 'Subcategory 2' },
+    },
+  ];
 
   const mockData = {
     json_metadata: {
@@ -27,19 +25,17 @@ describe('Microlearning utils tests', () => {
     parent_content_metadata: {
       title: 'Course Title',
     },
-    summary_transcripts: {
-      'list-item': ['Transcript 1', 'Transcript 2'],
-    },
-    skills: {
-      'list-item': [
-        {
-          name: 'Skill 1',
-          description: 'Description 1',
-          category: { name: 'Category 1' },
-          subcategory: { name: 'Subcategory 1' },
-        },
-      ],
-    },
+    summary_transcripts: [
+      'Transcript 1', 'Transcript 2',
+    ],
+    skills: [
+      {
+        name: 'Skill 1',
+        description: 'Description 1',
+        category: { name: 'Category 1' },
+        subcategory: { name: 'Subcategory 1' },
+      },
+    ],
   };
 
   it('should format 60 seconds correctly', () => {
@@ -64,14 +60,14 @@ describe('Microlearning utils tests', () => {
   });
 
   it('should handle empty skills list', () => {
-    expect(formatSkills({ 'list-item': [] })).toEqual([]);
+    expect(formatSkills([])).toEqual([]);
   });
 
   it('should transform video data correctly', () => {
     expect(transformVideoData(mockData)).toEqual({
       videoUrl: 'http://example.com/video.mp4',
       courseTitle: 'Course Title',
-      videoSummary: ['Transcript 1', 'Transcript 2'],
+      videoSummary: 'Transcript 1',
       transcriptUrls: ['http://example.com/transcript1.vtt'],
       videoSkills: [
         {