np-dms/lcbp3
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

260323:0954 fix CI : Run Tests frontend #01
CI / CD Pipeline / build (push) Failing after 15m27s
Details
CI / CD Pipeline / release (push) Has been skipped
Details
CI / CD Pipeline / deploy (push) Has been skipped
Details

Browse Source
This commit is contained in:
admin
2026-03-23 09:54:31 +07:00
parent 4422c68894
commit 32141f519a
8 changed files with 563 additions and 1073 deletions
Download Patch File Download Diff File Expand all files Collapse all files
frontend/hooks/__tests__/use-correspondence.test.ts
+19 -19
View File
@@ -18,7 +18,7 @@ import { toast } from 'sonner';
vi.mock('@/lib/services/correspondence.service', () => ({
correspondenceService: {
getAll: vi.fn(),
getById: vi.fn(),
getByUuid: vi.fn(),
create: vi.fn(),
update: vi.fn(),
delete: vi.fn(),
@@ -38,7 +38,7 @@ describe('use-correspondence hooks', () => {
expect(correspondenceKeys.lists()).toEqual(['correspondences', 'list']);
expect(correspondenceKeys.list({ projectId: 1 })).toEqual(['correspondences', 'list', { projectId: 1 }]);
expect(correspondenceKeys.details()).toEqual(['correspondences', 'detail']);
expect(correspondenceKeys.detail(1)).toEqual(['correspondences', 'detail', 1]);
expect(correspondenceKeys.detail('uuid-1')).toEqual(['correspondences', 'detail', 'uuid-1']);
});
});
@@ -81,27 +81,27 @@ describe('use-correspondence hooks', () => {
});
describe('useCorrespondence', () => {
it('should fetch single correspondence by id', async () => {
const mockData = { id: 1, title: 'Test Correspondence' };
vi.mocked(correspondenceService.getById).mockResolvedValue(mockData);
it('should fetch single correspondence by uuid', async () => {
const mockData = { id: 1, uuid: 'uuid-1', title: 'Test Correspondence' };
vi.mocked(correspondenceService.getByUuid).mockResolvedValue(mockData);
const { wrapper } = createTestQueryClient();
const { result } = renderHook(() => useCorrespondence(1), { wrapper });
const { result } = renderHook(() => useCorrespondence('uuid-1'), { wrapper });
await waitFor(() => {
expect(result.current.isSuccess).toBe(true);
});
expect(result.current.data).toEqual(mockData);
expect(correspondenceService.getById).toHaveBeenCalledWith(1);
expect(correspondenceService.getByUuid).toHaveBeenCalledWith('uuid-1');
});
it('should not fetch when id is falsy', () => {
it('should not fetch when uuid is falsy', () => {
const { wrapper } = createTestQueryClient();
const { result } = renderHook(() => useCorrespondence(0), { wrapper });
const { result } = renderHook(() => useCorrespondence(''), { wrapper });
expect(result.current.isFetching).toBe(false);
expect(correspondenceService.getById).not.toHaveBeenCalled();
expect(correspondenceService.getByUuid).not.toHaveBeenCalled();
});
});
@@ -173,12 +173,12 @@ describe('use-correspondence hooks', () => {
await act(async () => {
await result.current.mutateAsync({
id: 1,
uuid: 'uuid-1',
data: { subject: 'Updated Correspondence' },
});
});
expect(correspondenceService.update).toHaveBeenCalledWith(1, {
expect(correspondenceService.update).toHaveBeenCalledWith('uuid-1', {
subject: 'Updated Correspondence',
});
expect(toast.success).toHaveBeenCalledWith('Correspondence updated successfully');
@@ -193,10 +193,10 @@ describe('use-correspondence hooks', () => {
const { result } = renderHook(() => useDeleteCorrespondence(), { wrapper });
await act(async () => {
await result.current.mutateAsync(1);
await result.current.mutateAsync('uuid-1');
});
expect(correspondenceService.delete).toHaveBeenCalledWith(1);
expect(correspondenceService.delete).toHaveBeenCalledWith('uuid-1');
expect(toast.success).toHaveBeenCalledWith('Correspondence deleted successfully');
});
});
@@ -211,12 +211,12 @@ describe('use-correspondence hooks', () => {
await act(async () => {
await result.current.mutateAsync({
id: 1,
uuid: 'uuid-1',
data: { note: 'Ready for review' },
});
});
expect(correspondenceService.submit).toHaveBeenCalledWith(1, { note: 'Ready for review' });
expect(correspondenceService.submit).toHaveBeenCalledWith('uuid-1', { note: 'Ready for review' });
expect(toast.success).toHaveBeenCalledWith('Correspondence submitted successfully');
});
});
@@ -231,12 +231,12 @@ describe('use-correspondence hooks', () => {
await act(async () => {
await result.current.mutateAsync({
id: 1,
uuid: 'uuid-1',
data: { action: 'APPROVE', comments: 'LGTM' },
});
});
expect(correspondenceService.processWorkflow).toHaveBeenCalledWith(1, {
expect(correspondenceService.processWorkflow).toHaveBeenCalledWith('uuid-1', {
action: 'APPROVE',
comments: 'LGTM',
});
@@ -256,7 +256,7 @@ describe('use-correspondence hooks', () => {
await act(async () => {
try {
await result.current.mutateAsync({
id: 1,
uuid: 'uuid-1',
data: { action: 'APPROVE' },
});
} catch {
frontend/hooks/__tests__/use-drawing.test.ts
+29 -21
View File
@@ -10,7 +10,7 @@ import { toast } from 'sonner';
vi.mock('@/lib/services/contract-drawing.service', () => ({
contractDrawingService: {
getAll: vi.fn(),
getById: vi.fn(),
getByUuid: vi.fn(),
create: vi.fn(),
},
}));
@@ -18,7 +18,7 @@ vi.mock('@/lib/services/contract-drawing.service', () => ({
vi.mock('@/lib/services/shop-drawing.service', () => ({
shopDrawingService: {
getAll: vi.fn(),
getById: vi.fn(),
getByUuid: vi.fn(),
create: vi.fn(),
},
}));
@@ -38,7 +38,7 @@ describe('use-drawing hooks', () => {
'CONTRACT',
{ projectId: 1 },
]);
expect(drawingKeys.detail('SHOP', 1)).toEqual(['drawings', 'detail', 'SHOP', 1]);
expect(drawingKeys.detail('SHOP', 'uuid-1')).toEqual(['drawings', 'detail', 'SHOP', 'uuid-1']);
});
});
@@ -46,8 +46,8 @@ describe('use-drawing hooks', () => {
it('should fetch CONTRACT drawings successfully', async () => {
const mockData = {
data: [
{ id: 1, drawingNumber: 'CD-001' },
{ id: 2, drawingNumber: 'CD-002' },
{ id: 1, uuid: 'uuid-1', contractDrawingNo: 'CD-001' },
{ id: 2, uuid: 'uuid-2', contractDrawingNo: 'CD-002' },
],
meta: { total: 2, page: 1, limit: 10 },
};
@@ -61,14 +61,18 @@ describe('use-drawing hooks', () => {
expect(result.current.isSuccess).toBe(true);
});
expect(result.current.data).toEqual(mockData);
expect(result.current.data.data[0]).toMatchObject({
uuid: 'uuid-1',
drawingNumber: 'CD-001',
type: 'CONTRACT',
});
expect(contractDrawingService.getAll).toHaveBeenCalledWith({ projectId: 1 });
expect(shopDrawingService.getAll).not.toHaveBeenCalled();
});
it('should fetch SHOP drawings successfully', async () => {
const mockData = {
data: [{ id: 1, drawingNumber: 'SD-001' }],
data: [{ id: 1, uuid: 'uuid-1' }],
meta: { total: 1, page: 1, limit: 10 },
};
@@ -81,7 +85,11 @@ describe('use-drawing hooks', () => {
expect(result.current.isSuccess).toBe(true);
});
expect(result.current.data).toEqual(mockData);
expect(result.current.data.data[0]).toMatchObject({
uuid: 'uuid-1',
type: 'SHOP',
title: 'Untitled',
});
expect(shopDrawingService.getAll).toHaveBeenCalledWith({ projectId: 1 });
expect(contractDrawingService.getAll).not.toHaveBeenCalled();
});
@@ -100,42 +108,42 @@ describe('use-drawing hooks', () => {
});
describe('useDrawing', () => {
it('should fetch single CONTRACT drawing by id', async () => {
const mockData = { id: 1, drawingNumber: 'CD-001' };
vi.mocked(contractDrawingService.getById).mockResolvedValue(mockData);
it('should fetch single CONTRACT drawing by uuid', async () => {
const mockData = { id: 1, uuid: 'uuid-1', contractDrawingNo: 'CD-001' };
vi.mocked(contractDrawingService.getByUuid).mockResolvedValue(mockData);
const { wrapper } = createTestQueryClient();
const { result } = renderHook(() => useDrawing('CONTRACT', 1), { wrapper });
const { result } = renderHook(() => useDrawing('CONTRACT', 'uuid-1'), { wrapper });
await waitFor(() => {
expect(result.current.isSuccess).toBe(true);
});
expect(result.current.data).toEqual(mockData);
expect(contractDrawingService.getById).toHaveBeenCalledWith(1);
expect(contractDrawingService.getByUuid).toHaveBeenCalledWith('uuid-1');
});
it('should fetch single SHOP drawing by id', async () => {
const mockData = { id: 1, drawingNumber: 'SD-001' };
vi.mocked(shopDrawingService.getById).mockResolvedValue(mockData);
it('should fetch single SHOP drawing by uuid', async () => {
const mockData = { id: 1, uuid: 'uuid-1', title: 'SD-001' };
vi.mocked(shopDrawingService.getByUuid).mockResolvedValue(mockData);
const { wrapper } = createTestQueryClient();
const { result } = renderHook(() => useDrawing('SHOP', 1), { wrapper });
const { result } = renderHook(() => useDrawing('SHOP', 'uuid-1'), { wrapper });
await waitFor(() => {
expect(result.current.isSuccess).toBe(true);
});
expect(result.current.data).toEqual(mockData);
expect(shopDrawingService.getById).toHaveBeenCalledWith(1);
expect(shopDrawingService.getByUuid).toHaveBeenCalledWith('uuid-1');
});
it('should not fetch when id is falsy', () => {
it('should not fetch when uuid is falsy', () => {
const { wrapper } = createTestQueryClient();
const { result } = renderHook(() => useDrawing('CONTRACT', 0), { wrapper });
const { result } = renderHook(() => useDrawing('CONTRACT', ''), { wrapper });
expect(result.current.isFetching).toBe(false);
expect(contractDrawingService.getById).not.toHaveBeenCalled();
expect(contractDrawingService.getByUuid).not.toHaveBeenCalled();
});
});
frontend/hooks/__tests__/use-projects.test.ts
+8 -8
View File
@@ -9,7 +9,7 @@ import { toast } from 'sonner';
vi.mock('@/lib/services/project.service', () => ({
projectService: {
getAll: vi.fn(),
getById: vi.fn(),
getByUuid: vi.fn(),
create: vi.fn(),
update: vi.fn(),
delete: vi.fn(),
@@ -25,7 +25,7 @@ describe('use-projects hooks', () => {
it('should generate correct cache keys', () => {
expect(projectKeys.all).toEqual(['projects']);
expect(projectKeys.list({ search: 'test' })).toEqual(['projects', 'list', { search: 'test' }]);
expect(projectKeys.detail(1)).toEqual(['projects', 'detail', 1]);
expect(projectKeys.detail('uuid-1')).toEqual(['projects', 'detail', 'uuid-1']);
});
});
@@ -136,12 +136,12 @@ describe('use-projects hooks', () => {
await act(async () => {
await result.current.mutateAsync({
id: 1,
uuid: 'uuid-1',
data: { name: 'Updated Project' },
});
});
expect(projectService.update).toHaveBeenCalledWith(1, { name: 'Updated Project' });
expect(projectService.update).toHaveBeenCalledWith('uuid-1', { name: 'Updated Project' });
expect(toast.success).toHaveBeenCalledWith('Project updated successfully');
});
@@ -158,7 +158,7 @@ describe('use-projects hooks', () => {
await act(async () => {
try {
await result.current.mutateAsync({
id: 999,
uuid: 'uuid-999',
data: { name: 'Test' },
});
} catch {
@@ -180,10 +180,10 @@ describe('use-projects hooks', () => {
const { result } = renderHook(() => useDeleteProject(), { wrapper });
await act(async () => {
await result.current.mutateAsync(1);
await result.current.mutateAsync('uuid-1');
});
expect(projectService.delete).toHaveBeenCalledWith(1);
expect(projectService.delete).toHaveBeenCalledWith('uuid-1');
expect(toast.success).toHaveBeenCalledWith('Project deleted successfully');
});
@@ -199,7 +199,7 @@ describe('use-projects hooks', () => {
await act(async () => {
try {
await result.current.mutateAsync(1);
await result.current.mutateAsync('uuid-1');
} catch {
// Expected
}
frontend/hooks/__tests__/use-rfa.test.ts
+14 -14
View File
@@ -9,7 +9,7 @@ import { toast } from 'sonner';
vi.mock('@/lib/services/rfa.service', () => ({
rfaService: {
getAll: vi.fn(),
getById: vi.fn(),
getByUuid: vi.fn(),
create: vi.fn(),
update: vi.fn(),
processWorkflow: vi.fn(),
@@ -27,7 +27,7 @@ describe('use-rfa hooks', () => {
expect(rfaKeys.lists()).toEqual(['rfas', 'list']);
expect(rfaKeys.list({ projectId: 1 })).toEqual(['rfas', 'list', { projectId: 1 }]);
expect(rfaKeys.details()).toEqual(['rfas', 'detail']);
expect(rfaKeys.detail(1)).toEqual(['rfas', 'detail', 1]);
expect(rfaKeys.detail('uuid-1')).toEqual(['rfas', 'detail', 'uuid-1']);
});
});
@@ -67,27 +67,27 @@ describe('use-rfa hooks', () => {
});
describe('useRFA', () => {
it('should fetch single RFA by id', async () => {
const mockData = { id: 1, rfaNumber: 'RFA-001', status: 'pending' };
vi.mocked(rfaService.getById).mockResolvedValue(mockData);
it('should fetch single RFA by uuid', async () => {
const mockData = { id: 1, uuid: 'uuid-1', rfaNumber: 'RFA-001', status: 'pending' };
vi.mocked(rfaService.getByUuid).mockResolvedValue(mockData);
const { wrapper } = createTestQueryClient();
const { result } = renderHook(() => useRFA(1), { wrapper });
const { result } = renderHook(() => useRFA('uuid-1'), { wrapper });
await waitFor(() => {
expect(result.current.isSuccess).toBe(true);
});
expect(result.current.data).toEqual(mockData);
expect(rfaService.getById).toHaveBeenCalledWith(1);
expect(rfaService.getByUuid).toHaveBeenCalledWith('uuid-1');
});
it('should not fetch when id is falsy', () => {
it('should not fetch when uuid is falsy', () => {
const { wrapper } = createTestQueryClient();
const { result } = renderHook(() => useRFA(0), { wrapper });
const { result } = renderHook(() => useRFA(''), { wrapper });
expect(result.current.isFetching).toBe(false);
expect(rfaService.getById).not.toHaveBeenCalled();
expect(rfaService.getByUuid).not.toHaveBeenCalled();
});
});
@@ -151,12 +151,12 @@ describe('use-rfa hooks', () => {
await act(async () => {
await result.current.mutateAsync({
id: 1,
uuid: 'uuid-1',
data: { subject: 'Updated RFA' },
});
});
expect(rfaService.update).toHaveBeenCalledWith(1, { subject: 'Updated RFA' });
expect(rfaService.update).toHaveBeenCalledWith('uuid-1', { subject: 'Updated RFA' });
expect(toast.success).toHaveBeenCalledWith('RFA updated successfully');
});
});
@@ -171,12 +171,12 @@ describe('use-rfa hooks', () => {
await act(async () => {
await result.current.mutateAsync({
id: 1,
uuid: 'uuid-1',
data: { action: 'APPROVE', comments: 'Approved' },
});
});
expect(rfaService.processWorkflow).toHaveBeenCalledWith(1, {
expect(rfaService.processWorkflow).toHaveBeenCalledWith('uuid-1', {
action: 'APPROVE',
comments: 'Approved',
});
frontend/hooks/__tests__/use-users.test.ts
+8 -8
View File
@@ -9,7 +9,7 @@ import { toast } from 'sonner';
vi.mock('@/lib/services/user.service', () => ({
userService: {
getAll: vi.fn(),
getById: vi.fn(),
getByUuid: vi.fn(),
create: vi.fn(),
update: vi.fn(),
delete: vi.fn(),
@@ -26,7 +26,7 @@ describe('use-users hooks', () => {
it('should generate correct cache keys', () => {
expect(userKeys.all).toEqual(['users']);
expect(userKeys.list({ search: 'john' })).toEqual(['users', 'list', { search: 'john' }]);
expect(userKeys.detail(1)).toEqual(['users', 'detail', 1]);
expect(userKeys.detail('uuid-1')).toEqual(['users', 'detail', 'uuid-1']);
});
});
@@ -146,12 +146,12 @@ describe('use-users hooks', () => {
await act(async () => {
await result.current.mutateAsync({
id: 1,
uuid: 'uuid-1',
data: { email: 'updated@example.com' },
});
});
expect(userService.update).toHaveBeenCalledWith(1, { email: 'updated@example.com' });
expect(userService.update).toHaveBeenCalledWith('uuid-1', { email: 'updated@example.com' });
expect(toast.success).toHaveBeenCalledWith('User updated successfully');
});
@@ -168,7 +168,7 @@ describe('use-users hooks', () => {
await act(async () => {
try {
await result.current.mutateAsync({
id: 999,
uuid: 'uuid-999',
data: { email: 'test@example.com' },
});
} catch {
@@ -190,10 +190,10 @@ describe('use-users hooks', () => {
const { result } = renderHook(() => useDeleteUser(), { wrapper });
await act(async () => {
await result.current.mutateAsync(1);
await result.current.mutateAsync('uuid-1');
});
expect(userService.delete).toHaveBeenCalledWith(1);
expect(userService.delete).toHaveBeenCalledWith('uuid-1');
expect(toast.success).toHaveBeenCalledWith('User deleted successfully');
});
@@ -209,7 +209,7 @@ describe('use-users hooks', () => {
await act(async () => {
try {
await result.current.mutateAsync(1);
await result.current.mutateAsync('uuid-1');
} catch {
// Expected
}