Skip to content

Commit

Permalink
refactor: 커스텀 에러 코드 두자리를 가지도록 변경
Browse files Browse the repository at this point in the history
  • Loading branch information
@quiet-honey
quiet-honey committed Dec 5, 2023
1 parent 8c444c0 commit b77c7a3
Show file tree
Hide file tree
Showing 4 changed files with 26 additions and 26 deletions.
10 changes: 5 additions & 5 deletions BE/src/member/controller/member.controller.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ describe('MemberController 통합 테스트', () => {
});

describe('getMyInfo', () => {
it('쿠키를 가지고 회원 정보 반환 요청을 하면 200 상태 코드와 회원 정보가 반환된다.)', (done) => {
it('쿠키를 가지고 회원 정보 반환 요청을 하면 200 상태 코드와 회원 정보가 반환된다.', (done) => {
authService.login(oauthRequestFixture).then((validToken) => {
const agent = request.agent(app.getHttpServer());
agent
Expand All @@ -172,7 +172,7 @@ describe('MemberController 통합 테스트', () => {
});
});

it('유효하지 않은 토큰으로 회원 정보 반환을 요청하면 401 상태코드가 반환된다.)', () => {
it('유효하지 않은 토큰으로 회원 정보 반환을 요청하면 401 상태코드가 반환된다.', () => {
const agent = request.agent(app.getHttpServer());
agent
.get('/api/member')
Expand All @@ -182,7 +182,7 @@ describe('MemberController 통합 테스트', () => {
});

describe('getNameForInterview', () => {
it('쿠키를 가지고 면접 화면에 표출할 닉네임 반환 요청을 하면 200 상태 코드와 회원 닉네임이 들어간 채로 반환된다.)', (done) => {
it('쿠키를 가지고 면접 화면에 표출할 닉네임 반환 요청을 하면 200 상태 코드와 회원 닉네임이 들어간 채로 반환된다.', (done) => {
authService.login(oauthRequestFixture).then((validToken) => {
const agent = request.agent(app.getHttpServer());
agent
Expand All @@ -198,15 +198,15 @@ describe('MemberController 통합 테스트', () => {
});
});

it('유효하지 않은 토큰으로 면접 화면에 표출할 닉네임 반환을 요청하면 401 상태코드가 반환된다.)', () => {
it('유효하지 않은 토큰으로 면접 화면에 표출할 닉네임 반환을 요청하면 401 상태코드가 반환된다.', () => {
const agent = request.agent(app.getHttpServer());
agent
.get('/api/member/name')
.set('Cookie', [`accessToken=Bearer INVALID_TOKEN`])
.expect(401);
});

it('만료된 토큰으로 면접 화면에 표출할 닉네임 반환을 요청하면 410 상태코드가 반환된다.)', async () => {
it('만료된 토큰으로 면접 화면에 표출할 닉네임 반환을 요청하면 410 상태코드가 반환된다.', async () => {
const expirationTime = Math.floor(Date.now() / 1000) - 1;
const expiredToken = await jwtService.signAsync(
{ id: memberFixture.id } as TokenPayload,
Expand Down
2 changes: 1 addition & 1 deletion BE/src/member/exception/member.exception.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,6 @@ import { HttpNotFoundException } from 'src/util/exception.util';

export class MemberNotFoundException extends HttpNotFoundException {
constructor() {
super('회원을 찾을 수 없습니다.', 'M1');
super('회원을 찾을 수 없습니다.', 'M01');
}
}
24 changes: 12 additions & 12 deletions BE/src/video/controller/video.controller.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ export class VideoController {
PreSignedUrlResponse,
),
)
@ApiResponse(createApiResponseOption(500, 'V1, SERVER', null))
@ApiResponse(createApiResponseOption(500, 'V01, SERVER', null))
async getPreSignedUrl(@Req() req: Request) {
return await this.videoService.getPreSignedUrl(req.user as Member);
}
Expand Down Expand Up @@ -94,9 +94,9 @@ export class VideoController {
VideoDetailResponse,
),
)
@ApiResponse(createApiResponseOption(403, 'V2', null))
@ApiResponse(createApiResponseOption(404, 'V4, M1', null))
@ApiResponse(createApiResponseOption(500, 'V6', null))
@ApiResponse(createApiResponseOption(403, 'V02', null))
@ApiResponse(createApiResponseOption(404, 'V04, M01', null))
@ApiResponse(createApiResponseOption(500, 'V06', null))
async getVideoDetailByHash(@Param('hash') hash: string) {
return await this.videoService.getVideoDetailByHash(hash);
}
Expand All @@ -114,9 +114,9 @@ export class VideoController {
VideoDetailResponse,
),
)
@ApiResponse(createApiResponseOption(403, 'V2', null))
@ApiResponse(createApiResponseOption(404, 'V3', null))
@ApiResponse(createApiResponseOption(500, 'V8, SERVER', null))
@ApiResponse(createApiResponseOption(403, 'V02', null))
@ApiResponse(createApiResponseOption(404, 'V03', null))
@ApiResponse(createApiResponseOption(500, 'V08, SERVER', null))
async getVideoDetail(@Param('videoId') videoId: number, @Req() req: Request) {
return await this.videoService.getVideoDetail(videoId, req.user as Member);
}
Expand All @@ -130,9 +130,9 @@ export class VideoController {
@ApiResponse(
createApiResponseOption(200, '비디오 상태 전환 완료', VideoHashResponse),
)
@ApiResponse(createApiResponseOption(403, 'V2', null))
@ApiResponse(createApiResponseOption(404, 'V3', null))
@ApiResponse(createApiResponseOption(500, 'V5, V6, V7, SERVER', null))
@ApiResponse(createApiResponseOption(403, 'V02', null))
@ApiResponse(createApiResponseOption(404, 'V03', null))
@ApiResponse(createApiResponseOption(500, 'V05, V06, V07, SERVER', null))
async toggleVideoStatus(
@Param('videoId') videoId: number,
@Req() req: Request,
Expand All @@ -150,8 +150,8 @@ export class VideoController {
summary: '비디오 삭제',
})
@ApiResponse(createApiResponseOption(204, '비디오 삭제 완료', null))
@ApiResponse(createApiResponseOption(403, 'V2', null))
@ApiResponse(createApiResponseOption(404, 'V3', null))
@ApiResponse(createApiResponseOption(403, 'V02', null))
@ApiResponse(createApiResponseOption(404, 'V03', null))
@ApiResponse(createApiResponseOption(500, 'SERVER', null))
async deleteVideo(
@Param('videoId') videoId: number,
Expand Down
16 changes: 8 additions & 8 deletions BE/src/video/exception/video.exception.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6,48 +6,48 @@ import {

export class IDriveException extends HttpInternalServerError {
constructor() {
super('IDrive 제공 API 처리 도중 에러가 발생하였습니다.', 'V1');
super('IDrive 제공 API 처리 도중 에러가 발생하였습니다.', 'V01');
}
}

export class VideoAccessForbiddenException extends HttpForbiddenException {
constructor() {
super('해당 비디오에 접근 권한이 없습니다.', 'V2');
super('해당 비디오에 접근 권한이 없습니다.', 'V02');
}
}

export class VideoNotFoundException extends HttpNotFoundException {
constructor() {
super('존재하지 않는 비디오입니다.', 'V3');
super('존재하지 않는 비디오입니다.', 'V03');
}
}

export class VideoOfWithdrawnMemberException extends HttpNotFoundException {
constructor() {
super('탈퇴한 회원의 비디오를 조회할 수 없습니다.', 'V4');
super('탈퇴한 회원의 비디오를 조회할 수 없습니다.', 'V04');
}
}

export class RedisDeleteException extends HttpInternalServerError {
constructor() {
super('Redis에서 비디오 정보 삭제 중 오류가 발생하였습니다.', 'V5');
super('Redis에서 비디오 정보 삭제 중 오류가 발생하였습니다.', 'V05');
}
}

export class RedisRetrieveException extends HttpInternalServerError {
constructor() {
super('Redis에서 비디오 정보를 가져오는 중 오류가 발생하였습니다.', 'V6');
super('Redis에서 비디오 정보를 가져오는 중 오류가 발생하였습니다.', 'V06');
}
}

export class RedisSaveException extends HttpInternalServerError {
constructor() {
super('Redis에 비디오 정보 저장 중 오류가 발생하였습니다.', 'V7');
super('Redis에 비디오 정보 저장 중 오류가 발생하였습니다.', 'V07');
}
}

export class Md5HashException extends HttpInternalServerError {
constructor() {
super('MD5 해시 생성 중 오류가 발생했습니다.', 'V8');
super('MD5 해시 생성 중 오류가 발생했습니다.', 'V08');
}
}

0 comments on commit b77c7a3

Please sign in to comment.