search

odufrn / odufrn-downloader

Pacote para baixar os dados do portal de dados abertos da UFRN
https://odufrn.github.io/odufrn-downloader/
MIT License
32 stars 4 forks source link

Resolve #91 - PR de correção de Tests - assert_console #93

Open BrenoBaiardi opened 4 years ago

BrenoBaiardi commented 4 years ago

Pull request inicial para validação,

Se estiver de acordo com o esperado, darei continuidade indo mais afundo na issue e tratando outros pontos levantados

BrenoBaiardi commented 4 years ago

Continuo trabalhando para resolver os item mencionados...

Sobre o PEP8 e o comando mencionado para verificação, não consegui fazer ele ser executado aqui, mas pelo que vi o Travis faz essa verificação, correto?

johnazedo commented 4 years ago

Pelo o que eu vi, as falhas na verificação do Travis são só uns detalhes na codificação, como linhas em branco e/ou espaços em branco.

Screen Shot 2019-10-04 at 13 36 01

johnazedo commented 4 years ago

Para fazer a verificação no seu repositório local, basta instalar o pycodestyle, pip install pycodestyle, e executar o script a seguir:

$ find . -name \*.py -exec pycodestyle --ignore=E402 {} +
BrenoBaiardi commented 4 years ago

O comando que vocês usam me retorna dizendo que não foi possível encontrar os arquivos com esta sintaxe $ find "*.py". Não sei se é questão de sistema, estou utilizando Windows. Pesquisei como utilizar o módulo e fiz as verificações arquivo por arquivo. desculpem a demora, caso existam outros problemas além destes, podem informar.

johnazedo commented 4 years ago

Olá @BrenoBaiardi , acredito também que seja algo do windows. Vou procurar mais sobre como rodar os scripts nessa plataforma e caso eu ache, postarei aqui.

Caso você deseje algo de rápida solução ou uma solução temporária, deixo aqui links de como utilizar o bash do UNIX no prompt de comando.

BrenoBaiardi commented 4 years ago

O travis agora está acusando erros no unittest, vou verificar, mas não acredito que sejam nos códigos que eu alterei.

BrenoBaiardi commented 4 years ago

desculpem à demora de retorno, estou trabalhando ainda nos códigos para entrega final Obrigado pelos direcionamentos

BrenoBaiardi commented 4 years ago

Boa noite Srs, As Testes que encontrei e que achei válidos para alteração foram corrigidos e alterados para receber mensagens específicas.

Quando a complexidade das mensagens de comparação for maior, o método vai validar utilizando a lógica anterior, conforme o comentado por @JohnAzedo . Seria interessante desenvolver uma outra maneira e/ou padronizar todas as mensagens de "não sucesso" que possam ocorrer, dessa forma uma validação mais genérica poderia ser feita utilizando asserções com as strings recebidas, ao invés de utilizar a lógica de Len().

Dada a estrutura dos testes e dos erros que aparecem, uma vez que eles poderiam retornar Exceptions, não seria melhor criar uma ou mais classes herdando Exception que pudesse fazer a validação sem depender dos prints? {mesmo que somente para o caso de problemas mais complexos, e com mais possibilidades de outputs errados)

Agradeço a oportunidade de participar, poderiam validar o build que foi enviado e verificar se o merge é possível, ou se ainda é preciso mais algum progresso