rochacbruno / talkshow Goto Github PK

No create_app os ext e os blueprints estão carregando declarativamente.
https://github.com/rochacbruno/talkshow/blob/master/talkshow/app.py#L18

Trocar por um carregamento dinâmico usando werkzeug.utils.import_string

ex:

No settings.toml ou no .env depois de implementar o #1

[default]
EXTENSIONS = ["talkshow.ext.cli", "talkshow.ext.admin"]
BLUEPRINTS = ["talkshow.blueprints.webui", "talkshow.blueprints.restapi"]

Criar ext/extensions.py

from werkzeug.utils import import_string

def configure(app):
    for ext in app.config.get('EXTENSIONS'):
        mod = import_string(ext)
        mod.configure(app)
    
     # fazer o mesmo para os blueprints...

No factory app.py

from talkshow.ext import configuration, extensions

def create_app():
    app = Flask(__name__)
    configuration.configure(app)  # config loaded here
    extensions.configure(app)
    return app
....

Ajustar setup conforme branch extended

Atualizar PDF

Atualmente temos definição de campos e validadores em 5 locais

• cli (argumentos dos comandos)
• UI (formulário wtforms)
• Admin (Formulario wtform do admin)
• API (reqparse do Flask_Restful)
• Apidocs (YAML do flasgger)

Precisamos criar em um local base /definitions.py uma classe Python usando descriptor ou algo do tipo que seja capaz de a partir de um unico modelo gerar specs para esses 5 locais. ex: @definition.generate_cli_specs() para ser usado ao inves de @click.option no cli e um equivalente para os outros locais.

Essa solução pode se transformar em uma library de uso geral externa.

Adicionar coverage e escrever mais testes

Notei que por motivos de nome parecido, acabei usando a extensão flask-pytest ao invés da pytest-flask

A que está sendo usada flask-pytest na verdade é um test runner em background, é interessante mas não nesse projeto.

A que precisamos usar é a pytest-flask pois essa sim entrega as fixtures necessárias para os testes.

• mudar no setup.py para usar pytest-flask
• alterar os testes para receber a fixture client ao inves de usar with app.test_client

O projeto tem 2 blueprints que são arquivos restapi.py e webui.py transformar em modulos/pastas e separar forms.py, models.py, views.py etc...

Incluri pastas que não precisam ser testadas, como as dos venvs.

Adicionar um ext/profiling.py para incluir Flask-Profiler

Usar Dynaconf para carregar as configurações a partir dos arquivos .env ou settings.toml

IMPORTANTE: qualquer melhoria será aplicada somente a branch extended

Ajustar pytest conforme a branch extended

atualizar PDF

Após a maioria das issues de estrutura serem resolvidas a idéia é criar uma versão do mesmo app utilizando SQLAlchemy + SQLite. em uma nova branch.

Usar awesome-slugify para criar o slug ao salvar o evento e então aceitar o slug na hora de fazer a query em https://github.com/rochacbruno/talkshow/blob/master/talkshow/blueprints/webui.py#L38

ao inves de /23495873498573498753498 usaremos /flask-conf-2018 para acessar os detalhes do evento e o form.

criar um ext/debug.py para adicionar a FlaskDebugToolbar

Atualmente a data do evento e da submissão de proposta estão em plain text

implementar o tinydb_serialization https://github.com/schapman1974/tinymongo#handling-datetime-objects

Trocar Basic Auth por JWT nas endpoints da API

*manter basic auth apenas para gerar o token

HAbilitar travis CI, criar .travis.yml e configurar para rodar os testes