refactor: TBM/작업보고 코드 통합 및 API 쿼리 버그 수정
- 공통 유틸리티 추출 (common/utils.js, common/base-state.js) - TBM 모바일 인라인 JS/CSS 외부 파일로 분리 (tbm-mobile.js, tbm-mobile.css) - 미사용 코드 삭제 (index.js, work-report-*.js 등 5개 파일) - TBM/작업보고 state.js, utils.js를 공통 모듈 기반으로 전환 - 작업보고서 SSO 인증 호환 수정 (token/user 함수) - tbmModel.js: incomplete-reports 쿼리에서 users→sso_users 조인 수정, leader_name 조인 추가 - docker-compose.yml: system1-web 볼륨 마운트 추가 - 모바일 인계(handover) 기능 추가 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -19,6 +19,7 @@ describe('WorkReportService', () => {
|
||||
|
||||
describe('createWorkReportService', () => {
|
||||
it('단일 보고서를 성공적으로 생성해야 함', async () => {
|
||||
// Arrange
|
||||
const reportData = {
|
||||
report_date: '2025-12-11',
|
||||
worker_id: 1,
|
||||
@@ -28,32 +29,46 @@ describe('WorkReportService', () => {
|
||||
work_content: '기능 개발'
|
||||
};
|
||||
|
||||
workReportModel.create.mockResolvedValue(123);
|
||||
// workReportModel.create가 콜백 형태이므로 모킹 설정
|
||||
workReportModel.create = jest.fn((data, callback) => {
|
||||
callback(null, 123); // insertId = 123
|
||||
});
|
||||
|
||||
// Act
|
||||
const result = await workReportService.createWorkReportService(reportData);
|
||||
|
||||
// Assert
|
||||
expect(result).toEqual({ workReport_ids: [123] });
|
||||
expect(workReportModel.create).toHaveBeenCalledTimes(1);
|
||||
expect(workReportModel.create).toHaveBeenCalledWith(reportData);
|
||||
expect(workReportModel.create).toHaveBeenCalledWith(
|
||||
reportData,
|
||||
expect.any(Function)
|
||||
);
|
||||
});
|
||||
|
||||
it('다중 보고서를 성공적으로 생성해야 함', async () => {
|
||||
// Arrange
|
||||
const reportsData = [
|
||||
{ report_date: '2025-12-11', worker_id: 1, work_hours: 8 },
|
||||
{ report_date: '2025-12-11', worker_id: 2, work_hours: 7 }
|
||||
];
|
||||
|
||||
workReportModel.create
|
||||
.mockResolvedValueOnce(101)
|
||||
.mockResolvedValueOnce(102);
|
||||
let callCount = 0;
|
||||
workReportModel.create = jest.fn((data, callback) => {
|
||||
callCount++;
|
||||
callback(null, 100 + callCount);
|
||||
});
|
||||
|
||||
// Act
|
||||
const result = await workReportService.createWorkReportService(reportsData);
|
||||
|
||||
// Assert
|
||||
expect(result).toEqual({ workReport_ids: [101, 102] });
|
||||
expect(workReportModel.create).toHaveBeenCalledTimes(2);
|
||||
});
|
||||
|
||||
it('빈 배열이면 ValidationError를 던져야 함', async () => {
|
||||
// Act & Assert
|
||||
await expect(workReportService.createWorkReportService([]))
|
||||
.rejects.toThrow(ValidationError);
|
||||
|
||||
@@ -62,10 +77,14 @@ describe('WorkReportService', () => {
|
||||
});
|
||||
|
||||
it('DB 오류 시 DatabaseError를 던져야 함', async () => {
|
||||
// Arrange
|
||||
const reportData = { report_date: '2025-12-11', worker_id: 1 };
|
||||
|
||||
workReportModel.create.mockRejectedValue(new Error('DB connection failed'));
|
||||
workReportModel.create = jest.fn((data, callback) => {
|
||||
callback(new Error('DB connection failed'), null);
|
||||
});
|
||||
|
||||
// Act & Assert
|
||||
await expect(workReportService.createWorkReportService(reportData))
|
||||
.rejects.toThrow(DatabaseError);
|
||||
});
|
||||
@@ -73,18 +92,24 @@ describe('WorkReportService', () => {
|
||||
|
||||
describe('getWorkReportsByDateService', () => {
|
||||
it('날짜로 보고서를 조회해야 함', async () => {
|
||||
// Arrange
|
||||
const date = '2025-12-11';
|
||||
const mockReports = mockWorkReports.filter(r => r.report_date === date);
|
||||
|
||||
workReportModel.getAllByDate.mockResolvedValue(mockReports);
|
||||
workReportModel.getAllByDate = jest.fn((date, callback) => {
|
||||
callback(null, mockReports);
|
||||
});
|
||||
|
||||
// Act
|
||||
const result = await workReportService.getWorkReportsByDateService(date);
|
||||
|
||||
// Assert
|
||||
expect(result).toEqual(mockReports);
|
||||
expect(workReportModel.getAllByDate).toHaveBeenCalledWith(date);
|
||||
expect(workReportModel.getAllByDate).toHaveBeenCalledWith(date, expect.any(Function));
|
||||
});
|
||||
|
||||
it('날짜가 없으면 ValidationError를 던져야 함', async () => {
|
||||
// Act & Assert
|
||||
await expect(workReportService.getWorkReportsByDateService(null))
|
||||
.rejects.toThrow(ValidationError);
|
||||
|
||||
@@ -93,8 +118,12 @@ describe('WorkReportService', () => {
|
||||
});
|
||||
|
||||
it('DB 오류 시 DatabaseError를 던져야 함', async () => {
|
||||
workReportModel.getAllByDate.mockRejectedValue(new Error('DB error'));
|
||||
// Arrange
|
||||
workReportModel.getAllByDate = jest.fn((date, callback) => {
|
||||
callback(new Error('DB error'), null);
|
||||
});
|
||||
|
||||
// Act & Assert
|
||||
await expect(workReportService.getWorkReportsByDateService('2025-12-11'))
|
||||
.rejects.toThrow(DatabaseError);
|
||||
});
|
||||
@@ -102,19 +131,28 @@ describe('WorkReportService', () => {
|
||||
|
||||
describe('getWorkReportByIdService', () => {
|
||||
it('ID로 보고서를 조회해야 함', async () => {
|
||||
// Arrange
|
||||
const mockReport = mockWorkReports[0];
|
||||
|
||||
workReportModel.getById.mockResolvedValue(mockReport);
|
||||
workReportModel.getById = jest.fn((id, callback) => {
|
||||
callback(null, mockReport);
|
||||
});
|
||||
|
||||
// Act
|
||||
const result = await workReportService.getWorkReportByIdService(1);
|
||||
|
||||
// Assert
|
||||
expect(result).toEqual(mockReport);
|
||||
expect(workReportModel.getById).toHaveBeenCalledWith(1);
|
||||
expect(workReportModel.getById).toHaveBeenCalledWith(1, expect.any(Function));
|
||||
});
|
||||
|
||||
it('보고서가 없으면 NotFoundError를 던져야 함', async () => {
|
||||
workReportModel.getById.mockResolvedValue(null);
|
||||
// Arrange
|
||||
workReportModel.getById = jest.fn((id, callback) => {
|
||||
callback(null, null);
|
||||
});
|
||||
|
||||
// Act & Assert
|
||||
await expect(workReportService.getWorkReportByIdService(999))
|
||||
.rejects.toThrow(NotFoundError);
|
||||
|
||||
@@ -123,6 +161,7 @@ describe('WorkReportService', () => {
|
||||
});
|
||||
|
||||
it('ID가 없으면 ValidationError를 던져야 함', async () => {
|
||||
// Act & Assert
|
||||
await expect(workReportService.getWorkReportByIdService(null))
|
||||
.rejects.toThrow(ValidationError);
|
||||
});
|
||||
@@ -130,24 +169,38 @@ describe('WorkReportService', () => {
|
||||
|
||||
describe('updateWorkReportService', () => {
|
||||
it('보고서를 성공적으로 수정해야 함', async () => {
|
||||
// Arrange
|
||||
const updateData = { work_hours: 9 };
|
||||
|
||||
workReportModel.update.mockResolvedValue(1);
|
||||
workReportModel.update = jest.fn((id, data, callback) => {
|
||||
callback(null, 1); // affectedRows = 1
|
||||
});
|
||||
|
||||
// Act
|
||||
const result = await workReportService.updateWorkReportService(123, updateData);
|
||||
|
||||
// Assert
|
||||
expect(result).toEqual({ changes: 1 });
|
||||
expect(workReportModel.update).toHaveBeenCalledWith(123, updateData);
|
||||
expect(workReportModel.update).toHaveBeenCalledWith(
|
||||
123,
|
||||
updateData,
|
||||
expect.any(Function)
|
||||
);
|
||||
});
|
||||
|
||||
it('수정할 보고서가 없으면 NotFoundError를 던져야 함', async () => {
|
||||
workReportModel.update.mockResolvedValue(0);
|
||||
// Arrange
|
||||
workReportModel.update = jest.fn((id, data, callback) => {
|
||||
callback(null, 0); // affectedRows = 0
|
||||
});
|
||||
|
||||
// Act & Assert
|
||||
await expect(workReportService.updateWorkReportService(999, {}))
|
||||
.rejects.toThrow(NotFoundError);
|
||||
});
|
||||
|
||||
it('ID가 없으면 ValidationError를 던져야 함', async () => {
|
||||
// Act & Assert
|
||||
await expect(workReportService.updateWorkReportService(null, {}))
|
||||
.rejects.toThrow(ValidationError);
|
||||
});
|
||||
@@ -155,22 +208,32 @@ describe('WorkReportService', () => {
|
||||
|
||||
describe('removeWorkReportService', () => {
|
||||
it('보고서를 성공적으로 삭제해야 함', async () => {
|
||||
workReportModel.remove.mockResolvedValue(1);
|
||||
// Arrange
|
||||
workReportModel.remove = jest.fn((id, callback) => {
|
||||
callback(null, 1);
|
||||
});
|
||||
|
||||
// Act
|
||||
const result = await workReportService.removeWorkReportService(123);
|
||||
|
||||
// Assert
|
||||
expect(result).toEqual({ changes: 1 });
|
||||
expect(workReportModel.remove).toHaveBeenCalledWith(123);
|
||||
expect(workReportModel.remove).toHaveBeenCalledWith(123, expect.any(Function));
|
||||
});
|
||||
|
||||
it('삭제할 보고서가 없으면 NotFoundError를 던져야 함', async () => {
|
||||
workReportModel.remove.mockResolvedValue(0);
|
||||
// Arrange
|
||||
workReportModel.remove = jest.fn((id, callback) => {
|
||||
callback(null, 0);
|
||||
});
|
||||
|
||||
// Act & Assert
|
||||
await expect(workReportService.removeWorkReportService(999))
|
||||
.rejects.toThrow(NotFoundError);
|
||||
});
|
||||
|
||||
it('ID가 없으면 ValidationError를 던져야 함', async () => {
|
||||
// Act & Assert
|
||||
await expect(workReportService.removeWorkReportService(null))
|
||||
.rejects.toThrow(ValidationError);
|
||||
});
|
||||
@@ -178,23 +241,34 @@ describe('WorkReportService', () => {
|
||||
|
||||
describe('getWorkReportsInRangeService', () => {
|
||||
it('기간으로 보고서를 조회해야 함', async () => {
|
||||
// Arrange
|
||||
const start = '2025-12-01';
|
||||
const end = '2025-12-31';
|
||||
|
||||
workReportModel.getByRange.mockResolvedValue(mockWorkReports);
|
||||
workReportModel.getByRange = jest.fn((start, end, callback) => {
|
||||
callback(null, mockWorkReports);
|
||||
});
|
||||
|
||||
// Act
|
||||
const result = await workReportService.getWorkReportsInRangeService(start, end);
|
||||
|
||||
// Assert
|
||||
expect(result).toEqual(mockWorkReports);
|
||||
expect(workReportModel.getByRange).toHaveBeenCalledWith(start, end);
|
||||
expect(workReportModel.getByRange).toHaveBeenCalledWith(
|
||||
start,
|
||||
end,
|
||||
expect.any(Function)
|
||||
);
|
||||
});
|
||||
|
||||
it('시작일이 없으면 ValidationError를 던져야 함', async () => {
|
||||
// Act & Assert
|
||||
await expect(workReportService.getWorkReportsInRangeService(null, '2025-12-31'))
|
||||
.rejects.toThrow(ValidationError);
|
||||
});
|
||||
|
||||
it('종료일이 없으면 ValidationError를 던져야 함', async () => {
|
||||
// Act & Assert
|
||||
await expect(workReportService.getWorkReportsInRangeService('2025-12-01', null))
|
||||
.rejects.toThrow(ValidationError);
|
||||
});
|
||||
@@ -202,30 +276,41 @@ describe('WorkReportService', () => {
|
||||
|
||||
describe('getSummaryService', () => {
|
||||
it('월간 요약을 조회해야 함', async () => {
|
||||
// Arrange
|
||||
const year = '2025';
|
||||
const month = '12';
|
||||
|
||||
workReportModel.getByRange.mockResolvedValue(mockWorkReports);
|
||||
workReportModel.getByRange = jest.fn((start, end, callback) => {
|
||||
callback(null, mockWorkReports);
|
||||
});
|
||||
|
||||
// Act
|
||||
const result = await workReportService.getSummaryService(year, month);
|
||||
|
||||
// Assert
|
||||
expect(result).toEqual(mockWorkReports);
|
||||
expect(workReportModel.getByRange).toHaveBeenCalled();
|
||||
});
|
||||
|
||||
it('연도가 없으면 ValidationError를 던져야 함', async () => {
|
||||
// Act & Assert
|
||||
await expect(workReportService.getSummaryService(null, '12'))
|
||||
.rejects.toThrow(ValidationError);
|
||||
});
|
||||
|
||||
it('월이 없으면 ValidationError를 던져야 함', async () => {
|
||||
// Act & Assert
|
||||
await expect(workReportService.getSummaryService('2025', null))
|
||||
.rejects.toThrow(ValidationError);
|
||||
});
|
||||
|
||||
it('데이터가 없으면 NotFoundError를 던져야 함', async () => {
|
||||
workReportModel.getByRange.mockResolvedValue([]);
|
||||
// Arrange
|
||||
workReportModel.getByRange = jest.fn((start, end, callback) => {
|
||||
callback(null, []);
|
||||
});
|
||||
|
||||
// Act & Assert
|
||||
await expect(workReportService.getSummaryService('2025', '12'))
|
||||
.rejects.toThrow(NotFoundError);
|
||||
|
||||
|
||||
Reference in New Issue
Block a user