EW-1060 modified linked task of topic

hpi-schul-cloud · Dec 13, 2024 · 2d480a2 · 2d480a2
1 parent 01d36c3
commit 2d480a2
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 28 deletions.
diff --git a/...rver/src/modules/common-cartridge/common-cartridge-client/lesson-client/dto/lesson.dto.ts b/...rver/src/modules/common-cartridge/common-cartridge-client/lesson-client/dto/lesson.dto.ts
@@ -19,7 +19,7 @@ export class LessonDto {
 
 	public materials: LessonMaterialsDto[];
 
-	public linkedTasks?: LessonLinkedTaskDto[];
+	public linkedTasks: LessonLinkedTaskDto[];
 
 	constructor(props: LessonDto) {
 		this.lessonId = props.lessonId;

diff --git a/...odules/common-cartridge/common-cartridge-client/lesson-client/mapper/lesson-dto.mapper.ts b/...odules/common-cartridge/common-cartridge-client/lesson-client/mapper/lesson-dto.mapper.ts
@@ -50,6 +50,7 @@ export class LessonDtoMapper {
 			contents: lessonResponse.contents
 				.map((content) => this.mapToLessenContentDto(content))
 				.filter((contetnDto) => contetnDto !== null),
+			linkedTasks: [],
 			materials: lessonResponse.materials.map((material) => this.mapToLessonMaterialDto(material)),
 		});
 

diff --git a/apps/server/src/modules/common-cartridge/service/common-cartridge-export.service.spec.ts b/apps/server/src/modules/common-cartridge/service/common-cartridge-export.service.spec.ts
@@ -185,7 +185,7 @@ describe('CommonCartridgeExportService', () => {
 				const { archive, lesson } = await setup();
 				const manifest = archive.getEntry('imsmanifest.xml')?.getData().toString();
 
-				lesson.linkedTasks?.forEach((linkedTask) => {
+				lesson.linkedTasks.forEach((linkedTask) => {
 					expect(manifest).toContain(`<title>${linkedTask.name}</title>`);
 				});
 			});
@@ -254,7 +254,7 @@ describe('CommonCartridgeExportService', () => {
 				const { archive, lesson } = await setup();
 				const manifest = archive.getEntry('imsmanifest.xml')?.getData().toString();
 
-				lesson.linkedTasks?.forEach((linkedTask) => {
+				lesson.linkedTasks.forEach((linkedTask) => {
 					expect(manifest).toContain(`<title>${linkedTask.name}</title>`);
 				});
 			});
@@ -337,29 +337,5 @@ describe('CommonCartridgeExportService', () => {
 				);
 			});
 		});
-
-		describe('when topics has no linked tasks', () => {
-			const setup = async () => setupParams(CommonCartridgeVersion.V_1_1_0, false, true, true);
-
-			it('should add lesson without linked tasks', async () => {
-				const { archive, lesson } = await setup();
-				lesson.linkedTasks = undefined;
-
-				expect(getFileContent(archive, 'imsmanifest.xml')).not.toContain(createXmlString('title', lesson.name));
-			});
-		});
-
-		describe('when columnBoards has no cards', () => {
-			const setup = async () => setupParams(CommonCartridgeVersion.V_1_1_0, true, true, false);
-
-			it('should add column boards without cards', async () => {
-				const { archive, boardSkeleton } = await setup();
-				boardSkeleton.columns[0].cards = [];
-
-				expect(getFileContent(archive, 'imsmanifest.xml')).not.toContain(
-					createXmlString('title', boardSkeleton.columns[0].title)
-				);
-			});
-		});
 	});
 });
diff --git a/apps/server/src/modules/common-cartridge/service/common-cartridge-export.service.ts b/apps/server/src/modules/common-cartridge/service/common-cartridge-export.service.ts
@@ -106,7 +106,7 @@ export class CommonCartridgeExportService {
 				this.addComponentToOrganization(content, lessonsOrganization);
 			});
 
-			lesson.linkedTasks?.forEach((task) => {
+			lesson.linkedTasks.forEach((task) => {
 				lessonsOrganization.addResource(this.mapper.mapLinkedTaskToResource(task, version));
 			});
 		});