Skip to content

[PB-5208] test(file): increase test coverage for file module#950

Draft
douglas-xt wants to merge 1 commit intomasterfrom
test/file-test-coverage
Draft

[PB-5208] test(file): increase test coverage for file module#950
douglas-xt wants to merge 1 commit intomasterfrom
test/file-test-coverage

Conversation

@douglas-xt
Copy link
Contributor

Increase file module test coverage by adding tests for uncovered repository, use case and controller methods, aligning mock patterns with the rest of the codebase

@sonarqubecloud
Copy link

@douglas-xt douglas-xt self-assigned this Feb 26, 2026
service.moveFilesToTrash(userMocked, [fileId]),
).resolves.not.toThrow();

await Promise.resolve();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this needed?

Comment on lines +1451 to +1454
const model: FileModel = {
...mockFile,
toJSON: mockFile.toJSON,
} as any;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is repeated a lot, consider refactoring into a helper method

@douglas-xt douglas-xt marked this pull request as draft March 2, 2026 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants