refactor: Generate the migration summary in the front using migration blocks data

Author: ChrisChV

src/data/api.mocks.ts

@@ -48,14 +48,6 @@ mockGetMigrationStatus.migrationStatusData = {
         key: 'coll',
         title: 'Test Collection',
       },
-      migrationSummary: {
-        totalBlocks: 6,
-        sections: 1,
-        subsections: 1,
-        units: 1,
-        components: 3,
-        unsupported: 0,
-      },
     },
   ],
 } as api.MigrateTaskStatusData;
@@ -82,14 +74,6 @@ mockGetMigrationStatus.migrationStatusFailedData = {
       forwardSourceToTarget: true,
       isFailed: true,
       targetCollection: null,
-      migrationSummary: {
-        totalBlocks: 0,
-        sections: 0,
-        subsections: 0,
-        units: 0,
-        components: 0,
-        unsupported: 0,
-      },
     },
   ],
 } as api.MigrateTaskStatusData;
@@ -116,14 +100,6 @@ mockGetMigrationStatus.migrationStatusFailedMultipleData = {
       forwardSourceToTarget: true,
       isFailed: true,
       targetCollection: null,
-      migrationSummary: {
-        totalBlocks: 0,
-        sections: 0,
-        subsections: 0,
-        units: 0,
-        components: 0,
-        unsupported: 0,
-      },
     },
     {
       id: 2,
@@ -136,14 +112,6 @@ mockGetMigrationStatus.migrationStatusFailedMultipleData = {
       forwardSourceToTarget: true,
       isFailed: true,
       targetCollection: null,
-      migrationSummary: {
-        totalBlocks: 0,
-        sections: 0,
-        subsections: 0,
-        units: 0,
-        components: 0,
-        unsupported: 0,
-      },
     },
   ],
 } as api.MigrateTaskStatusData;
@@ -170,14 +138,6 @@ mockGetMigrationStatus.migrationStatusFailedOneLibraryData = {
       forwardSourceToTarget: true,
       isFailed: true,
       targetCollection: null,
-      migrationSummary: {
-        totalBlocks: 0,
-        sections: 0,
-        subsections: 0,
-        units: 0,
-        components: 0,
-        unsupported: 0,
-      },
     },
     {
       id: 2,
@@ -190,14 +150,6 @@ mockGetMigrationStatus.migrationStatusFailedOneLibraryData = {
       forwardSourceToTarget: true,
       isFailed: false,
       targetCollection: null,
-      migrationSummary: {
-        totalBlocks: 0,
-        sections: 0,
-        subsections: 0,
-        units: 0,
-        components: 0,
-        unsupported: 0,
-      },
     },
   ],
 } as api.MigrateTaskStatusData;
@@ -225,14 +177,6 @@ mockGetMigrationStatus.migrationStatusInProgressData = {
       forwardSourceToTarget: true,
       isFailed: false,
       targetCollection: null,
-      migrationSummary: {
-        totalBlocks: 0,
-        sections: 0,
-        subsections: 0,
-        units: 0,
-        components: 0,
-        unsupported: 0,
-      },
     },
   ],
 } as api.MigrateTaskStatusData;
@@ -262,14 +206,6 @@ mockGetMigrationStatus.migrationStatusPartialData = {
         key: 'coll',
         title: 'Test Collection',
       },
-      migrationSummary: {
-        totalBlocks: 7,
-        sections: 1,
-        subsections: 1,
-        units: 1,
-        components: 3,
-        unsupported: 1,
-      },
     },
   ],
 } as api.MigrateTaskStatusData;

src/data/api.ts

@@ -97,14 +97,6 @@ export interface MigrateParameters {
     key: string;
     title: string;
   } | null;
-  migrationSummary: {
-    totalBlocks: number;
-    sections: number;
-    subsections: number;
-    units: number;
-    components: number;
-    unsupported: number;
-  }
 }
 
 export interface MigrateTaskStatusData {

src/generic/key-utils.ts

@@ -126,3 +126,18 @@ export function normalizeContainerType(containerType: ContainerType | string) {
       return containerType;
   }
 }
+
+/**
+ * Given a usage key of V1 block like `block-v1:org+type@html+block@1`, get the type (e.g. `html`)
+ * @param usageKey e.g. `block-v1:org+type@html+block@1`
+ * @returns The block type as a string
+ */
+export function getBlockTypeBlockV1(usageKey: string): string {
+  if (usageKey && usageKey.startsWith('block-v1:')) {
+    const blockType = usageKey.match(/type@([^+]+)/);
+    if (blockType) {
+      return blockType[1];
+    }
+  }
+  throw new Error(`Invalid usageKey: ${usageKey}`);
+}

src/library-authoring/data/api.mocks.ts

@@ -34,6 +34,52 @@ export const mockGetContentLibraryV2List = {
   }),
 };
 
+export const mockGetModulestoreMigratedBlocksInfo = {
+  applyMockSuccess: () => jest.spyOn(api, 'getModulestoreMigrationBlocksInfo').mockResolvedValue(
+    [
+      {
+        sourceKey: 'block-v1:UNIX+UX2+2025_T2+type@chapter+block@1',
+        targetKey: '1',
+        unsupportedReason: undefined,
+      },
+      {
+        sourceKey: 'block-v1:UNIX+UX2+2025_T2+type@sequential+block@2',
+        targetKey: '2',
+        unsupportedReason: undefined,
+      },
+      {
+        sourceKey: 'block-v1:UNIX+UX2+2025_T2+type@vertical+block@2',
+        targetKey: '3',
+        unsupportedReason: undefined,
+      },
+      {
+        sourceKey: 'block-v1:UNIX+UX2+2025_T2+type@html+block@3',
+        targetKey: '4',
+        unsupportedReason: undefined,
+      },
+    ],
+  ),
+  applyMockPartial: () => jest.spyOn(api, 'getModulestoreMigrationBlocksInfo').mockResolvedValue(
+    [
+      {
+        sourceKey: 'block-v1:UNIX+UX2+2025_T2+type@library_content+block@test_lib_content',
+        targetKey: null,
+        unsupportedReason: 'The "library_content" XBlock (ID: "test_lib_content") has children, so it not supported in content libraries. It has 2 children blocks.',
+      },
+      {
+        sourceKey: 'block-v1:UNIX+UX2+2025_T2+type@html+block@1',
+        targetKey: '1',
+        unsupportedReason: undefined,
+      },
+      {
+        sourceKey: 'block-v1:UNIX+UX2+2025_T2+type@chapter+block@1',
+        targetKey: '2',
+        unsupportedReason: undefined,
+      },
+    ],
+  ),
+};
+
 /**
  * Mock for `getContentLibrary()`
  *

src/library-authoring/import-course/ImportDetailsPage.test.tsx

@@ -10,8 +10,7 @@ import { bulkModulestoreMigrateUrl } from '@src/data/api';
 import { useGetContentHits } from '@src/search-manager';
 import { ImportDetailsPage } from './ImportDetailsPage';
 import { LibraryProvider } from '../common/context/LibraryContext';
-import { mockContentLibrary } from '../data/api.mocks';
-import { getModulestoreMigratedBlocksInfoUrl } from '../data/api';
+import { mockContentLibrary, mockGetModulestoreMigratedBlocksInfo } from '../data/api.mocks';
 import { libraryComponentsMock } from '../__mocks__';
 
 mockContentLibrary.applyMock();
@@ -93,10 +92,13 @@ describe('', () => {
   });
 
   it('should render Succeeded state', async () => {
+    mockGetModulestoreMigratedBlocksInfo.applyMockSuccess();
     render(mockGetMigrationStatus.migrationId);
     expect(await screen.findByText(
       /test course has been imported to your library in a collection called test collection/i,
     ));
+    expect(await screen.findByText(/Total Blocks/i)).toBeInTheDocument();
+    expect(await screen.findByText('4')).toBeInTheDocument();
 
     const viewImportedContentBtn = screen.getByRole('button', {
       name: /view imported content/i,
@@ -107,13 +109,7 @@ describe('', () => {
   });
 
   it('should render Partial Succeeded state', async () => {
-    axiosMock.onGet(getModulestoreMigratedBlocksInfoUrl()).reply(200, [
-      {
-        sourceKey: 'block-v1:UNIX+UX2+2025_T2+type@library_content+block@test_lib_content',
-        targetKey: null,
-        unsupportedReason: 'The "library_content" XBlock (ID: "test_lib_content") has children, so it not supported in content libraries. It has 2 children blocks.',
-      },
-    ]);
+    mockGetModulestoreMigratedBlocksInfo.applyMockPartial();
     (useGetContentHits as jest.Mock).mockReturnValue({
       isPending: false,
       data: {
@@ -146,8 +142,13 @@ describe('', () => {
     render(mockGetMigrationStatus.migrationIdPartial);
     expect(await screen.findByText(/partial import successful/i)).toBeInTheDocument();
 
-    expect(screen.getByText(
-      /85% of course test course has been imported successfully/i,
+    expect(await screen.findByText(/Total Blocks/i)).toBeInTheDocument();
+    expect(await screen.findByText('2/5')).toBeInTheDocument();
+    expect(await screen.findByText(/Components/i)).toBeInTheDocument();
+    expect(await screen.findByText('1/4')).toBeInTheDocument();
+
+    expect(await screen.findByText(
+      /40% of course test course has been imported successfully/i,
     )).toBeInTheDocument();
 
     expect(await screen.findByRole('cell', {