diff options
| author | Carlton Gibson | 2014-04-10 15:31:21 +0200 |
|---|---|---|
| committer | Carlton Gibson | 2014-04-10 15:31:21 +0200 |
| commit | 8bd5e7e612e499753dcffb3cfc10abd410d438a2 (patch) | |
| tree | 21a36aae75c2ba0eceff0e9c8e3a87951dfafbee /rest_framework | |
| parent | b0ba8d61ecf3c74470fa6ac019caff1fd4ca1be6 (diff) | |
| parent | a73498d7974b15a25902fbdd1024742b95a166d4 (diff) | |
| download | django-rest-framework-8bd5e7e612e499753dcffb3cfc10abd410d438a2.tar.bz2 | |
Merge pull request #1515 from carltongibson/#1408
Added test for #1408 — Already Passes
Diffstat (limited to 'rest_framework')
| -rw-r--r-- | rest_framework/relations.py | 2 | ||||
| -rw-r--r-- | rest_framework/tests/test_relations.py | 24 |
2 files changed, 26 insertions, 0 deletions
diff --git a/rest_framework/relations.py b/rest_framework/relations.py index 308545ce..3463954d 100644 --- a/rest_framework/relations.py +++ b/rest_framework/relations.py @@ -59,6 +59,8 @@ class RelatedField(WritableField): super(RelatedField, self).__init__(*args, **kwargs) if not self.required: + # Accessed in ModelChoiceIterator django/forms/models.py:1034 + # If set adds empty choice. self.empty_label = BLANK_CHOICE_DASH[0][1] self.queryset = queryset diff --git a/rest_framework/tests/test_relations.py b/rest_framework/tests/test_relations.py index f52e0e1e..37ac826b 100644 --- a/rest_framework/tests/test_relations.py +++ b/rest_framework/tests/test_relations.py @@ -2,8 +2,10 @@ General tests for relational fields. """ from __future__ import unicode_literals +from django import get_version from django.db import models from django.test import TestCase +from django.utils import unittest from rest_framework import serializers from rest_framework.tests.models import BlogPost @@ -118,3 +120,25 @@ class RelatedFieldSourceTests(TestCase): (serializers.ModelSerializer,), attrs) with self.assertRaises(AttributeError): TestSerializer(data={'name': 'foo'}) + +@unittest.skipIf(get_version() < '1.6.0', 'Upstream behaviour changed in v1.6') +class RelatedFieldChoicesTests(TestCase): + """ + Tests for #1408 "Web browseable API doesn't have blank option on drop down list box" + https://github.com/tomchristie/django-rest-framework/issues/1408 + """ + def test_blank_option_is_added_to_choice_if_required_equals_false(self): + """ + + """ + post = BlogPost(title="Checking blank option is added") + post.save() + + queryset = BlogPost.objects.all() + field = serializers.RelatedField(required=False, queryset=queryset) + + choice_count = BlogPost.objects.count() + widget_count = len(field.widget.choices) + + self.assertEqual(widget_count, choice_count + 1, 'BLANK_CHOICE_DASH option should have been added') + |
