fix ci, set up tox, set up pre-commit, various cleanups

Author: Julien Nakache

.pre-commit-config.yaml

@@ -0,0 +1,18 @@
+default_language_version:
+    python: python3.7
+repos:
+-   repo: git://github.com/pre-commit/pre-commit-hooks
+    rev: c8bad492e1b1d65d9126dba3fe3bd49a5a52b9d6  # v2.1.0
+    hooks:
+    -   id: check-merge-conflict
+    -   id: check-yaml
+    -   id: debug-statements
+    -   id: end-of-file-fixer
+        exclude: ^docs/.*$
+    # TODO Enable in its own PR
+    # -   id: trailing-whitespace
+    #     exclude: README.md
+-   repo: git://github.com/PyCQA/flake8
+    rev: 88caf5ac484f5c09aedc02167c59c66ff0af0068  # 3.7.7
+    hooks:
+    -   id: flake8

.travis.yml

@@ -1,42 +1,47 @@
 language: python
-sudo: false
-python:
-- 2.7
-- 3.4
-- 3.5
-- 3.6
-before_install:
-install:
-- |
-  if [ "$TEST_TYPE" = build ]; then
-    pip install pytest==3.0.2 pytest-cov pytest-benchmark coveralls six mock sqlalchemy_utils
-    pip install -e .
-    python setup.py develop
-  elif [ "$TEST_TYPE" = lint ]; then
-    pip install flake8
-  fi
-script:
-- |
-  if [ "$TEST_TYPE" = lint ]; then
-    echo "Checking Python code lint."
-    flake8 graphene_sqlalchemy
-    exit
-  elif [ "$TEST_TYPE" = build ]; then
-    py.test --cov=graphene_sqlalchemy graphene_sqlalchemy examples
-  fi
-after_success:
-- |
-  if [ "$TEST_TYPE" = build ]; then
-      coveralls
-  fi
-env:
-  matrix:
-  - TEST_TYPE=build
 matrix:
-  fast_finish: true
   include:
-  - python: '2.7'
-    env: TEST_TYPE=lint
+    # Python 2.7
+    # TODO Fix enum and add back tests for py27
+    # See https://github.com/graphql-python/graphene-sqlalchemy/pull/177
+    # - env: TOXENV=py27
+    #   python: 2.7
+    # Python 3.5
+    - env: TOXENV=py34
+      python: 3.4
+    # Python 3.5
+    - env: TOXENV=py35
+      python: 3.5
+    # Python 3.6
+    - env: TOXENV=py36
+      python: 3.6
+    # Python 3.7
+    - env: TOXENV=py37
+      python: 3.7
+      dist: xenial
+    # SQLAlchemy 1.1
+    - env: TOXENV=py37-sql11
+      python: 3.7
+      dist: xenial
+    # SQLAlchemy 1.2
+    - env: TOXENV=py37-sql12
+      python: 3.7
+      dist: xenial
+    # SQLAlchemy 1.3
+    - env: TOXENV=py37-sql13
+      python: 3.7
+      dist: xenial
+    # Pre-commit
+    - env: TOXENV=pre-commit
+      python: 3.7
+      dist: xenial
+install: pip install .[dev]
+script: tox
+after_success: coveralls
+cache:
+  directories:
+    - $HOME/.cache/pip
+    - $HOME/.cache/pre-commit
 deploy:
   provider: pypi
   user: syrusakbary

README.md

@@ -112,14 +112,24 @@ To learn more check out the following [examples](examples/):
 
 ## Contributing
 
-After cloning this repo, ensure dependencies are installed by running:
+Set up our development dependencies:
 
 ```sh
-python setup.py install
+pip install -e ".[dev]"
+pre-commit install  
 ```
 
-After developing, the full test suite can be evaluated by running:
+We use `tox` to test this library against different versions of `python` and `SQLAlchemy`.
+While developping locally, it is usually fine to run the tests against the most recent versions:
 
 ```sh
-python setup.py test # Use --pytest-args="-v -s" for verbose mode
+tox -e py37  # Python 3.7, SQLAlchemy < 2.0
+tox -e py37 -- -v -s  # Verbose output
+tox -e py37 -- -k test_query  # Only test_query.py 
+```
+
+Our linters will run automatically when committing via git hooks but you can also run them manually:
+
+```sh
+tox -e pre-commit
 ```

examples/nameko_sqlalchemy/README.md

@@ -51,4 +51,4 @@ Now the following command will setup the database, and start the server:
 
 Now head on over to postman and send POST request to:
 [http://127.0.0.1:5000/graphql](http://127.0.0.1:5000/graphql)
-and run some queries!
+and run some queries!

examples/nameko_sqlalchemy/app.py

@@ -33,4 +33,4 @@ def parse_body(self,request):
       elif content_type in ('application/x-www-form-urlencoded', 'multipart/form-data'):
           return request.form
 
-      return {}
+      return {}

examples/nameko_sqlalchemy/config.yml

@@ -1 +1 @@
-WEB_SERVER_ADDRESS: '0.0.0.0:5000'
+WEB_SERVER_ADDRESS: '0.0.0.0:5000'

examples/nameko_sqlalchemy/requirements.txt

@@ -1,4 +1,4 @@
 graphene[sqlalchemy]
 SQLAlchemy==1.0.11
 nameko
-graphql-server-core
+graphql-server-core

examples/nameko_sqlalchemy/run.sh

@@ -1,4 +1,4 @@
 #!/bin/sh
 echo "Starting application service server"
 # Run Service
-nameko run --config config.yml service
+nameko run --config config.yml service

examples/nameko_sqlalchemy/service.py

@@ -8,4 +8,4 @@ class DepartmentService:
 
     @http('POST', '/graphql')
     def query(self, request):
-        return App().query(request)
+        return App().query(request)

graphene_sqlalchemy/tests/test_converter.py

@@ -168,7 +168,7 @@ class Test(Base):
         )
 
     graphene_type = convert_sqlalchemy_column(Test.column)
-    assert graphene_type.kwargs["required"] == False
+    assert not graphene_type.kwargs["required"]
 
 
 def test_should_scalar_list_convert_list():
@@ -343,4 +343,3 @@ def __init__(self, col1, col2):
         )
 
     assert "Don't know how to convert the composite field" in str(excinfo.value)
-