openstack/manila
Code Issues Proposed changes

Merge "HDS_HNAS: Fix improper error message"

Browse Source
This commit is contained in:
Jenkins 2016-05-23 06:04:01 +00:00 committed by Gerrit Code Review
commit 670f1a63f7
2 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
manila/share/drivers/hitachi/hds_hnas.py
View File

@ -607,6 +607,9 @@ class HDSHNASDriver(driver.ShareDriver):
:param share_id: ID of share that snapshot was created.
:param snapshot_id: ID of snapshot.
"""
self._check_fs_mounted()
path = '/snapshots/' + share_id + '/' + snapshot_id
self.hnas.tree_delete(path)
path = '/snapshots/' + share_id

2
manila/tests/share/drivers/hitachi/test_hds_hnas.py
View File

@ -306,6 +306,7 @@ class HDSHNASTestCase(test.TestCase):
def test_delete_snapshot(self):
self.mock_object(hds_hnas.HDSHNASDriver, "_get_hnas_share_id",
mock.Mock(return_value=share['id']))
self.mock_object(hds_hnas.HDSHNASDriver, "_check_fs_mounted")
self.mock_object(ssh.HNASSSHBackend, "tree_delete", mock.Mock())
self.mock_object(ssh.HNASSSHBackend, "delete_directory", mock.Mock())
@ -313,6 +314,7 @@ class HDSHNASTestCase(test.TestCase):
self.assertTrue(self.mock_log.debug.called)
self.assertTrue(self.mock_log.info.called)
hds_hnas.HDSHNASDriver._check_fs_mounted.assert_called_once_with()
ssh.HNASSSHBackend.tree_delete.assert_called_once_with(
'/snapshots/' + share['id'] + '/' + snapshot['id'])
ssh.HNASSSHBackend.delete_directory.assert_called_once_with(