search

Apress / pivotal-certified-pro-spring-dev-exam-02

Source Code for 'Pivotal Certified Professional Core Spring 5 Developer Exam' by Iuliana Cosmina
Other
168 stars 239 forks source link

Comments over PersonMonitorTest.testSave and PersonMonitorTest.testBadUpdate can be more precise #12

Closed A60678 closed 4 years ago

A60678 commented 4 years ago

A) I would suggest the following adjustments of the comments, as TODO 21 - is about @Before.repoFind TODO 22 - is about @After.serviceFind || @After.repoFind TODO 23 - is about @AfterReturing.Service.save TODO 24 - is about @AfterThrowing.Repo.update TODO 25 - is about @Around.serviceFind || @Around.repoFind and the method testSave is calling only personService.save and the methdod testBadUpdate is calling personRepo.findById and personService.updateFirstName

B) Missing keyword TODO