in required Likert field, require answer for each question

src/collective/easyform/fields.py

@@ -27,6 +27,7 @@
 from zope.schema import TextLine
 from zope.schema._bootstrapinterfaces import IFromUnicode
 from zope.schema.interfaces import IField
+from zope.schema.interfaces import ValidationError
 
 
 def superAdapter(specific_interface, adapter, objects, name=u""):
@@ -247,6 +248,10 @@ class NorobotCaptcha(TextLine):
 NorobotCaptchaHandler = BaseHandler(NorobotCaptcha)
 
 
+class AllAnswersRequired(ValidationError):
+    __doc__ = _("Answers are required for each question.")
+
+
 @implementer(ILikert)
 class Likert(TextLine):
     """A Likert field"""
@@ -262,7 +267,9 @@ def __init__(self, **kwargs):
 
     def _validate(self, value):
         super(Likert, self)._validate(value)
-        self.parse(value)
+        result = self.parse(value)
+        if self.required and len(result) != len(self.questions):
+            raise AllAnswersRequired()
 
     def parse(self, value):
         result = dict()

src/collective/easyform/locales/collective.easyform.pot

@@ -51,6 +51,10 @@ msgstr ""
 msgid "Allowed Fields"
 msgstr ""
 
+#: collective/easyform/fields.py
+msgid "Answers are required for each question."
+msgstr ""
+
 #: collective/easyform/interfaces/fields.py:105
 msgid "CSS Class"
 msgstr ""

src/collective/easyform/locales/fr/LC_MESSAGES/collective.easyform.po

@@ -48,6 +48,10 @@ msgstr "Avancé"
 msgid "Allowed Fields"
 msgstr "Champs autorisés"
 
+#: collective/easyform/fields.py
+msgid "Answers are required for each question."
+msgstr "Chaque question nécessite une réponse."
+
 #: collective/easyform/interfaces/fields.py:105
 msgid "CSS Class"
 msgstr ""

src/collective/easyform/locales/nl/LC_MESSAGES/collective.easyform.po

@@ -52,6 +52,10 @@ msgstr "Geavanceerd"
 msgid "Allowed Fields"
 msgstr "Toegestane velden"
 
+#: collective/easyform/fields.py
+msgid "Answers are required for each question."
+msgstr "Elke vraag dient beantwoord te worden."
+
 #: collective/easyform/interfaces/fields.py:105
 msgid "CSS Class"
 msgstr ""

src/collective/easyform/tests/testLikert.py

@@ -2,6 +2,7 @@
 import six
 
 from collective.easyform.tests.base import EasyFormFunctionalTestCase
+from zope.schema.interfaces import RequiredMissing
 
 
 class LikertFieldTests(unittest.TestCase):
@@ -42,6 +43,16 @@ def test_validate_with_more_answers(self):
         self.assertRaises(ValueError, field.validate, u'-1:agree')
         self.assertRaises(ValueError, field.validate, u'Agree')
 
+    def test_validate_required(self):
+        from collective.easyform.fields import AllAnswersRequired
+
+        field = self._makeOne(required=True, questions=[u'Question 1', u'Question 2'], answers=[u'Agree', u'Disagree'])
+
+        field.validate(u'1: Disagree, 2: Agree')
+        self.assertRaises(RequiredMissing, field.validate, None)
+        self.assertRaises(AllAnswersRequired, field.validate, u'1:Agree')
+        self.assertRaises(AllAnswersRequired, field.validate, u'')
+
     def test_parse(self):
         field = self._makeOne(required=False, questions=[u'Question 1', u'Question 2'], answers=[u'Agree', u'Disagree'])
         field.validate(None)