OK, so I'm half wrong, half right. Any `AttachmentData` loaded directly from the database will report a clean path, regardless of where the file actually is. But because of a quirk in the way that `Whitehall::QuarantinedFileStorage` works, newly created  `AttachmentData` will report the incoming path until they are reloaded. This is because the `store!` method has the side effect of caching the path as the incoming location. This will mean that some of your tests may incorrectly report the incoming path when you are working with instances of newly created `AttachmentData`. I think it's probably better to reload these particular attachment_data instances where required so that it will better reflect what the system will see in real world use, rather than have this conditional here to work around it.