Skip to content

Commit 255724f

Browse files
committed
Document update
Use `filesize` in create body instead of `length`
1 parent 096daf5 commit 255724f

5 files changed

Lines changed: 11 additions & 11 deletions

File tree

app/src/controllers/document.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ export const createDocumentController = async (
1111
req: Request<
1212
never,
1313
never,
14-
{ documentId: string; activityId: string; filename: string; mimeType: string; length: number }
14+
{ documentId: string; activityId: string; filename: string; mimeType: string; filesize: number }
1515
>,
1616
res: Response
1717
) => {
@@ -22,7 +22,7 @@ export const createDocumentController = async (
2222
req.body.activityId,
2323
req.body.filename,
2424
req.body.mimeType,
25-
req.body.length,
25+
req.body.filesize,
2626
generateCreateStamps(req.currentContext)
2727
);
2828

app/src/validators/document.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ const schema = {
1010
documentId: uuidv4.required(),
1111
filename: Joi.string().max(255).required(),
1212
mimeType: Joi.string().min(0).max(255),
13-
length: Joi.number().required()
13+
filesize: Joi.number().required()
1414
})
1515
},
1616
deleteDocument: {

app/tests/unit/controllers/document.spec.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ describe('createDocumentController', () => {
4141
activityId: 'ACTI1234',
4242
filename: 'testfile',
4343
mimeType: 'imgjpg',
44-
length: 1234567
44+
filesize: 1234567
4545
},
4646
currentContext: TEST_CURRENT_CONTEXT
4747
};
@@ -52,7 +52,7 @@ describe('createDocumentController', () => {
5252
req as unknown as Request<
5353
never,
5454
never,
55-
{ documentId: string; activityId: string; filename: string; mimeType: string; length: number }
55+
{ documentId: string; activityId: string; filename: string; mimeType: string; filesize: number }
5656
>,
5757
res as unknown as Response
5858
);
@@ -64,7 +64,7 @@ describe('createDocumentController', () => {
6464
req.body.activityId,
6565
req.body.filename,
6666
req.body.mimeType,
67-
req.body.length,
67+
req.body.filesize,
6868
{ createdAt: expect.any(Date), createdBy: TEST_CURRENT_CONTEXT.userId }
6969
);
7070
expect(res.status).toHaveBeenCalledWith(201);
@@ -78,7 +78,7 @@ describe('createDocumentController', () => {
7878
activityId: 'ACTI1234',
7979
filename: 'testfile',
8080
mimeType: 'imgjpg',
81-
length: 1234567
81+
filesize: 1234567
8282
},
8383
currentContext: TEST_CURRENT_CONTEXT
8484
};
@@ -92,7 +92,7 @@ describe('createDocumentController', () => {
9292
req as unknown as Request<
9393
never,
9494
never,
95-
{ documentId: string; activityId: string; filename: string; mimeType: string; length: number }
95+
{ documentId: string; activityId: string; filename: string; mimeType: string; filesize: number }
9696
>,
9797
res as unknown as Response
9898
);
@@ -104,7 +104,7 @@ describe('createDocumentController', () => {
104104
req.body.activityId,
105105
req.body.filename,
106106
req.body.mimeType,
107-
req.body.length,
107+
req.body.filesize,
108108
{ createdAt: expect.any(Date), createdBy: TEST_CURRENT_CONTEXT.userId }
109109
);
110110
expect(readUserSpy).toHaveBeenCalledTimes(1);

frontend/src/services/documentService.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ export default {
4242
documentId: comsResponse.data.id,
4343
filename: comsResponse.data.name,
4444
mimeType: comsResponse.data.mimeType,
45-
length: comsResponse.data.length
45+
filesize: comsResponse.data.length
4646
});
4747
} catch (e: any) {
4848
// In event of any error try to Delete COMS object if it was created

frontend/tests/unit/service/documentService.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ describe('documentService', () => {
120120
documentId: testFileData.id,
121121
filename: testFileData.name,
122122
mimeType: testFileData.mimeType,
123-
length: testFileData.length
123+
filesize: testFileData.length
124124
});
125125
});
126126

0 commit comments

Comments
 (0)