Merge "fix workflow exception on host aggregate create form"
This commit is contained in:
commit
fc8c13231a
@ -117,6 +117,17 @@ class CreateAggregateWorkflowTests(BaseAggregateWorkflowTests):
|
|||||||
self._test_generic_create_aggregate(workflow_data, aggregate, (), 1,
|
self._test_generic_create_aggregate(workflow_data, aggregate, (), 1,
|
||||||
u'This field is required')
|
u'This field is required')
|
||||||
|
|
||||||
|
def test_create_aggregate_fails_missing_fields_existing_aggregates(self):
|
||||||
|
aggregate = self.aggregates.first()
|
||||||
|
existing_aggregates = self.aggregates.list()
|
||||||
|
workflow_data = self._get_create_workflow_data(aggregate)
|
||||||
|
workflow_data['name'] = ''
|
||||||
|
workflow_data['availability_zone'] = ''
|
||||||
|
|
||||||
|
self._test_generic_create_aggregate(workflow_data, aggregate,
|
||||||
|
existing_aggregates, 1,
|
||||||
|
u'This field is required')
|
||||||
|
|
||||||
def test_create_aggregate_fails_duplicated_name(self):
|
def test_create_aggregate_fails_duplicated_name(self):
|
||||||
aggregate = self.aggregates.first()
|
aggregate = self.aggregates.first()
|
||||||
existing_aggregates = self.aggregates.list()
|
existing_aggregates = self.aggregates.list()
|
||||||
|
@ -37,7 +37,7 @@ class SetAggregateInfoAction(workflows.Action):
|
|||||||
|
|
||||||
def clean(self):
|
def clean(self):
|
||||||
cleaned_data = super(SetAggregateInfoAction, self).clean()
|
cleaned_data = super(SetAggregateInfoAction, self).clean()
|
||||||
name = cleaned_data.get('name')
|
name = cleaned_data.get('name', '')
|
||||||
|
|
||||||
try:
|
try:
|
||||||
aggregates = api.nova.aggregate_details_list(self.request)
|
aggregates = api.nova.aggregate_details_list(self.request)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user