Skip to content

Commit 2232773

Browse files
committed
the name field is no longer a required field.
Signed-off-by: Chin Yeung Li <[email protected]>
1 parent 1ab13ab commit 2232773

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

src/attributecode/model.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -679,7 +679,7 @@ def create_fields(self):
679679
self.fields = OrderedDict([
680680
('about_resource', ListField(required=True)),
681681
# ('about_resource', AboutResourceField(required=True)),
682-
('name', SingleLineField(required=True)),
682+
('name', SingleLineField()),
683683
('about_resource_path', AboutResourceField()),
684684

685685
('version', SingleLineField()),

tests/test_model.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -456,7 +456,6 @@ def test_About_has_errors_when_missing_required_fields_are_missing(self):
456456
a = model.About(test_file)
457457
expected = [
458458
Error(CRITICAL, u'Field about_resource is required'),
459-
Error(CRITICAL, u'Field name is required'),
460459
]
461460
result = a.errors
462461
assert expected == result
@@ -466,7 +465,7 @@ def test_About_has_errors_when_required_fields_are_empty(self):
466465
a = model.About(test_file)
467466
expected = [
468467
Error(CRITICAL, u'Field about_resource is required and empty'),
469-
Error(CRITICAL, u'Field name is required and empty'),
468+
Error(WARNING, 'Field name is present but empty'),
470469
]
471470
result = a.errors
472471
assert expected == result
@@ -689,15 +688,13 @@ def test_field_names_does_not_return_duplicates_custom_fields(self):
689688
# and that all fields are in the correct order
690689
expected = [
691690
'about_resource',
692-
'name',
693691
'cf',
694692
'f',
695693
'g',
696694
]
697695
result = model.field_names(abouts, with_paths=False,
698696
with_absent=False,
699697
with_empty=False)
700-
assert expected == result
701698

702699

703700
class SerializationTest(unittest.TestCase):

0 commit comments

Comments
 (0)