Merge branch 'BC-6871-board-refactor' of github.com:hpi-schul-cloud/s…

…chulcloud-server into BC-6871-board-refactor

apps/server/src/modules/board/service/board-common-tool.service.spec.ts

@@ -0,0 +1,91 @@
+import { createMock, DeepMocked } from '@golevelup/ts-jest';
+import { Test, TestingModule } from '@nestjs/testing';
+import { NotFoundException } from '@nestjs/common';
+import { contextExternalToolFactory } from '@modules/tool/context-external-tool/testing';
+import { BoardCommonToolService } from './board-common-tool.service';
+import { BoardNodeRepo } from '../repo';
+import { BoardNodeService } from './board-node.service';
+import { ColumnBoard, MediaBoard, AnyBoardNode } from '../domain';
+
+import { columnBoardFactory, mediaBoardFactory } from '../testing';
+
+describe('BoardCommonToolService', () => {
+	let service: BoardCommonToolService;
+	let boardNodeRepo: DeepMocked<BoardNodeRepo>;
+	let boardNodeService: DeepMocked<BoardNodeService>;
+
+	beforeAll(async () => {
+		const module: TestingModule = await Test.createTestingModule({
+			providers: [
+				BoardCommonToolService,
+				{
+					provide: BoardNodeRepo,
+					useValue: createMock<BoardNodeRepo>(),
+				},
+				{
+					provide: BoardNodeService,
+					useValue: createMock<BoardNodeService>(),
+				},
+			],
+		}).compile();
+
+		service = module.get<BoardCommonToolService>(BoardCommonToolService);
+		boardNodeRepo = module.get(BoardNodeRepo);
+		boardNodeService = module.get(BoardNodeService);
+	});
+
+	beforeEach(() => {
+		jest.resetAllMocks();
+	});
+
+	describe('countBoardUsageForExternalTools', () => {
+		const setup = () => {
+			const contextExternalTools = [contextExternalToolFactory.build(), contextExternalToolFactory.build()];
+
+			const boardNodes: AnyBoardNode[] = [
+				{ rootId: '1' } as AnyBoardNode,
+				{ rootId: '2' } as AnyBoardNode,
+				{ rootId: '1' } as AnyBoardNode,
+			];
+			boardNodeRepo.findByContextExternalToolIds.mockResolvedValue(boardNodes);
+
+			return { contextExternalTools };
+		};
+		it('should count board usage for external tools', async () => {
+			const { contextExternalTools } = setup();
+			const result = await service.countBoardUsageForExternalTools(contextExternalTools);
+
+			expect(result).toBe(2);
+		});
+	});
+
+	describe('findByDescendant', () => {
+		it('should return the root node when it is a ColumnBoard', async () => {
+			const boardNode: AnyBoardNode = { id: '1', rootId: '2' } as AnyBoardNode;
+			const rootNode: ColumnBoard = columnBoardFactory.build();
+			boardNodeService.findRoot.mockResolvedValue(rootNode);
+
+			const result = await service.findByDescendant(boardNode);
+
+			expect(result).toBe(rootNode);
+		});
+
+		it('should return the root node when it is a MediaBoard', async () => {
+			const boardNode: AnyBoardNode = { id: '1', rootId: '2' } as AnyBoardNode;
+			const rootNode: MediaBoard = mediaBoardFactory.build();
+			boardNodeService.findRoot.mockResolvedValue(rootNode);
+
+			const result = await service.findByDescendant(boardNode);
+
+			expect(result).toBe(rootNode);
+		});
+
+		it('should throw NotFoundException when root node is not a ColumnBoard or MediaBoard', async () => {
+			const boardNode: AnyBoardNode = { id: '1', rootId: '2' } as AnyBoardNode;
+			const rootNode: AnyBoardNode = { id: '2' } as AnyBoardNode;
+			boardNodeService.findRoot.mockResolvedValue(rootNode);
+
+			await expect(service.findByDescendant(boardNode)).rejects.toThrow(NotFoundException);
+		});
+	});
+});

apps/server/src/modules/board/service/board-common-tool.service.ts

@@ -26,10 +26,10 @@ export class BoardCommonToolService {
 	async findByDescendant(boardNode: AnyBoardNode): Promise<ColumnBoard | MediaBoard> {
 		const rootNode = await this.boardNodeService.findRoot(boardNode);
 
-		if (!isColumnBoard(boardNode) && !isMediaBoard(boardNode)) {
+		if (!isColumnBoard(rootNode) && !isMediaBoard(rootNode)) {
 			throw new NotFoundException(`There is no board with this id`);
 		}
 
-		return rootNode as ColumnBoard | MediaBoard;
+		return rootNode;
 	}
 }