Introduce Black formatting, additional tests

Author: Paul Hallett

.travis.yml

@@ -1,19 +1,21 @@
 language: python
-sudo: false
+sudo: required
+dist: xenial
 python:
 - 2.7
 - 3.4
 - 3.5
 - 3.6
+- 3.7
 install:
 - |
   if [ "$TEST_TYPE" = build ]; then
     pip install -e .[test]
-    pip install psycopg2 # Required for Django postgres fields testing
+    pip install psycopg2==2.8.2 # Required for Django postgres fields testing
     pip install django==$DJANGO_VERSION
     python setup.py develop
   elif [ "$TEST_TYPE" = lint ]; then
-    pip install flake8
+    pip install flake8==3.7.7
   fi
 script:
 - |
@@ -45,10 +47,16 @@ matrix:
     env: TEST_TYPE=build DJANGO_VERSION=2.1
   - python: '3.6'
     env: TEST_TYPE=build DJANGO_VERSION=2.1
+  - python: '3.6'
+    env: TEST_TYPE=build DJANGO_VERSION=2.2
+  - python: '3.7'
+    env: TEST_TYPE=build DJANGO_VERSION=2.2
   - python: '2.7'
     env: TEST_TYPE=lint
   - python: '3.6'
     env: TEST_TYPE=lint
+  - python: '3.7'
+    env: TEST_TYPE=lint
 deploy:
   provider: pypi
   user: syrusakbary

CONTRIBUTING.md

@@ -31,6 +31,19 @@ After developing, the full test suite can be evaluated by running:
 make tests
 ```
 
+## Opening Pull Requests
+
+Please fork the project and open a pull request against the master branch.
+
+This will trigger a series of test and lint checks.
+
+We advise that you format and run lint locally before doing this to save time:
+
+```sh
+make format
+make lint
+```
+
 ## Documentation
 
 The [documentation](http://docs.graphene-python.org/projects/django/en/latest/) is generated using the excellent [Sphinx](http://www.sphinx-doc.org/) and a custom theme.

Makefile

@@ -1,5 +1,11 @@
 dev-setup:
-	pip install -e ".[test]"
+	pip install -e ".[dev]"
 
 tests:
-	py.test graphene_django --cov=graphene_django -vv
+	py.test graphene_django --cov=graphene_django -vv
+
+format:
+	black graphene_django
+
+lint:
+	flake8 graphene_django

README.md

@@ -96,4 +96,4 @@ To learn more check out the following [examples](examples/):
 
 ## Contributing
 
-See [CONTRIBUTING.md](contributing.md)
+See [CONTRIBUTING.md](CONTRIBUTING.md)

graphene_django/compat.py

@@ -5,7 +5,11 @@ class MissingType(object):
 try:
     # Postgres fields are only available in Django with psycopg2 installed
     # and we cannot have psycopg2 on PyPy
-    from django.contrib.postgres.fields import (ArrayField, HStoreField,
-                                                JSONField, RangeField)
+    from django.contrib.postgres.fields import (
+        ArrayField,
+        HStoreField,
+        JSONField,
+        RangeField,
+    )
 except ImportError:
     ArrayField, HStoreField, JSONField, RangeField = (MissingType,) * 4

graphene_django/debug/sql/types.py

@@ -3,9 +3,7 @@
 
 class DjangoDebugSQL(ObjectType):
     class Meta:
-        description = (
-            "Represents a single database query made to a Django managed DB."
-        )
+        description = "Represents a single database query made to a Django managed DB."
 
     vendor = String(
         required=True,
@@ -14,37 +12,26 @@ class Meta:
         ),
     )
     alias = String(
-        required=True,
-        description="The Django database alias (e.g. 'default').",
+        required=True, description="The Django database alias (e.g. 'default')."
     )
     sql = String(description="The actual SQL sent to this database.")
     duration = Float(
-        required=True,
-        description="Duration of this database query in seconds.",
+        required=True, description="Duration of this database query in seconds."
     )
     raw_sql = String(
-        required=True,
-        description="The raw SQL of this query, without params.",
+        required=True, description="The raw SQL of this query, without params."
     )
     params = String(
-        required=True,
-        description="JSON encoded database query parameters.",
-    )
-    start_time = Float(
-        required=True,
-        description="Start time of this database query.",
-    )
-    stop_time = Float(
-        required=True,
-        description="Stop time of this database query.",
+        required=True, description="JSON encoded database query parameters."
     )
+    start_time = Float(required=True, description="Start time of this database query.")
+    stop_time = Float(required=True, description="Stop time of this database query.")
     is_slow = Boolean(
         required=True,
         description="Whether this database query took more than 10 seconds.",
     )
     is_select = Boolean(
-        required=True,
-        description="Whether this database query was a SELECT.",
+        required=True, description="Whether this database query was a SELECT."
     )
 
     # Postgres

graphene_django/debug/types.py

@@ -7,7 +7,4 @@ class DjangoDebug(ObjectType):
     class Meta:
         description = "Debugging information for the current query."
 
-    sql = List(
-        DjangoDebugSQL,
-        description="Executed SQL queries for this API query.",
-    )
+    sql = List(DjangoDebugSQL, description="Executed SQL queries for this API query.")

graphene_django/filter/filterset.py

@@ -45,8 +45,7 @@ class GrapheneFilterSetMixin(BaseFilterSet):
 
     FILTER_DEFAULTS = dict(
         itertools.chain(
-            FILTER_FOR_DBFIELD_DEFAULTS.items(),
-            GRAPHENE_FILTER_SET_OVERRIDES.items()
+            FILTER_FOR_DBFIELD_DEFAULTS.items(), GRAPHENE_FILTER_SET_OVERRIDES.items()
         )
     )
 
@@ -59,7 +58,6 @@ class GrapheneFilterSetMixin(BaseFilterSet):
     from django.utils.text import capfirst
 
     class GrapheneFilterSetMixinPython2(GrapheneFilterSetMixin):
-
         @classmethod
         def filter_for_reverse_field(cls, f, name):
             """Handles retrieving filters for reverse relationships

graphene_django/forms/tests/test_mutation.py

@@ -13,7 +13,7 @@ class MyForm(forms.Form):
 class PetForm(forms.ModelForm):
     class Meta:
         model = Pet
-        fields = '__all__'
+        fields = "__all__"
 
 
 def test_needs_form_class():
@@ -66,7 +66,7 @@ def test_exclude_fields_input_meta_fields(self):
         class PetMutation(DjangoModelFormMutation):
             class Meta:
                 form_class = PetForm
-                exclude_fields = ['id']
+                exclude_fields = ["id"]
 
         self.assertEqual(PetMutation._meta.model, Pet)
         self.assertEqual(PetMutation._meta.return_field_name, "pet")
@@ -102,7 +102,9 @@ class Meta:
 
         pet = Pet.objects.create(name="Axel", age=10)
 
-        result = PetMutation.mutate_and_get_payload(None, None, id=pet.pk, name="Mia", age=10)
+        result = PetMutation.mutate_and_get_payload(
+            None, None, id=pet.pk, name="Mia", age=10
+        )
 
         self.assertEqual(Pet.objects.count(), 1)
         pet.refresh_from_db()
@@ -132,7 +134,6 @@ class Meta:
         # A pet was not created
         self.assertEqual(Pet.objects.count(), 0)
 
-
         fields_w_error = [e.field for e in result.errors]
         self.assertEqual(len(result.errors), 2)
         self.assertIn("name", fields_w_error)

graphene_django/management/commands/graphql_schema.py

@@ -64,7 +64,7 @@ def handle(self, *args, **options):
 
         indent = options.get("indent")
         schema_dict = {"data": schema.introspect()}
-        if out == '-':
+        if out == "-":
             self.stdout.write(json.dumps(schema_dict, indent=indent, sort_keys=True))
         else:
             self.save_file(out, schema_dict, indent)