fix: backend 테스트 오류 수정

Author: ezcolin2

backend/src/node/node.service.spec.ts

@@ -24,6 +24,7 @@ describe('NodeService', () => {
             save: jest.fn(),
             delete: jest.fn(),
             findOneBy: jest.fn(),
+            findOne: jest.fn(),
           },
         },
         {
@@ -101,7 +102,7 @@ describe('NodeService', () => {
       const node = new Node();
       node.page = new Page();
 
-      jest.spyOn(nodeRepository, 'findOneBy').mockResolvedValue(node);
+      jest.spyOn(nodeRepository, 'findOne').mockResolvedValue(node);
       jest.spyOn(pageRepository, 'findOneBy').mockResolvedValue(node.page);
       jest.spyOn(nodeRepository, 'save').mockResolvedValue(node);
 
@@ -119,7 +120,7 @@ describe('NodeService', () => {
     });
 
     it('업데이트할 노드가 존재하지 않으면 NodeNotFoundException을 throw한다.', async () => {
-      jest.spyOn(nodeRepository, 'findOneBy').mockResolvedValue(undefined);
+      jest.spyOn(nodeRepository, 'findOne').mockResolvedValue(undefined);
 
       await expect(service.updateNode(1, {} as any)).rejects.toThrow(
         NodeNotFoundException,
@@ -131,12 +132,24 @@ describe('NodeService', () => {
     it('노드 아이디를 받아 해당 노드의 좌표를 반환한다.', async () => {
       const node = { id: 1, x: 1, y: 2 } as Node;
 
-      nodeRepository.findOneBy.mockResolvedValue(node);
+      jest.spyOn(nodeRepository, 'findOne').mockResolvedValue(node);
 
       const coordinates = await service.getCoordinates(1);
 
       expect(coordinates).toEqual({ x: 1, y: 2 });
-      expect(nodeRepository.findOneBy).toHaveBeenCalledWith({ id: 1 });
+      expect(nodeRepository.findOne).toHaveBeenCalledWith({
+        relations: ['page'],
+        select: {
+          id: true,
+          page: {
+            id: true,
+            title: true,
+          },
+        },
+        where: {
+          id: 1,
+        },
+      });
     });
 
     it('노드를 찾을 수 없으면 NodeNotFoundException을 throw한다.', async () => {
@@ -159,20 +172,44 @@ describe('NodeService', () => {
         outgoingEdges: [],
         incomingEdges: [],
       } as Node;
-      jest.spyOn(nodeRepository, 'findOneBy').mockResolvedValue(node);
+      jest.spyOn(nodeRepository, 'findOne').mockResolvedValue(node);
 
       const result = await service.findNodeById(1);
 
       expect(result).toEqual(node);
-      expect(nodeRepository.findOneBy).toHaveBeenCalledWith({ id: 1 });
+      expect(nodeRepository.findOne).toHaveBeenCalledWith({
+        relations: ['page'],
+        select: {
+          id: true,
+          page: {
+            id: true,
+            title: true,
+          },
+        },
+        where: {
+          id: 1,
+        },
+      });
     });
 
     it('노드를 찾을 수 없으면 NodeNotFoundException을 던진다.', async () => {
-      jest.spyOn(nodeRepository, 'findOneBy').mockResolvedValue(undefined);
+      jest.spyOn(nodeRepository, 'findOne').mockResolvedValue(undefined);
       await expect(service.findNodeById(1)).rejects.toThrow(
         NodeNotFoundException,
       );
-      expect(nodeRepository.findOneBy).toHaveBeenCalledWith({ id: 1 });
+      expect(nodeRepository.findOne).toHaveBeenCalledWith({
+        relations: ['page'],
+        select: {
+          id: true,
+          page: {
+            id: true,
+            title: true,
+          },
+        },
+        where: {
+          id: 1,
+        },
+      });
     });
   });
 });

backend/src/node/node.service.ts

@@ -47,9 +47,25 @@ export class NodeService {
   }
 
   async updateNode(id: number, dto: UpdateNodeDto): Promise<Node> {
-    // 갱신할 노드를 조회한다.
-    const node = await this.findNodeById(id);
+    // 노드를 조회한다.
+    const node = await this.nodeRepository.findOne({
+      relations: ['page'],
+      select: {
+        id: true,
+        page: {
+          id: true,
+          title: true, // content 제외하고 title만 선택
+        },
+      },
+      where: {
+        id,
+      },
+    });
 
+    // 노드가 없으면 NotFound 에러
+    if (!node) {
+      throw new NodeNotFoundException();
+    }
     // 노드와 연결된 페이지를 조회한다.
     const linkedPage = await this.pageRepository.findOneBy({
       id: node.page.id,
@@ -66,12 +82,20 @@ export class NodeService {
 
   async findNodeById(id: number): Promise<Node> {
     // 노드를 조회한다.
-    const node = await this.nodeRepository
-      .createQueryBuilder('node')
-      .leftJoinAndSelect('node.page', 'page')
-      .select(['node.id', 'node.x', 'node.y', 'page.id', 'page.title'])
-      .where('node.id = :id', { id })
-      .getOne();
+    const node = await this.nodeRepository.findOne({
+      relations: ['page'],
+      select: {
+        id: true,
+        page: {
+          id: true,
+          title: true, // content 제외하고 title만 선택
+        },
+      },
+      where: {
+        id,
+      },
+    });
+
     // 노드가 없으면 NotFound 에러
     if (!node) {
       throw new NodeNotFoundException();
@@ -81,11 +105,16 @@ export class NodeService {
 
   async findNodes(): Promise<Node[]> {
     // 노드를 조회한다.
-    const nodes = await this.nodeRepository
-      .createQueryBuilder('node')
-      .leftJoinAndSelect('node.page', 'page')
-      .select(['node.id', 'node.x', 'node.y', 'page.id', 'page.title'])
-      .getMany();
+    const nodes = await this.nodeRepository.find({
+      relations: ['page'],
+      select: {
+        id: true,
+        page: {
+          id: true,
+          title: true, // content 제외하고 title만 선택
+        },
+      },
+    });
     // 노드가 없으면 NotFound 에러
     if (!nodes) {
       throw new NodeNotFoundException();