Refactor test_volume_backup due to leftovers
Current test class does not clean volume / backup resources.
Change the create for volume and backups from base and not from client.
The base create contains waiters and run cleanup resources.
Change-Id: I8a6a7945870d30f180bd41940b8ee8b82e2f87b7
Benny Kopilov
4 years ago
| 12 | 12 | # License for the specific language governing permissions and limitations |
| 13 | 13 | # under the License. |
| 14 | 14 | |
| 15 | from tempest.common import waiters | |
| 16 | 15 | from tempest import config |
| 17 | 16 | from tempest.lib import decorators |
| 18 | 17 | from tempest.lib import exceptions |
| 40 | 39 | def test_backup_crossproject_admin_negative(self): |
| 41 | 40 | |
| 42 | 41 | # create vol as user |
| 43 | volume = self.volumes_client.create_volume( | |
| 44 | size=CONF.volume.volume_size)['volume'] | |
| 45 | waiters.wait_for_volume_resource_status( | |
| 46 | self.volumes_client, | |
| 47 | volume['id'], 'available') | |
| 42 | volume = self.create_volume(size=CONF.volume.volume_size) | |
| 48 | 43 | |
| 49 | 44 | # create backup as user |
| 50 | backup = self.backups_client.create_backup( | |
| 51 | volume_id=volume['id'])['backup'] | |
| 52 | waiters.wait_for_volume_resource_status( | |
| 53 | self.backups_client, | |
| 54 | backup['id'], 'available') | |
| 55 | ||
| 45 | self.create_backup(volume_id=volume['id']) | |
| 56 | 46 | # try to create incremental backup as admin |
| 57 | 47 | self.assertRaises( |
| 58 | 48 | exceptions.BadRequest, self.admin_backups_client.create_backup, |
| 62 | 52 | def test_backup_crossproject_user_negative(self): |
| 63 | 53 | |
| 64 | 54 | # create vol as user |
| 65 | volume = self.volumes_client.create_volume( | |
| 66 | size=CONF.volume.volume_size)['volume'] | |
| 67 | waiters.wait_for_volume_resource_status( | |
| 68 | self.volumes_client, | |
| 69 | volume['id'], 'available') | |
| 55 | volume = self.create_volume(size=CONF.volume.volume_size) | |
| 70 | 56 | |
| 71 | 57 | # create backup as admin |
| 72 | backup = self.admin_backups_client.create_backup( | |
| 73 | volume_id=volume['id'])['backup'] | |
| 74 | waiters.wait_for_volume_resource_status( | |
| 75 | self.admin_backups_client, | |
| 76 | backup['id'], 'available') | |
| 58 | ||
| 59 | self.create_backup(volume_id=volume['id'], | |
| 60 | backup_client=self.admin_backups_client) | |
| 77 | 61 | |
| 78 | 62 | # try to create incremental backup as user |
| 79 | 63 | self.assertRaises( |
| 84 | 68 | def test_incremental_backup_respective_parents(self): |
| 85 | 69 | |
| 86 | 70 | # create vol as user |
| 87 | volume = self.volumes_client.create_volume( | |
| 88 | size=CONF.volume.volume_size)['volume'] | |
| 89 | waiters.wait_for_volume_resource_status( | |
| 90 | self.volumes_client, | |
| 91 | volume['id'], 'available') | |
| 71 | volume = self.create_volume(size=CONF.volume.volume_size) | |
| 92 | 72 | |
| 93 | 73 | # create backup as admin |
| 94 | backup_adm = self.admin_backups_client.create_backup( | |
| 95 | volume_id=volume['id'])['backup'] | |
| 96 | waiters.wait_for_volume_resource_status( | |
| 97 | self.admin_backups_client, | |
| 98 | backup_adm['id'], 'available') | |
| 74 | backup_adm = self.create_backup( | |
| 75 | volume_id=volume['id'], backup_client=self.admin_backups_client) | |
| 99 | 76 | |
| 100 | 77 | # create backup as user |
| 101 | backup_usr = self.backups_client.create_backup( | |
| 102 | volume_id=volume['id'])['backup'] | |
| 103 | waiters.wait_for_volume_resource_status( | |
| 104 | self.backups_client, | |
| 105 | backup_usr['id'], 'available') | |
| 78 | backup_usr = self.create_backup(volume_id=volume['id']) | |
| 106 | 79 | |
| 107 | 80 | # refresh admin backup and assert no child backups |
| 108 | 81 | backup_adm = self.admin_backups_client.show_backup( |
| 110 | 83 | self.assertFalse(backup_adm['has_dependent_backups']) |
| 111 | 84 | |
| 112 | 85 | # create incremental backup as admin |
| 113 | backup_adm_inc = self.admin_backups_client.create_backup( | |
| 114 | volume_id=volume['id'], incremental=True)['backup'] | |
| 115 | waiters.wait_for_volume_resource_status( | |
| 116 | self.admin_backups_client, | |
| 117 | backup_adm_inc['id'], 'available') | |
| 86 | self.create_backup(volume_id=volume['id'], incremental=True, | |
| 87 | backup_client=self.admin_backups_client) | |
| 118 | 88 | |
| 119 | 89 | # refresh user backup and assert no child backups |
| 120 | 90 | backup_usr = self.backups_client.show_backup( |
| 127 | 97 | self.assertTrue(backup_adm['has_dependent_backups']) |
| 128 | 98 | |
| 129 | 99 | # create incremental backup as user |
| 130 | backup_usr_inc = self.backups_client.create_backup( | |
| 131 | volume_id=volume['id'], incremental=True)['backup'] | |
| 132 | waiters.wait_for_volume_resource_status( | |
| 133 | self.backups_client, | |
| 134 | backup_usr_inc['id'], 'available') | |
| 100 | self.create_backup(volume_id=volume['id'], | |
| 101 | incremental=True) | |
| 135 | 102 | |
| 136 | 103 | # refresh user backup and assert it has childs |
| 137 | 104 | backup_usr = self.backups_client.show_backup( |