feat: Add comprehensive unit tests - #1229
Conversation
|
@sadeeq6400 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
|
@sadeeq6400 is attempting to deploy a commit to the Mftee's projects Team on Vercel. A member of the Team first needs to authorize it. |
mftee
left a comment
There was a problem hiding this comment.
@sadeeq6400 This PR currently has a merge conflict with the main branch (mergeable_state: dirty), so it can't be merged as-is. Could you please rebase or merge main into your branch and resolve the conflicting files? Requesting changes until this is resolved.
mftee
left a comment
There was a problem hiding this comment.
Conflict has been resolved and the branch is clean against main now. The comprehensive unit test additions look good - approving.
Description:
This pull request introduces a comprehensive suite of unit tests for the dispute, external-validation, queue, and access-logs modules. The tests cover a wide range of scenarios, including service logic, controller behavior, and entity interactions.
Key Features:
Dispute Module: Added unit tests for the DisputeService and DisputeController, ensuring that dispute creation, lifecycle, and access control are working as expected.
Closes [BE-123] Unit tests for the dispute module #982
External Validation Module: Implemented unit tests for the ExternalValidationService and ExternalValidationController, covering success and failure paths, as well as resilience.
Closes [BE-124] Unit tests for the external-validation module #983
Queue Module: Added unit tests for the QueueService and DocumentProcessor, verifying that jobs are enqueued and processed correctly.
Closes [BE-125] Unit tests for the queue module #984
Access Logs Module: Expanded test coverage for the AccessLogsService, ensuring that logs are created, queried, and filtered as expected.
Closes [BE-126] Expand access-logs module test coverage #985
Testing: