This PR ensures that we properly handle renaming of directories. I'm keeping it as a DRAFT for a little while longer to extend the testing but also to ensure this is the right approach. I have yet to check whether delimiter vs no delimiter (for S3) is at play here. I was also quite surprised to find directories are not scoped to buckets.
Closing this out. We call RenameObjects (and not RenameObject) if the paths are directories, we do this in the handler which is why I didn't spot it when reproducing it in a unit test.
This PR ensures that we properly handle renaming of directories. I'm keeping it as a
DRAFT
for a little while longer to extend the testing but also to ensure this is the right approach. I have yet to check whether delimiter vs no delimiter (for S3) is at play here. I was also quite surprised to find directories are not scoped to buckets.Fixes #1599