encode/django-rest-framework
1
1
Fork 0
mirror of https://github.com/encode/django-rest-framework.git synced 2025-08-02 19:40:13 +03:00
Code Issues Packages Projects Releases Wiki Activity

Add test for allow_null + required=False

Browse Source 
Ref #5708: allow_null should imply default=None, even for non-required fields.

flake8
This commit is contained in:
Carlton Gibson 2018-03-20 13:43:52 +01:00
parent 12569f83c9
commit 7bb92d3ba5
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
tests/test_serializer.py
View File

@ -486,12 +486,16 @@ class TestDefaultOutput:
assert Serializer({'nested': {'a': '3', 'b': {'c': '4'}}}).data == {'nested': {'a': '3', 'c': '4'}} assert Serializer({'nested': {'a': '3', 'b': {'c': '4'}}}).data == {'nested': {'a': '3', 'c': '4'}}
def test_default_for_allow_null(self): def test_default_for_allow_null(self):
# allow_null=True should imply default=None """
Without an explicit default, allow_null implies default=None when serializing. #5518 #5708
"""
class Serializer(serializers.Serializer): class Serializer(serializers.Serializer):
foo = serializers.CharField() foo = serializers.CharField()
bar = serializers.CharField(source='foo.bar', allow_null=True) bar = serializers.CharField(source='foo.bar', allow_null=True)
optional = serializers.CharField(required=False, allow_null=True)
assert Serializer({'foo': None}).data == {'foo': None, 'bar': None} # allow_null=True should imply default=None when serialising:
assert Serializer({'foo': None}).data == {'foo': None, 'bar': None, 'optional': None, }
class TestCacheSerializerData: class TestCacheSerializerData: