openstack/deb-designate-dashboard
Code Issues Proposed changes

Merge "Add option for records pointing to domain"

Browse Source
This commit is contained in:
Jenkins 2016-07-11 12:18:46 +00:00 committed by Gerrit Code Review
commit 5627ddb4a6
2 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
designatedashboard/dashboards/project/dns_domains/forms.py
View File

@ -395,9 +395,9 @@ class RecordForm(forms.SelfHandlingForm):
# Name field
if self._is_field_blank(cleaned_data, 'name'):
if record_type in ['A', 'AAAA', 'CNAME', 'SRV', 'TXT', 'PTR']:
if record_type in ['CNAME', 'SRV', 'PTR']:
self._add_required_field_error('name')
elif record_type == 'MX':
elif record_type in ['MX', 'A', 'AAAA', 'TXT']:
cleaned_data['name'] = domain_name
else:
if record_type == 'SRV':

9
designatedashboard/tests/test_designatedashboard.py
View File

@ -100,7 +100,8 @@ class ARecordFormTests(base.BaseRecordFormCleanTests):
def test_missing_name_field(self):
self.form.cleaned_data['name'] = ''
self.form.clean()
self.assert_required_error('name')
self.assert_no_errors()
self.assertIsNotNone(self.form.cleaned_data['name'])
def test_missing_data_field(self):
self.form.cleaned_data['data'] = ''
@ -162,7 +163,8 @@ class AAAARecordFormTests(base.BaseRecordFormCleanTests):
def test_missing_name_field(self):
self.form.cleaned_data['name'] = ''
self.form.clean()
self.assert_required_error('name')
self.assert_no_errors()
self.assertIsNotNone(self.form.cleaned_data['name'])
def test_missing_data_field(self):
self.form.cleaned_data['data'] = ''
@ -317,7 +319,8 @@ class TXTRecordFormTests(base.BaseRecordFormCleanTests):
def test_missing_name_field(self):
self.form.cleaned_data['name'] = ''
self.form.clean()
self.assert_required_error('name')
self.assert_no_errors()
self.assertIsNotNone(self.form.cleaned_data['name'])
def test_missing_txt_field(self):
self.form.cleaned_data['txt'] = ''