-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Resolve #48 - Mostrando arquivos contidos em grupos e pacotes #55
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ficou faltando adicionar os testes dessas novas funções e na documentação também
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Colocar as novas funções na doc geral.
docs/guia-group.md
Outdated
@@ -90,3 +90,27 @@ print(list_groups) | |||
# Output: | |||
# ['pesquisa'] | |||
``` | |||
|
|||
## print_files_from_group | |||
Printa no terminal a lista de arquivos referentes ao grupo de entrada. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Printa" é uma palavra que não existe no português.
docs/guia-package.md
Outdated
@@ -161,3 +161,29 @@ print(list_discentes) | |||
# Output: | |||
# ['dados-complementares-de-discentes', 'dados-socio-economicos-de-discentes', 'discentes'] | |||
``` | |||
|
|||
## print_files_from_package |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Colocar em ordem alfabética.
def __init__(self): | ||
self.url_base = 'http://dados.ufrn.br/' | ||
self.url_action = self.url_base + 'api/action/' | ||
|
||
def _print_exception(self, ex: Exception): | ||
def _print_exception(self, ex: Exception, | ||
msg: str = MSG_ERRORS['download_error']): | ||
"""Imprime mensagem padrão para exceções.""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mensagem não é padrão se você recebe ela como parâmetro. Existe a mensagem padrão, mas ela pode ser modificada.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Não entendi isso, @alvarofpp, poderia elaborar melhor?
Antes tinha uma mensagem padrão, entretanto ela não fazia sentido pra minha utilização, só coloquei a mensagem padrão como global e deixei ela por default, podendo ser modificada posteriormente
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nem me lembro o que eu queria dizer aí.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mas me lembro de ter comentado pessoalmente com você que vc tem duas MSG_ERRORS
, uma na classe Env
e outra na classe Package
. Seria interessante você colocar elas na classe Env
, visto que Package
herda dela, assim a variável fica em apenas um só lugar.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Acho que queria que mudasse a nomenclatura no dosctring, já que existe uma mensagem padrão mas nem sempre ela é exibida
odufrn_downloader/modules/Group.py
Outdated
@@ -137,3 +137,8 @@ def search_related_groups(self, keyword: str, | |||
) | |||
|
|||
return related | |||
|
|||
def print_files_from_group(self, name: str): | |||
request = self._request_get(self.url_group + name) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Usa a variável que armazena grupos.
odufrn_downloader/modules/Package.py
Outdated
@@ -18,6 +18,8 @@ class Package(Env, FilterMixin): | |||
instância da classe Tag usada na classe. | |||
""" | |||
|
|||
MSG_ERRORS = {'none_package': 'Nenhum pacote foi encontrado'} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seria melhor colocar essa variável na Env
, visto que se usa em mais de uma classe.
tests/test_group.py
Outdated
def test_can_print_files_from_group(self): | ||
"""Verifica se os arquivos de um grupo podem ser impresso na tela.""" | ||
assert_console( | ||
lambda: self.ufrn_data.print_files_from_group('processos')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quebra o parenteses para facilitar saber onde termina.
tests/test_group.py
Outdated
def test_can_print_files_from_group_with_typo(self): | ||
"""Verifica se o tratamento de erro com o Levenshtein funciona.""" | ||
assert_console( | ||
lambda: self.ufrn_data.print_files_from_package('process')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mesma coisa da anterior.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolver o que foi levantado por Álvaro.
def __init__(self): | ||
self.url_base = 'http://dados.ufrn.br/' | ||
self.url_action = self.url_base + 'api/action/' | ||
|
||
def _print_exception(self, ex: Exception): | ||
def _print_exception(self, ex: Exception, | ||
msg: str = MSG_ERRORS['download_error']): | ||
"""Imprime mensagem padrão para exceções.""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mas me lembro de ter comentado pessoalmente com você que vc tem duas MSG_ERRORS
, uma na classe Env
e outra na classe Package
. Seria interessante você colocar elas na classe Env
, visto que Package
herda dela, assim a variável fica em apenas um só lugar.
…ões da documentação em ordem alfabética. Centraliza mensagens globais de erro na classe Env. Adiciona formatação textual da lista levenshtein para string. Quebra parenteses de linhas mal formatadas em testes
Resolve #48