Improved syntax. Added autolinter. Added automatic flake8 checker in tests. Fixed #17

Author: syrusakbary

.travis.yml

@@ -13,6 +13,6 @@ install:
 - python setup.py develop
 script:
 - py.test --cov=graphene
-# - flake8
+- flake8
 after_success:
 - coveralls

bin/autolinter

@@ -0,0 +1,4 @@
+#!/bin/bash
+
+autoflake ./ -r  --remove-unused-variables --remove-all-unused-imports --in-place
+isort -rc .

examples/starwars/schema.py

@@ -1,8 +1,8 @@
-from graphql.core.type import GraphQLEnumValue
 import graphene
 from graphene import resolve_only_args
+from graphql.core.type import GraphQLEnumValue
 
-from .data import getHero, getHuman, getCharacter, getDroid
+from .data import getCharacter, getDroid, getHero, getHuman
 
 Episode = graphene.Enum('Episode', dict(
     NEWHOPE=GraphQLEnumValue(4),

examples/starwars/tests/test_query.py

@@ -1,9 +1,10 @@
-from ..schema import Schema, Query
-from graphql.core import graphql
+
 from ..data import setup
+from ..schema import Schema
 
 setup()
 
+
 def test_hero_name_query():
     query = '''
         query HeroNameQuery {

examples/starwars_django/data.py

@@ -1,4 +1,4 @@
-from .models import Ship, Faction, Character
+from .models import Character, Faction, Ship
 
 
 def initialize():

examples/starwars_django/models.py

@@ -1,4 +1,5 @@
 from __future__ import absolute_import
+
 from django.db import models
 
 

examples/starwars_django/schema.py

@@ -1,24 +1,18 @@
 import graphene
-from graphene import resolve_only_args, relay
-from graphene.contrib.django import (
-    DjangoObjectType,
-    DjangoNode
-)
-from .models import (
-    Ship as ShipModel, Faction as FactionModel, Character as CharacterModel)
-from .data import (
-    get_faction,
-    get_ship,
-    get_ships,
-    get_rebels,
-    get_empire,
-    create_ship
-)
+from graphene import relay, resolve_only_args
+from graphene.contrib.django import DjangoNode, DjangoObjectType
+
+from .data import (create_ship, get_empire, get_faction, get_rebels, get_ship,
+                   get_ships)
+from .models import Character as CharacterModel
+from .models import Faction as FactionModel
+from .models import Ship as ShipModel
 
 schema = graphene.Schema(name='Starwars Django Relay Schema')
 
 
 class Ship(DjangoNode):
+
     class Meta:
         model = ShipModel
 
@@ -29,11 +23,13 @@ def get_node(cls, id):
 
 @schema.register
 class Character(DjangoObjectType):
+
     class Meta:
         model = CharacterModel
 
 
 class Faction(DjangoNode):
+
     class Meta:
         model = FactionModel
 
@@ -43,6 +39,7 @@ def get_node(cls, id):
 
 
 class IntroduceShip(relay.ClientIDMutation):
+
     class Input:
         ship_name = graphene.StringField(required=True)
         faction_id = graphene.StringField(required=True)

examples/starwars_django/tests/test_connections.py

@@ -1,9 +1,7 @@
 import pytest
-from graphql.core import graphql
 
-from ..models import *
-from ..schema import schema
 from ..data import initialize
+from ..schema import schema
 
 pytestmark = pytest.mark.django_db
 

examples/starwars_django/tests/test_mutation.py

@@ -1,6 +1,7 @@
 import pytest
-from ..schema import schema
+
 from ..data import initialize
+from ..schema import schema
 
 pytestmark = pytest.mark.django_db
 

examples/starwars_django/tests/test_objectidentification.py

@@ -1,8 +1,6 @@
 import pytest
-from pytest import raises
-from graphql.core import graphql
-from ..data import initialize
 
+from ..data import initialize
 from ..schema import schema
 
 pytestmark = pytest.mark.django_db