You are mocking out `detect_block_device`. This is a pretty integral part and I don't see tests for it. I would not recommend adding specific tests for that. I have seen the agent go into an infinite loop if there is a problem in detecting the scsi device.