Refactor for fun and profit

- attach/detach takes on_success|failure block
- removes double from Cli spec, use Fog mocks