PersonServiceImpl.voidRelationship currently looks like
due to the BaseVoidHandler which is being called via AOP the Relationship is already voided, therefore the guard clause at the top of the voidRelationship method exits.
This also explains why the coverage of the method is low despite 3 tests aimed at testing the different branches in this method.
I suggest we remove all the code except the return Context.getPersonService().saveRelationship(relationship). This would make the implementation equal to voidPersonAddress, voidPersonName. Remaining question, what if given null? voidPerson then returns null, the other two methods dont.