Skip to content

Commit 3440fa2

Browse files
committed
modify: 코드래빗 리뷰 1차 반영
1 parent 37b9633 commit 3440fa2

File tree

3 files changed

+9
-4
lines changed

3 files changed

+9
-4
lines changed

src/services/__test__/qr.service.test.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import { UserService } from '@/services/user.service';
22
import { UserRepository } from '@/repositories/user.repository';
33
import { DBError } from '@/exception';
44
import { QRLoginToken } from '@/types/models/QRLoginToken.type';
5+
import { Pool } from 'pg';
56

67
jest.mock('@/repositories/user.repository');
78

@@ -10,7 +11,8 @@ describe('UserService', () => {
1011
let repo: jest.Mocked<UserRepository>;
1112

1213
beforeEach(() => {
13-
const repoInstance = new UserRepository({} as any);
14+
const mockPool = {} as jest.Mocked<Pool>;
15+
const repoInstance = new UserRepository(mockPool);
1416
repo = repoInstance as jest.Mocked<UserRepository>;
1517
service = new UserService(repo);
1618
});

src/services/user.service.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,8 +125,8 @@ export class UserService {
125125
return await this.userRepo.updateTokens(userData.id, userData.accessToken, userData.refreshToken);
126126
}
127127

128-
public getDecryptedTokens(userId: number, accessToken: string, refreshToken: string) {
129-
return this.decryptTokens(userId, accessToken, refreshToken);
128+
public getDecryptedTokens(groupId: number, accessToken: string, refreshToken: string) {
129+
return this.decryptTokens(groupId, accessToken, refreshToken);
130130
}
131131

132132
async create(userId: number, ip: string, userAgent: string): Promise<string> {

src/utils/generateRandomToken.util.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,11 @@
1+
import crypto from 'crypto';
2+
13
export function generateRandomToken(length: number = 10): string {
24
const chars = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789';
35
let result = '';
6+
const randomBytes = crypto.randomBytes(length);
47
for (let i = 0; i < length; i++) {
5-
result += chars.charAt(Math.floor(Math.random() * chars.length));
8+
result += chars.charAt(randomBytes[i] % chars.length);
69
}
710
return result;
811
}

0 commit comments

Comments
 (0)