mirror of
https://github.com/graphql-python/graphene-django.git
synced 2024-11-25 02:54:06 +03:00
Update mutation.py to serialize Enum objects into input values (#1431)
* Fix for issue #1385: Update mutation.py to serialize Enum objects into input values for ChoiceFields * Update graphene_django/rest_framework/mutation.py Co-authored-by: Steven DeMartini <1647130+sjdemartini@users.noreply.github.com> --------- Co-authored-by: Steven DeMartini <1647130+sjdemartini@users.noreply.github.com>
This commit is contained in:
parent
3172710d12
commit
5d7a04fce9
|
@ -14,3 +14,14 @@ class MyFakeModelWithPassword(models.Model):
|
||||||
class MyFakeModelWithDate(models.Model):
|
class MyFakeModelWithDate(models.Model):
|
||||||
cool_name = models.CharField(max_length=50)
|
cool_name = models.CharField(max_length=50)
|
||||||
last_edited = models.DateField()
|
last_edited = models.DateField()
|
||||||
|
|
||||||
|
|
||||||
|
class MyFakeModelWithChoiceField(models.Model):
|
||||||
|
class ChoiceType(models.Choices):
|
||||||
|
ASDF = "asdf"
|
||||||
|
HI = "hi"
|
||||||
|
|
||||||
|
choice_type = models.CharField(
|
||||||
|
max_length=4,
|
||||||
|
default=ChoiceType.HI.name,
|
||||||
|
)
|
||||||
|
|
|
@ -1,3 +1,5 @@
|
||||||
|
from enum import Enum
|
||||||
|
|
||||||
from collections import OrderedDict
|
from collections import OrderedDict
|
||||||
|
|
||||||
from django.shortcuts import get_object_or_404
|
from django.shortcuts import get_object_or_404
|
||||||
|
@ -124,8 +126,10 @@ class SerializerMutation(ClientIDMutation):
|
||||||
def get_serializer_kwargs(cls, root, info, **input):
|
def get_serializer_kwargs(cls, root, info, **input):
|
||||||
lookup_field = cls._meta.lookup_field
|
lookup_field = cls._meta.lookup_field
|
||||||
model_class = cls._meta.model_class
|
model_class = cls._meta.model_class
|
||||||
|
|
||||||
if model_class:
|
if model_class:
|
||||||
|
for input_dict_key, maybe_enum in input.items():
|
||||||
|
if isinstance(maybe_enum, Enum):
|
||||||
|
input[input_dict_key] = maybe_enum.value
|
||||||
if "update" in cls._meta.model_operations and lookup_field in input:
|
if "update" in cls._meta.model_operations and lookup_field in input:
|
||||||
instance = get_object_or_404(
|
instance = get_object_or_404(
|
||||||
model_class, **{lookup_field: input[lookup_field]}
|
model_class, **{lookup_field: input[lookup_field]}
|
||||||
|
|
|
@ -7,7 +7,12 @@ from graphene import Field, ResolveInfo
|
||||||
from graphene.types.inputobjecttype import InputObjectType
|
from graphene.types.inputobjecttype import InputObjectType
|
||||||
|
|
||||||
from ...types import DjangoObjectType
|
from ...types import DjangoObjectType
|
||||||
from ..models import MyFakeModel, MyFakeModelWithDate, MyFakeModelWithPassword
|
from ..models import (
|
||||||
|
MyFakeModel,
|
||||||
|
MyFakeModelWithDate,
|
||||||
|
MyFakeModelWithPassword,
|
||||||
|
MyFakeModelWithChoiceField,
|
||||||
|
)
|
||||||
from ..mutation import SerializerMutation
|
from ..mutation import SerializerMutation
|
||||||
|
|
||||||
|
|
||||||
|
@ -268,6 +273,39 @@ def test_perform_mutate_success():
|
||||||
assert result.days_since_last_edit == 4
|
assert result.days_since_last_edit == 4
|
||||||
|
|
||||||
|
|
||||||
|
def test_perform_mutate_success_with_enum_choice_field():
|
||||||
|
class ListViewChoiceFieldSerializer(serializers.ModelSerializer):
|
||||||
|
choice_type = serializers.ChoiceField(
|
||||||
|
choices=[(x.name, x.value) for x in MyFakeModelWithChoiceField.ChoiceType],
|
||||||
|
required=False,
|
||||||
|
)
|
||||||
|
|
||||||
|
class Meta:
|
||||||
|
model = MyFakeModelWithChoiceField
|
||||||
|
fields = "__all__"
|
||||||
|
|
||||||
|
class SomeCreateSerializerMutation(SerializerMutation):
|
||||||
|
class Meta:
|
||||||
|
serializer_class = ListViewChoiceFieldSerializer
|
||||||
|
|
||||||
|
choice_type = {
|
||||||
|
"choice_type": SomeCreateSerializerMutation.Input.choice_type.type.get("ASDF")
|
||||||
|
}
|
||||||
|
name = MyFakeModelWithChoiceField.ChoiceType.ASDF.name
|
||||||
|
result = SomeCreateSerializerMutation.mutate_and_get_payload(
|
||||||
|
None, mock_info(), **choice_type
|
||||||
|
)
|
||||||
|
assert result.errors is None
|
||||||
|
assert result.choice_type == name
|
||||||
|
kwargs = SomeCreateSerializerMutation.get_serializer_kwargs(
|
||||||
|
None, mock_info(), **choice_type
|
||||||
|
)
|
||||||
|
assert kwargs["data"]["choice_type"] == name
|
||||||
|
assert 1 == MyFakeModelWithChoiceField.objects.count()
|
||||||
|
item = MyFakeModelWithChoiceField.objects.first()
|
||||||
|
assert item.choice_type == name
|
||||||
|
|
||||||
|
|
||||||
def test_mutate_and_get_payload_error():
|
def test_mutate_and_get_payload_error():
|
||||||
class MyMutation(SerializerMutation):
|
class MyMutation(SerializerMutation):
|
||||||
class Meta:
|
class Meta:
|
||||||
|
|
Loading…
Reference in New Issue
Block a user