Skip to content

Commit 3752f37

Browse files
vitormattosvitormattos
committed
test(validation): require numeric file status
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
1 parent fe3fb2a commit 3752f37

1 file changed

Lines changed: 22 additions & 22 deletions

File tree

‎src/tests/components/validation/FileStatusList.spec.ts‎

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ type FileStatusConstants = typeof import('../../../constants.js').FILE_STATUS
1717
type FileStatusResponse = {
1818
id: number
1919
name: string
20-
status?: string | number
20+
status: number
2121
size: number
2222
signed?: string
2323
}
@@ -27,9 +27,9 @@ type FileStatusListVm = {
2727
updateTimer: ReturnType<typeof setInterval> | null
2828
$nextTick: () => Promise<void>
2929
loadFiles: () => Promise<void>
30-
getStatusClass: (status: string | number) => string
31-
getStatusLabel: (status: string | number) => string
32-
getStatusIcon: (status: string | number) => string
30+
getStatusClass: (status: number) => string
31+
getStatusLabel: (status: number) => string
32+
getStatusIcon: (status: number) => string
3333
formatDate: (date: string | null) => string
3434
startUpdatePolling: () => void
3535
stopUpdatePolling: () => void
@@ -75,7 +75,7 @@ vi.mock('@nextcloud/axios', () => ({
7575
}))
7676

7777
vi.mock('../../../utils/fileStatus.js', () => ({
78-
getStatusLabel: vi.fn((status: string | number) => {
78+
getStatusLabel: vi.fn((status: number) => {
7979
const labels: Record<string, string> = {
8080
'0': 'Draft',
8181
'1': 'Ready',
@@ -86,7 +86,7 @@ vi.mock('../../../utils/fileStatus.js', () => ({
8686
}
8787
return labels[String(status)] ?? 'Unknown'
8888
}),
89-
getStatusIcon: vi.fn((status: string | number) => 'mdiFileStatus'),
89+
getStatusIcon: vi.fn((status: number) => 'mdiFileStatus'),
9090
}))
9191

9292
vi.mock('../../../constants.js', () => ({
@@ -170,7 +170,7 @@ describe('FileStatusList', () => {
170170
mockAxios.get.mockResolvedValueOnce({
171171
data: {
172172
ocs: {
173-
data: { id: 1, name: 'file1.pdf', size: 0 },
173+
data: { id: 1, name: 'file1.pdf', size: 0, status: FILE_STATUS.DRAFT },
174174
},
175175
},
176176
})
@@ -186,14 +186,14 @@ describe('FileStatusList', () => {
186186
.mockResolvedValueOnce({
187187
data: {
188188
ocs: {
189-
data: { id: 1, name: 'test.pdf', size: 1024, status: '3' },
189+
data: { id: 1, name: 'test.pdf', size: 1024, status: FILE_STATUS.SIGNED },
190190
},
191191
},
192192
})
193193
.mockResolvedValueOnce({
194194
data: {
195195
ocs: {
196-
data: { id: 1, name: 'test.pdf', size: 1024, status: '3' },
196+
data: { id: 1, name: 'test.pdf', size: 1024, status: FILE_STATUS.SIGNED },
197197
},
198198
},
199199
})
@@ -210,28 +210,28 @@ describe('FileStatusList', () => {
210210
.mockResolvedValueOnce({
211211
data: {
212212
ocs: {
213-
data: { id: 1, name: 'file1.pdf', size: 0 },
213+
data: { id: 1, name: 'file1.pdf', size: 0, status: FILE_STATUS.DRAFT },
214214
},
215215
},
216216
})
217217
.mockResolvedValueOnce({
218218
data: {
219219
ocs: {
220-
data: { id: 2, name: 'file2.pdf', size: 0 },
220+
data: { id: 2, name: 'file2.pdf', size: 0, status: FILE_STATUS.SIGNED },
221221
},
222222
},
223223
})
224224
.mockResolvedValueOnce({
225225
data: {
226226
ocs: {
227-
data: { id: 1, name: 'file1.pdf', size: 0 },
227+
data: { id: 1, name: 'file1.pdf', size: 0, status: FILE_STATUS.DRAFT },
228228
},
229229
},
230230
})
231231
.mockResolvedValueOnce({
232232
data: {
233233
ocs: {
234-
data: { id: 2, name: 'file2.pdf', size: 0 },
234+
data: { id: 2, name: 'file2.pdf', size: 0, status: FILE_STATUS.SIGNED },
235235
},
236236
},
237237
})
@@ -246,7 +246,7 @@ describe('FileStatusList', () => {
246246
mockAxios.get.mockResolvedValueOnce({
247247
data: {
248248
ocs: {
249-
data: { id: 1, name: 'test.pdf', size: 0 },
249+
data: { id: 1, name: 'test.pdf', size: 0, status: FILE_STATUS.DRAFT },
250250
},
251251
},
252252
})
@@ -390,7 +390,7 @@ describe('FileStatusList', () => {
390390
wrapper = createWrapper({ fileIds: [1] })
391391

392392
mockAxios.get.mockResolvedValue({
393-
data: { ocs: { data: { id: 1, name: 'test.pdf', size: 0 } } },
393+
data: { ocs: { data: { id: 1, name: 'test.pdf', size: 0, status: FILE_STATUS.DRAFT } } },
394394
})
395395

396396
wrapper.vm.startUpdatePolling()
@@ -433,7 +433,7 @@ describe('FileStatusList', () => {
433433
wrapper = localWrapper
434434

435435
mockAxios.get.mockResolvedValue({
436-
data: { ocs: { data: { id: 1, name: 'test.pdf', size: 0 } } },
436+
data: { ocs: { data: { id: 1, name: 'test.pdf', size: 0, status: FILE_STATUS.DRAFT } } },
437437
})
438438

439439
await localWrapper.setProps({ fileIds: [1] })
@@ -456,7 +456,7 @@ describe('FileStatusList', () => {
456456
wrapper = localWrapper
457457

458458
mockAxios.get.mockResolvedValue({
459-
data: { ocs: { data: { id: 2, name: 'file2.pdf', size: 0 } } },
459+
data: { ocs: { data: { id: 2, name: 'file2.pdf', size: 0, status: FILE_STATUS.SIGNED } } },
460460
})
461461

462462
await localWrapper.setProps({ fileIds: [] })
@@ -469,7 +469,7 @@ describe('FileStatusList', () => {
469469
describe('RULE: mounted lifecycle initializes data loading', () => {
470470
it('loads files on mount when fileIds provided', async () => {
471471
mockAxios.get.mockResolvedValue({
472-
data: { ocs: { data: { id: 1, name: 'test.pdf', size: 0 } } },
472+
data: { ocs: { data: { id: 1, name: 'test.pdf', size: 0, status: FILE_STATUS.DRAFT } } },
473473
})
474474

475475
wrapper = createWrapper({ fileIds: [1] })
@@ -575,16 +575,16 @@ describe('FileStatusList', () => {
575575
it('renders all loaded files', async () => {
576576
mockAxios.get
577577
.mockResolvedValueOnce({
578-
data: { ocs: { data: { id: 1, name: 'file1.pdf', size: 0, status: '0' } } },
578+
data: { ocs: { data: { id: 1, name: 'file1.pdf', size: 0, status: FILE_STATUS.DRAFT } } },
579579
})
580580
.mockResolvedValueOnce({
581-
data: { ocs: { data: { id: 2, name: 'file2.pdf', size: 0, status: '3' } } },
581+
data: { ocs: { data: { id: 2, name: 'file2.pdf', size: 0, status: FILE_STATUS.SIGNED } } },
582582
})
583583
.mockResolvedValueOnce({
584-
data: { ocs: { data: { id: 1, name: 'file1.pdf', size: 0, status: '0' } } },
584+
data: { ocs: { data: { id: 1, name: 'file1.pdf', size: 0, status: FILE_STATUS.DRAFT } } },
585585
})
586586
.mockResolvedValueOnce({
587-
data: { ocs: { data: { id: 2, name: 'file2.pdf', size: 0, status: '3' } } },
587+
data: { ocs: { data: { id: 2, name: 'file2.pdf', size: 0, status: FILE_STATUS.SIGNED } } },
588588
})
589589

590590
wrapper = createWrapper({ fileIds: [1, 2] })

0 commit comments

Comments
 (0)