Refatoração da biblioteca do CNAB #3
Conversation
mileo
commented
Jul 10, 2017
•
mileo
commented
- PEP8
- Refatoração para suportar outras transações alem de cobrança de boleto bancário
- Melhoria nos testes
- Inclusão do item 3.1 - Pagamentos (Pagamento Através de Crédito em Conta, cheque, OP, DOC, TED ou Pagamento com Autenticação).
…to E e estava causando conflitos no cnab240
Banco Bradesco
Feature/importacao pagamento
Layout sicoob
| f.close() | ||
|
|
||
| campos = spec.get('campos') | ||
| for campo in campos.values(): |
There was a problem hiding this comment.
campos.values() vai falhar com attribute error se spec.get('campos') for None.
Se spec.get('campos') não deve retornar None, então o certo é usar:
spec['campos']
Se campos pode estar ausente de spec, então o certo é usar:
spec.get('campos', {})
| """Tentativa de atribuicao de valor mais longo que o campo suportaia""" | ||
|
|
||
| class NumDigitosExcedidoError(AtribuicaoCampoError): | ||
| """Tentativa de atribuicao de valor mais longo que o campo suportaia""" |
|
|
||
| class LoteCobranca(LoteBase): | ||
| pass | ||
| # HeaderCls = LoteCobranca.banco.registros.HeaderLoteCobranca |
| @@ -25,41 +29,54 @@ def valor(self): | |||
| def valor(self, valor): | |||
| if self.formato == 'alfa': | |||
| if not isinstance(valor, unicode): | |||
| print ("{0} - {1}".format(self.nome, self.valor)) | |||
| @@ -153,7 +171,7 @@ def carregar(self, registro_str): | |||
| try: | |||
| campo.valor = Decimal(dec) | |||
| except InvalidOperation: | |||
| raise # raise custom? | |||
| raise # raise custom? | |||
There was a problem hiding this comment.
Qual a função desse except: raise aqui? Parece melhor remover...
| @@ -1,17 +1,17 @@ | |||
|
|
|||
| try: | |||
| try: | |||
There was a problem hiding this comment.
Isso é necessário? unitest2 só é útil para Python 2.6
| @@ -3,22 +3,22 @@ | |||
|
|
|||
| setup( | |||
| name='cnab240', | |||
| version='0.01', | |||
| version='0.02.1', | |||
There was a problem hiding this comment.
0.02.1 não é um identificador de versão válido. Setuptools e pip normalizam esse número para 0.2.1
| author='Tracy Web Technologies', | ||
| author_email='[email protected]', | ||
| url='https://github.com/TracyWebTech/cnab240', | ||
| packages=find_packages(), | ||
| package_data={ | ||
| 'cnab240': ['bancos/*/*/*.json'] | ||
| }, | ||
| zip_safe=False, | ||
| zip_safe=True, |
There was a problem hiding this comment.
Isto é verdade? Existem vários arquivos .json dentro deste pacote...
| - "python -m unittest discover || unit2 discover" | ||
|
|
||
| after_success: | ||
| coveralls |
There was a problem hiding this comment.
faltou quebra de linha no fim do arquivo.
| @@ -0,0 +1,4 @@ | |||
| unittest2 | |||
There was a problem hiding this comment.
Todos estes arquivos são backports do Python 2.7 para versões anteriores.
Existe alguma necessidade de testar esse módulo com Python 2.6 ou menor? Senão, acho melhor jogar fora este arquivo, ou colocar uma única linha:
.
