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
Add CompanyServiceImpl Test #149
Conversation
Hi @bglamadrid, Please check my changes in Changelog.md if its correct format |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Outstanding work @NyorJa 💯
I do have one minor concern, a single comment below that you may address if possible. Besides that, this is exactly what was required 👌
I will merge this PR perhaps before 6PM UTC, but not later than that.
Thank you once again!
CHANGELOG.md
Outdated
### Tests | ||
- CompanyServiceImpl - validate the proper mapping behaviour of method "readDetails" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a Tests
section at line 34, you can move this change there
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops..ill fix that.
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
PR Checklist
mvn test
returns exit code 0, without errors)PR Type
Summary
Validate the proper mapping behaviour of CompanyServiceImpl method "readDetails"