Merge "Tests: Improve cascade delete coverage"

Marblelsp · Jun 18, 2021 · fa05c2e · fa05c2e
2 parents 7a854c1 + 8f6a3ca
commit fa05c2e
Showing 1 changed file with 14 additions and 0 deletions.
diff --git a/cinder/tests/unit/volume/test_volume.py b/cinder/tests/unit/volume/test_volume.py
@@ -1003,6 +1003,20 @@ def test_unmanage_encrypted_volume_fails(self):
                           unmanage_only=True)
         self.volume.delete_volume(self.context, volume)
 
+    def test_unmanage_cascade_delete_fails(self):
+        volume = tests_utils.create_volume(
+            self.context,
+            **self.volume_params)
+        self.volume.create_volume(self.context, volume)
+        manager = vol_manager.VolumeManager()
+        self.assertRaises(exception.Invalid,
+                          manager.delete_volume,
+                          self.context,
+                          volume,
+                          unmanage_only=True,
+                          cascade=True)
+        self.volume.delete_volume(self.context, volume)
+
     def test_get_volume_different_tenant(self):
         """Test can't get volume of another tenant when viewable_admin_meta."""
         volume = tests_utils.create_volume(self.context,