Merge pull request #101 from novafloss/80_validate_data_through_api

Validate data through api

Reviewed by @wo0dyn.

Author: wo0dyn

demo/tests/tests_integration.py

@@ -9,7 +9,7 @@
 from formidable.models import Formidable
 from formidable.accesses import get_accesses
 from formidable.forms import FormidableForm, fields
-from formidable import validators
+from formidable import validators, constants
 
 form_data = {
     "label": "test create",
@@ -275,3 +275,49 @@ def test_jedi_form_form_invalid(self):
             form.errors['birth_date'][0],
             'You cannot be a jedi until your 21'
         )
+
+
+class MyForm(FormidableForm):
+    first_name = fields.CharField(
+        accesses={'padawan': constants.REQUIRED},
+    )
+    last_name = fields.CharField(
+        accesses={'padawan': constants.REQUIRED},
+        validators=[validators.MinLengthValidator(5)]
+    )
+
+
+class TestValidationEndPoint(APITestCase):
+
+    def setUp(self):
+        super(TestValidationEndPoint, self).setUp()
+        self.formidable = MyForm.to_formidable(label='title')
+
+    def test_validate_data_ok(self):
+        parameters = {
+            'first_name': 'Guillaume',
+            'last_name': 'Gérard',
+        }
+        session = self.client.session
+        session['role'] = 'padawan'
+        session.save()
+        res = self.client.get(
+            reverse('formidable:form_validation', args=[self.formidable.pk]),
+            parameters, format='json'
+        )
+        self.assertEqual(res.status_code, 204)
+
+    def test_validate_data_ko(self):
+        parameters = {
+            'last_name': 'Gérard',
+        }
+        session = self.client.session
+        session['role'] = 'padawan'
+        session.save()
+        res = self.client.get(
+            reverse('formidable:form_validation', args=[self.formidable.pk]),
+            parameters, format='json'
+        )
+        self.assertEqual(res.status_code, 400)
+        errors = res.data
+        self.assertIn('first_name', errors)

formidable/accesses.py

@@ -46,6 +46,9 @@ def get_accesses():
 
 def get_context(request, kwargs):
     module, meth_name = settings.FORMIDABLE_CONTEXT_LOADER.rsplit('.', 1)
-    mod = importlib.import_module(module, [meth_name])
+    if six.PY3:
+        mod = importlib.import_module(module)
+    else:
+        mod = importlib.import_module(module, [meth_name])
     meth = getattr(mod, meth_name)
     return meth(request, kwargs)

formidable/urls.py

@@ -2,12 +2,14 @@
 
 from formidable.views import FormidableDetail, FormidableCreate, AccessList
 from formidable.views import ContextFormDetail
-from formidable.views import PresetsList
+from formidable.views import PresetsList, ValidateView
 
 urlpatterns = patterns(
     r'',
     url(r'^forms/(?P<pk>\d+)/$', ContextFormDetail.as_view(),
         name='form_detail'),
+    url(r'^forms/(?P<pk>\d+)/validate/$', ValidateView.as_view(),
+        name='form_validation'),
     url(r'^builder/forms/(?P<pk>\d+)/$', FormidableDetail.as_view(),
         name='form_detail'),
     url(r'^builder/forms/$', FormidableCreate.as_view(), name='form_create'),

formidable/views.py

@@ -8,6 +8,7 @@
     RetrieveAPIView,
 )
 from rest_framework.response import Response
+from rest_framework import exceptions
 
 from formidable.models import Formidable, Field
 from formidable.serializers import FormidableSerializer, SimpleAccessSerializer
@@ -69,3 +70,20 @@ def get(self, request, format=None):
             instance=presets_declarations
         )
         return Response(serializer.data)
+
+
+class ValidateView(APIView):
+
+    def get(self, request, **kwargs):
+        try:
+            formidable = Formidable.objects.get(pk=kwargs['pk'])
+        except Formidable.DoesNotExist:
+            raise exceptions.Http404()
+
+        role = get_context(request, kwargs)
+        form_class = formidable.get_django_form_class(role)
+        form = form_class(data=request.GET)
+        if form.is_valid():
+            return Response(status=204)
+        else:
+            return Response(form.errors, status=400)