-
Notifications
You must be signed in to change notification settings - Fork 28
Tests/controller MMU-API micro services ( Completed . Ready to Merge) #113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
@vanitha1822 can u guide me on what should I do about the merge conflicts they are minor changes on the controller , for |
@Zapper9982 Do the 'Sync repo' from your forked repo and then take the latest pull from release-3.4.0 and resolve them in merge editor. It is easy for you to identify the changes. |
@Zapper9982 please resolve the conflicts |
sure will do it by tonight . |
32d0097
to
c35ab71
Compare
|
PR: Complete Controller Test Coverage for MMU-API
📋 Description
JIRA ID:
This PR adds complete unit test coverage for all controller classes in the MMU-API project. The motivation behind this change is to ensure robust validation of all controller endpoints, covering all logic branches, error handling, and exception scenarios. This improves code reliability, maintainability, and confidence in future changes.
✅ Type of Change
ℹ️ Additional Information
All controller methods are now covered by comprehensive unit tests using MockMvc and Mockito. Tests include all success, error, and exception branches. The test suite passes with 100% coverage for controller logic.
No production code was modified—only test files were added or updated.
See test run output and coverage reports for details.
Screenshots
Build Success
JaCoCo Report
@drtechie @vanitha1822