Merge "Cinder: Correct exception logging during attach"

1 files changed, 9 insertions, 0 deletions

diff --git a/glance_store/tests/unit/common/test_attachment_state_manager.py b/glance_store/tests/unit/common/test_attachment_state_manager.py
index f019c9b..d8c5189 100644
--- a/glance_store/tests/unit/common/test_attachment_state_manager.py
+++ b/glance_store/tests/unit/common/test_attachment_state_manager.py
@@ -18,6 +18,7 @@ from unittest import mock
 from oslo_config import cfg
 from oslotest import base
 
+from cinderclient import exceptions as cinder_exception
 from glance_store.common import attachment_state_manager as attach_manager
 from glance_store.common import cinder_utils
 from glance_store import exceptions
@@ -106,3 +107,11 @@ class AttachmentStateTestCase(base.BaseTestCase):
             *self.disconnect_vol_call)
         mock_attach_delete.assert_called_once_with(
             *self.detach_call)
+
+    @mock.patch.object(cinder_utils.API, 'attachment_create')
+    def test_attach_fails(self, mock_attach_create):
+        mock_attach_create.side_effect = cinder_exception.BadRequest(code=400)
+        self.assertRaises(
+            cinder_exception.BadRequest, self.m.attach,
+            mock.sentinel.client, mock.sentinel.volume_id,
+            mock.sentinel.host, mode=mock.sentinel.mode)