Skip to content

Commit

Permalink
fix: test, lint 및 build 오류 수정
Browse files Browse the repository at this point in the history
  • Loading branch information
ezcolin2 committed Nov 29, 2024
1 parent ba70c75 commit a4f9000
Show file tree
Hide file tree
Showing 9 changed files with 17 additions and 15 deletions.
2 changes: 1 addition & 1 deletion apps/backend/src/auth/auth.module.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Module } from '@nestjs/common';
import { UserRepository } from '../user/user.repository';
import { UserModule } from 'src/user/user.module';
import { UserModule } from '../user/user.module';
import { AuthService } from './auth.service';
import { AuthController } from './auth.controller';
import { NaverStrategy } from './strategies/naver.strategy';
Expand Down
6 changes: 3 additions & 3 deletions apps/backend/src/auth/auth.service.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ import { SignUpDto } from './dtos/signUp.dto';
import { User } from '../user/user.entity';
import { UpdateUserDto } from './dtos/UpdateUser.dto';
import { UserNotFoundException } from '../exception/user.exception';
import { Snowflake } from '@theinternetfolks/snowflake';

describe('AuthService', () => {
let authService: AuthService;
Expand Down Expand Up @@ -68,9 +69,8 @@ describe('AuthService', () => {
provider: 'naver',
email: 'new@naver.com',
};
const generated

flakeId = Snowflake.generate(); // Snowflake.generate()의 mock 값을 준비

const generatedSnowflakeId = Snowflake.generate(); // Snowflake.generate()의 mock 값을 준비
const newDate = new Date();
const createdUser = {
providerId: dto.providerId,
Expand Down
1 change: 0 additions & 1 deletion apps/backend/src/auth/auth.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import { UpdateUserDto } from './dtos/UpdateUser.dto';
import { UserNotFoundException } from '../exception/user.exception';
import { Snowflake } from '@theinternetfolks/snowflake';


@Injectable()
export class AuthService {
constructor(private readonly userRepository: UserRepository) {}
Expand Down
2 changes: 1 addition & 1 deletion apps/backend/src/node/node.service.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,14 +9,14 @@ import { CreateNodeDto } from './dtos/createNode.dto';
import { UpdateNodeDto } from './dtos/updateNode.dto';
import { MoveNodeDto } from './dtos/moveNode.dto';
import { WorkspaceRepository } from '../workspace/workspace.repository';
import { Workspace } from '../workspace/workspace.entity';

describe('NodeService', () => {
let service: NodeService;
let nodeRepository: jest.Mocked<NodeRepository>;
let pageRepository: jest.Mocked<PageRepository>;
let workspaceRepository: jest.Mocked<WorkspaceRepository>;


beforeEach(async () => {
const module: TestingModule = await Test.createTestingModule({
providers: [
Expand Down
1 change: 0 additions & 1 deletion apps/backend/src/page/page.entity.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import {
CreateDateColumn,
UpdateDateColumn,
VersionColumn,
IsNull,
} from 'typeorm';
import { Node } from '../node/node.entity';
import { Workspace } from '../workspace/workspace.entity';
Expand Down
11 changes: 8 additions & 3 deletions apps/backend/src/page/page.service.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ describe('PageService', () => {
save: jest.fn(),
delete: jest.fn(),
findOneBy: jest.fn(),
findOne: jest.fn(),
findPagesByWorkspace: jest.fn(),
},
},
Expand Down Expand Up @@ -135,6 +136,7 @@ describe('PageService', () => {
content: {} as JSON,
emoji: undefined,
workspace: workspace1,
node: newNode,
});
});
});
Expand Down Expand Up @@ -235,19 +237,22 @@ describe('PageService', () => {
emoji: null,
workspace: null,
};
jest.spyOn(pageRepository, 'findOneBy').mockResolvedValue(expectedPage);
jest.spyOn(pageRepository, 'findOne').mockResolvedValue(expectedPage);

await expect(service.findPageById(1)).resolves.toEqual(expectedPage);
});

it('id에 해당하는 페이지가 없을 경우 PageNotFoundException을 throw한다.', async () => {
jest.spyOn(pageRepository, 'findOneBy').mockResolvedValue(undefined);
jest.spyOn(pageRepository, 'findOne').mockResolvedValue(undefined);

await expect(service.findPageById(1)).rejects.toThrow(
PageNotFoundException,
);

expect(pageRepository.findOneBy).toHaveBeenCalledWith({ id: 1 });
expect(pageRepository.findOne).toHaveBeenCalledWith({
where: { id: 1 },
relations: ['node'],
});
});
});

Expand Down
1 change: 0 additions & 1 deletion apps/backend/src/tasks/tasks.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ import { Injectable, Logger } from '@nestjs/common';
import { Cron, CronExpression } from '@nestjs/schedule';
import { RedisService } from '../redis/redis.service';
import { PageService } from '../page/page.service';
import { PageNotFoundException } from 'src/exception/page.exception';

@Injectable()
export class TasksService {
Expand Down
2 changes: 1 addition & 1 deletion apps/backend/src/yjs/yjs.module.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { Module } from '@nestjs/common';
import { YjsService } from './yjs.service';
import { NodeModule } from 'src/node/node.module';
import { NodeModule } from '../node/node.module';
import { PageModule } from '../page/page.module';
import { EdgeModule } from '../edge/edge.module';
import { RedisModule } from '../redis/redis.module';
Expand Down
6 changes: 3 additions & 3 deletions apps/backend/src/yjs/yjs.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,11 @@ import {
} from 'y-prosemirror';
import { novelEditorSchema } from './yjs.schema';
import { EdgeService } from '../edge/edge.service';
import { Node } from 'src/node/node.entity';
import { Edge } from 'src/edge/edge.entity';
import { Node } from '../node/node.entity';
import { Edge } from '../edge/edge.entity';
import { YMapEdge } from './yjs.type';
import { RedisService } from '../redis/redis.service';
import { PageNotFoundException } from 'src/exception/page.exception';
import { PageNotFoundException } from '../exception/page.exception';

// Y.Doc에는 name 컬럼이 없어서 생성했습니다.
class CustomDoc extends Y.Doc {
Expand Down

0 comments on commit a4f9000

Please sign in to comment.