From 3b07c1dfc6d40e3f7d7b71d388b39e40725c232e Mon Sep 17 00:00:00 2001 From: Warren Jin Date: Thu, 29 Jan 2015 14:20:42 -0500 Subject: [PATCH] required not actually necessary if allow_null present --- tests/test_recursive.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/test_recursive.py b/tests/test_recursive.py index f3e60d2b2..a4524bbb7 100644 --- a/tests/test_recursive.py +++ b/tests/test_recursive.py @@ -3,7 +3,7 @@ from rest_framework import serializers class LinkSerializer(serializers.Serializer): name = serializers.CharField(max_length=25) - next = serializers.RecursiveField(required=False, allow_null=True) + next = serializers.RecursiveField(allow_null=True) class NodeSerializer(serializers.Serializer): @@ -104,6 +104,7 @@ class TestRecursiveField: 'name': 'something', 'next': { 'name': 'inner something', + 'next': None, } } } @@ -116,6 +117,7 @@ class TestRecursiveField: 'nullable': 'not null', 'links': { 'name': 'something', + 'next': None, } } serializer = SillySerializer(data=max_length) @@ -128,6 +130,7 @@ class TestRecursiveField: 'nullable': 'not null', 'links': { 'name': 'something', + 'next': None, } } serializer = SillySerializer(data=nulled_out) @@ -142,6 +145,7 @@ class TestRecursiveField: 'name': 'something', 'next': { 'name': 'inner something that is much too long', + 'next': None, } } }