test: Tests added for getBlockTypeBlockV1

Author: ChrisChV

src/data/api.mocks.ts

@@ -14,8 +14,6 @@ export async function mockGetMigrationStatus(migrationId: string): Promise<api.M
       return new Promise(() => {});
     case mockGetMigrationStatus.migrationIdInProgress:
       return mockGetMigrationStatus.migrationStatusInProgressData;
-    case mockGetMigrationStatus.migrationIdPartial:
-      return mockGetMigrationStatus.migrationStatusPartialData;
     default:
       /* istanbul ignore next */
       throw new Error(`mockGetMigrationStatus: unknown migration ID "${migrationId}"`);
@@ -180,33 +178,4 @@ mockGetMigrationStatus.migrationStatusInProgressData = {
     },
   ],
 } as api.MigrateTaskStatusData;
-mockGetMigrationStatus.migrationIdPartial = '7';
-mockGetMigrationStatus.migrationStatusPartialData = {
-  uuid: mockGetMigrationStatus.migrationIdPartial,
-  state: 'Succeeded',
-  stateText: 'Succeeded',
-  completedSteps: 9,
-  totalSteps: 9,
-  attempts: 1,
-  created: '',
-  modified: '',
-  artifacts: [],
-  parameters: [
-    {
-      id: 1,
-      source: 'legacy-lib-1',
-      target: 'lib',
-      compositionLevel: 'component',
-      repeatHandlingStrategy: 'update',
-      preserveUrlSlugs: false,
-      targetCollectionSlug: 'coll-1',
-      forwardSourceToTarget: true,
-      isFailed: false,
-      targetCollection: {
-        key: 'coll',
-        title: 'Test Collection',
-      },
-    },
-  ],
-} as api.MigrateTaskStatusData;
 mockGetMigrationStatus.applyMock = () => jest.spyOn(api, 'getModulestoreMigrationStatus').mockImplementation(mockGetMigrationStatus);

src/generic/key-utils.test.ts

@@ -2,6 +2,7 @@ import {
   buildCollectionUsageKey,
   ContainerType,
   getBlockType,
+  getBlockTypeBlockV1,
   getLibraryId,
   isLibraryKey,
   isLibraryV1Key,
@@ -142,4 +143,25 @@ describe('component utils', () => {
       });
     }
   });
+
+  describe('getBlockTypeBlockV1', () => {
+    for (const [input, expected] of [
+      ['block-v1:org+type@html+block@1', 'html'],
+      ['block-v1:OpenCraftX+type@html+block@1571fe018-f3ce-45c9-8f53-5dafcb422fdd', 'html'],
+      ['block-v1:Axim+type@problem+block@571fe018-f3ce-45c9-8f53-5dafcb422fdd', 'problem'],
+      ['block-v1:org+type@unit+block@1', 'unit'],
+      ['block-v1:org+type@section+block@1', 'section'],
+      ['block-v1:org+type@subsection+block@1', 'subsection'],
+    ]) {
+      it(`returns '${expected}' for usage key '${input}'`, () => {
+        expect(getBlockTypeBlockV1(input)).toStrictEqual(expected);
+      });
+    }
+
+    for (const input of ['', undefined, null, 'not a key', 'block-v1:foo']) {
+      it(`throws an exception for usage key '${input}'`, () => {
+        expect(() => getBlockTypeBlockV1(input as any)).toThrow(`Invalid usageKey: ${input}`);
+      });
+    }
+  });
 });

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

@@ -139,7 +139,7 @@ describe('', () => {
       hits: libraryComponentsMock,
     });
 
-    render(mockGetMigrationStatus.migrationIdPartial);
+    render(mockGetMigrationStatus.migrationId);
     expect(await screen.findByText(/partial import successful/i)).toBeInTheDocument();
 
     expect(await screen.findByText(/Total Blocks/i)).toBeInTheDocument();