Convert Django form decimals to Graphene decimals

Author: ulgens

graphene_django/filter/tests/test_fields.py

@@ -5,7 +5,7 @@
 from django.db.models import TextField, Value
 from django.db.models.functions import Concat
 
-from graphene import Argument, Boolean, Field, Float, ObjectType, Schema, String
+from graphene import Argument, Boolean, Decimal, Field, ObjectType, Schema, String
 from graphene.relay import Node
 from graphene_django import DjangoObjectType
 from graphene_django.forms import GlobalIDFormField, GlobalIDMultipleChoiceField
@@ -388,7 +388,7 @@ class Meta:
     field = DjangoFilterConnectionField(ArticleNode, filterset_class=ArticleIdFilter)
     max_time = field.args["max_time"]
     assert isinstance(max_time, Argument)
-    assert max_time.type == Float
+    assert max_time.type == Decimal
     assert max_time.description == "The maximum time"
 
 

graphene_django/forms/converter.py

@@ -1,12 +1,23 @@
 from django import forms
 from django.core.exceptions import ImproperlyConfigured
 
-from graphene import ID, Boolean, Float, Int, List, String, UUID, Date, DateTime, Time
+from graphene import (
+    Boolean,
+    Date,
+    DateTime,
+    Decimal,
+    Float,
+    ID,
+    Int,
+    List,
+    String,
+    Time,
+    UUID,
+)
 
 from .forms import GlobalIDFormField, GlobalIDMultipleChoiceField
 from ..utils import import_single_dispatch
 
-
 singledispatch = import_single_dispatch()
 
 
@@ -52,6 +63,10 @@ def convert_form_field_to_nullboolean(field):
 
 
 @convert_form_field.register(forms.DecimalField)
+def convert_field_to_decimal(field):
+    return Decimal(description=field.help_text, required=field.required)
+
+
 @convert_form_field.register(forms.FloatField)
 def convert_form_field_to_float(field):
     return Float(description=field.help_text, required=field.required)

graphene_django/forms/tests/test_converter.py

@@ -1,19 +1,19 @@
 from django import forms
 from py.test import raises
 
-import graphene
 from graphene import (
-    String,
-    Int,
     Boolean,
+    Date,
+    DateTime,
+    Decimal,
     Float,
     ID,
-    UUID,
+    Int,
     List,
     NonNull,
-    DateTime,
-    Date,
+    String,
     Time,
+    UUID,
 )
 
 from ..converter import convert_form_field
@@ -93,12 +93,12 @@ def test_should_nullboolean_convert_boolean():
     assert not isinstance(field.type, NonNull)
 
 
-def test_should_float_convert_float():
-    assert_conversion(forms.FloatField, Float)
+def test_should_decimal_convert_decimal():
+    assert_conversion(forms.DecimalField, Decimal)
 
 
-def test_should_decimal_convert_float():
-    assert_conversion(forms.DecimalField, Float)
+def test_should_float_convert_float():
+    assert_conversion(forms.FloatField, Float)
 
 
 def test_should_multiple_choice_convert_list():