FileStore cleanup should not leak out file handles

FileStore.cleanup() currently returns a list of File instances relying on the caller to remove those files. This breaks encapsulation as the file store is the sole owner of these files and only the file store should be removing them.

I suggest to replace the current cleanup method with one that returns void.