Merge pull request #37 from vuLgAr/flake8

Flake8

Author: mjhea0

.travis.yml

@@ -14,4 +14,5 @@ install:
   - "pipenv install --dev"
 
 script:
+  - flake8 .
   - python manage.py cov

README.md

@@ -72,3 +72,17 @@ With coverage:
 ```sh
 $ python manage.py cov
 ```
+
+###
+
+Run flake8 on the app:
+
+```sh
+$ python manage.py flake
+```
+
+or 
+
+```
+$ flake8 .
+```

{{cookiecutter.app_slug}}/Pipfile

@@ -14,13 +14,15 @@ flask-login = "*"
 flask-migrate = "*"
 flask-sqlalchemy = "*"
 flask-wtf = "*"
+configparser = "*"
 
 
 [dev-packages]
 
 coverage = "*"
 flask-debugtoolbar = "*"
 flask-testing = "*"
+"flake8" = "*"
 
 
 [requires]

{{cookiecutter.app_slug}}/Pipfile.lock

@@ -4,11 +4,12 @@
 import unittest
 
 import coverage
+
 from flask.cli import FlaskGroup
 
 from project.server import create_app, db
 from project.server.models import User
-
+import subprocess
 
 app = create_app()
 cli = FlaskGroup(create_app=create_app)
@@ -25,10 +26,12 @@
 )
 COV.start()
 
+
 @app.shell_context_processor
 def make_shell_context():
     return dict(app=app, db=db)
 
+
 @cli.command()
 def create_db():
     db.drop_all()
@@ -81,6 +84,11 @@ def cov():
     return 1
 
 
+@cli.command()
+def flake():
+    """Runs flake8 on the project."""
+    subprocess.run(['flake8', '.'])
+
 
 if __name__ == '__main__':
     cli()

{{cookiecutter.app_slug}}/manage.py

@@ -13,7 +13,7 @@ def test_index(self):
         response = self.client.get('/', follow_redirects=True)
         self.assertEqual(response.status_code, 200)
         self.assertIn(b'Welcome!', response.data)
-        self.assertIn(b'Register/Login', response.data)
+        self.assertIn(b'Login', response.data)
 
     def test_about(self):
         # Ensure about route behaves correctly.

{{cookiecutter.app_slug}}/project/tests/test_main.py

@@ -96,7 +96,7 @@ def test_validate_invalid_password(self):
     def test_register_route(self):
         # Ensure about route behaves correctly.
         response = self.client.get('/register', follow_redirects=True)
-        self.assertIn(b'<h1>Please Register</h1>\n', response.data)
+        self.assertIn(b'<h1>Register</h1>\n', response.data)
 
     def test_user_registration(self):
         # Ensure registration behaves correctlys.

{{cookiecutter.app_slug}}/project/tests/test_user.py

@@ -1,4 +1,6 @@
+configparser==3.5.0
 coverage==4.5.1
+flake8==3.5.0
 Flask==0.12.2
 Flask-Bcrypt==0.7.1
 Flask-Bootstrap==3.3.7.1

{{cookiecutter.app_slug}}/requirements.txt

@@ -0,0 +1,3 @@
+[flake8]
+max-line-length=79
+exclude=migrations