don't crash on empty post with fieldsets

wq · Jun 12, 2023 · aab0284 · aab0284
1 parent 7898de4
commit aab0284
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/rest/serializers.py b/rest/serializers.py
@@ -567,7 +567,7 @@ def to_virtual_fieldsets(self, data):
         return data
 
     def to_internal_value(self, data):
-        if not getattr(self.Meta, "wq_fieldsets", None):
+        if not data or not getattr(self.Meta, "wq_fieldsets", None):
             return super().to_internal_value(data)
 
         for name, conf in self.Meta.wq_fieldsets.items():

diff --git a/tests/test_post.py b/tests/test_post.py
@@ -266,3 +266,10 @@ def test_rest_virtual_fieldset(self):
         self.assertEqual(
             response.data, {"general": {"title": ["This field is required."]}}
         )
+
+    def test_empty_post(self):
+        """
+        Empty post should return 400, not 500
+        """
+        response = self.client.post("/entities.json", data={})
+        self.assertTrue(status.is_client_error(response.status_code), response.data)