Fixes bug 1006405-Additional test cases to be added to test_volumes_negative.py
Change-Id: If77721041909a01c205b18b59fcec53278e7eaae
diff --git a/tempest/tests/test_volumes_negative.py b/tempest/tests/test_volumes_negative.py
index 83eef52..5346b06 100644
--- a/tempest/tests/test_volumes_negative.py
+++ b/tempest/tests/test_volumes_negative.py
@@ -3,6 +3,7 @@
from tempest import openstack
from tempest.common.utils.data_utils import rand_name
from tempest import exceptions
+from nose.tools import raises
class VolumesTest(unittest.TestCase):
@@ -52,3 +53,76 @@
pass
else:
self.fail('Should not be able to DELETE a nonexistant volume')
+
+ @unittest.skip('Until bug 1006857 is fixed.')
+ @raises(exceptions.BadRequest)
+ @attr(type='negative')
+ def test_create_volume_with_invalid_size(self):
+ """
+ Negative: Should not be able to create volume with invalid size
+ in request
+ """
+ v_name = rand_name('Volume-')
+ metadata = {'Type': 'work'}
+ resp, volume = self.client.create_volume(size='#$%',
+ display_name=v_name,
+ metadata=metadata)
+
+ @unittest.skip('Until bug 1006857 is fixed.')
+ @raises(exceptions.BadRequest)
+ @attr(type='negative')
+ def test_create_volume_with_out_passing_size(self):
+ """
+ Negative: Should not be able to create volume without passing size
+ in request
+ """
+ v_name = rand_name('Volume-')
+ metadata = {'Type': 'work'}
+ resp, volume = self.client.create_volume(size='',
+ display_name=v_name,
+ metadata=metadata)
+
+ @unittest.skip('Until bug 1006875 is fixed.')
+ @raises(exceptions.BadRequest)
+ @attr(type='negative')
+ def test_create_volume_with_size_zero(self):
+ """
+ Negative: Should not be able to create volume with size zero
+ """
+ v_name = rand_name('Volume-')
+ metadata = {'Type': 'work'}
+ resp, volume = self.client.create_volume(size='0',
+ display_name=v_name,
+ metadata=metadata)
+
+ @raises(exceptions.NotFound)
+ @attr(type='negative')
+ def test_get_invalid_volume_id(self):
+ """
+ Negative: Should not be able to get volume with invalid id
+ """
+ resp, volume = self.client.get_volume('#$%%&^&^')
+
+ @raises(exceptions.NotFound)
+ @attr(type='negative')
+ def test_get_volume_without_passing_volume_id(self):
+ """
+ Negative: Should not be able to get volume when empty ID is passed
+ """
+ resp, volume = self.client.get_volume('')
+
+ @raises(exceptions.NotFound)
+ @attr(type='negative')
+ def test_delete_invalid_volume_id(self):
+ """
+ Negative: Should not be able to delete volume when invalid ID is passed
+ """
+ resp, volume = self.client.delete_volume('!@#$%^&*()')
+
+ @raises(exceptions.NotFound)
+ @attr(type='negative')
+ def test_delete_volume_without_passing_volume_id(self):
+ """
+ Negative: Should not be able to delete volume when empty ID is passed
+ """
+ resp, volume = self.client.delete_volume('')