tree c60472ef3029d0f1be1857cf47605529c56d5d9f
parent b85642e5ce94dbe5cae7061dd56770b7d0b86d2d
author Hector Oswaldo Caballero <hector.caballero@ericsson.com> 1521990159 -0400
committer Hector Oswaldo Caballero <hector.caballero@ericsson.com> 1535626668 -0400

Extract deletion preconditions to its own class

Instead of spreading the assertions done before deleting a change in
several classes, extract them in a dedicated class, DeletePreconditions.
This eases the unit testing of the now simplified classes and better
reflects the fact that some of these assertions do not belong anymore to
the class where they were hosted. One example: the check for submodules
was in the DatabaseDeleteHandler class because this information used to
be in the database, but this is no longer the case.

Additional test coverage for this class will be provided in an upcoming
change adding IT tests.

Change-Id: Ie1f513f5981035aba217200f608c6ca6e6357cb1