NeoAnd/netbox
1
0
Fork 0
mirror of https://github.com/netbox-community/netbox.git synced 2025-08-26 09:16:10 -06:00
Code Issues Actions 7 Packages Projects Releases Wiki Activity

Add test for invalid custom field names

Browse Source
This commit is contained in:
jeremystretch 2023-05-12 15:59:07 -04:00
parent 050a3b789c
commit 970241e8c6
3 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
netbox/extras/migrations/0066_customfield_name_validation.py
View File

@ -25,7 +25,7 @@ class Migration(migrations.Migration):
django.core.validators.RegexValidator(
flags=re.RegexFlag['IGNORECASE'],
inverse_match=True,
message='No double-underscores are allowed.',
message='Double underscores are not permitted in custom field names.',
regex=r'__',
),
],

2
netbox/extras/models/customfields.py
View File

@ -87,7 +87,7 @@ class CustomField(CloningMixin, ExportTemplatesMixin, ChangeLoggedModel):
),
RegexValidator(
regex=r'__',
message="No double-underscores are allowed.",
message="Double underscores are not permitted in custom field names.",
flags=re.IGNORECASE,
inverse_match=True
),

11
netbox/extras/tests/test_customfields.py
View File

@ -29,6 +29,17 @@ class CustomFieldTest(TestCase):
cls.object_type = ContentType.objects.get_for_model(Site)
def test_invalid_name(self):
"""
Try creating a CustomField with an invalid name.
"""
with self.assertRaises(ValidationError):
# Invalid character
CustomField(name='?', type=CustomFieldTypeChoices.TYPE_TEXT).full_clean()
with self.assertRaises(ValidationError):
# Double underscores not permitted
CustomField(name='foo__bar', type=CustomFieldTypeChoices.TYPE_TEXT).full_clean()
def test_text_field(self):
value = 'Foobar!'