tree 7827534424dbf5b4e89b952391795513626460f9
parent ae4ee2acf4a72c508d079ccd9666a64e8bcf6dae
author Christoforos Miliotis <christoforos.miliotis.dev@gmail.com> 1682614655 +0100
committer Christoforos Miliotis <christoforos.miliotis.dev@gmail.com> 1683121392 +0100

Extract the repository deletion logic so it becomes reusable

The code that deletes a repository is generally quite robust and
as such it will be very useful for the pull-replication plugin.
However, the logic is heavily baked into the plugin itself;
it requires archiving settings read from the plugin config, and
deletion listeners to notify downstream systems - both are not
relevant to the pull-replication plugin (or anything else that
might want to just delete a repo).

Extract the repo deletion logic into a `RepositoryDelete` class,
free from any nuances of the delete-project plugin itself, so it
can be reused. The `FilesystemDeleteHandler` is now essentially
delegating everything into that class. I've also taken the
liberty to add a couple of missing tests for the notifying of
downstream systems through the deletion listeners.

Bug: Issue 16730
Change-Id: Ib28dbed344ef8a419cf7c318d0e87aadd7dadd5e