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

Criar classes de exception #56

Closed itepifanio closed 4 years ago

itepifanio commented 5 years ago

Feature

Hoje tratamos as exceptions de forma bem geral, porém poderíamos criar classes de exceção para nos auxiliarmos a lidar com os erros. Isso ajuda bastante nos testes, tornando-os mais assertivos, visto que o unittest tem a função assertRaises()

Participe do Hacktoberfest!

Contribua com uma issue com a label hacktoberfest e abra um pull request durante o mês de outubro para ganhar os brindes do GitHub! Para se inscrever, acesse https://hacktoberfest.digitalocean.com/register

suv27 commented 4 years ago

Me puedes dar un pouito mas de explicacion de lo que necesitas? Puedo ayudar agregar mas unittest solamente necitatere que me encamines a los files que lo necesitan

itepifanio commented 4 years ago

Hola, @Physsix27

No enumeramos todas las exceptions que queremos escribir, pero tenemos un ejemplo. En el test_file tenemos la function test_can_print_exception_download_packages_from_file pero no tenemos esta exception, solo printamos como self._print_exception(ex) en la class File.

Se tenemos la custom exception podriamos escribir raise CustomException y entonces ir a los tests y escribir assertRaises(CustomException) Si tu puderes escribir una exception initial como esta descrita, entonces ayudaría mucho.

suv27 commented 4 years ago

Ok, deja ver si entiendo, quieres que escriba un custom CustonException pata poder usarla en self._print_exception(CustonException) y luego poder usarla en el test? si eso es lo que quieres puedo ayudar.

Mi Idioma: Espanol

suv27 commented 4 years ago

Puedes darme una descripcion de lo que hara el custonexception cuando tenga esa description puedo poner el PR. ya casi termino

suv27 commented 4 years ago

Hey @itepifanio dejame saber si necesito cambiar algo de mi PR para poder darle merged. Ya esta completo.

PR #95