Atualização do Boleto, Remessa/Retorno CNAB400 do banco UNICRED.

[mbcosta]

Atualização do Boleto, Remessa/Retorno CNAB400 do banco UNICRED. https://github.com/kivanio/brcobranca/issues/214

Trabalho em Andamento

[coveralls]

Coverage decreased (-2.1%) to 97.019% when pulling ed71bf79cc8fbcc7c8bb7fe172f173360c726936 on akretion:unicred_cnab400 into 9bb3defe9e075c9efd94700ab680937c910369a5 on kivanio:master.

[coveralls]

Coverage decreased (-0.8%) to 98.364% when pulling bbcee354677891ea9755fc7f5d120d6db79086ff on akretion:unicred_cnab400 into 9bb3defe9e075c9efd94700ab680937c910369a5 on kivanio:master.

[coveralls]

Coverage decreased (-0.8%) to 98.364% when pulling bbcee354677891ea9755fc7f5d120d6db79086ff on akretion:unicred_cnab400 into 9bb3defe9e075c9efd94700ab680937c910369a5 on kivanio:master.

[kivanio]

@mbcosta Show! Quando finalizar, avisa que faço o merge.

[mbcosta]

ola @kivanio o PR está pronto para revisão, os testes aqui estão retornando erros mas gostaria que você confirmasse se é algo que eu alterei, desculpe a demora no retorno acabei deixando funcional mas faltava a atualização dos outros arquivos.

[kivanio]

@mbcosta falta uma configuração no CI. Rodou na sua máquina todos os testes?

[mbcosta]

ola @kivanio , eu estava vendo de corrigir a questão das aspas duplas e os testes, tinha duas alterações que iriam ser feitas:

Os testes estavam retornando um erro que eu não estava conseguindo identificar que é um campo que existia codigo_transmissao e passou a ser codigo_beneficiario:

Mesmo fazendo um $ grep -rn 'codigo_transmissao' . --color=always para tentar encontrar onde estaria a referencia no caso do UNICRED o comando retorna apenas um campo com esse nome no Santander e não no UNICRED o que eu acabei não entendendo o porque isso estava gerando erro:

[kivanio]

Se mudou o nome, basta mudar o nome nos testes: codigo_beneficiario

[mbcosta]

Certo, o problema é que eu mudei e não encontrei em nenhum arquivo do UNICRED referencia a esse campo apenas no Santander, por isso não entendi o motivo do erro.

[kivanio]

Vi agora que tem vários coisas quebradas, não deveria ter feito o merge 💩 O que eu falei: https://github.com/kivanio/brcobranca/commit/ff6d1f7c95cafe7fe0a05de607a134939ddbbf43

[mbcosta]

humm o erro estava nesse arquivo spec/support/shared_examples/cnab400.rb ( preciso entender melhor a forma como os testes rodam ), não sei se vai afetar os testes mas o campo codigo_beneficiario tem limite de 10 caracteres e é preenchido com zeros para ficar com 20 caracteres, no arquivo .rem eu alterei isso mas no commit que você mencionou está codigo_beneficiario: '12345678901234567890'

Caso seja possível e se você quiser ver de reverter esse "merge" posso ver de criar outro PR, ou você apontando o que precisa ser feito posso ver de fazer nos próximos dias.

[kivanio]

@mbcosta Arrumei todos os testes. Agora quando mandar o PR, já envia com testes!

Comentei os códigos de teste de remessa e retorno do UNICRED, notei que não está feito esses arquivos, certo?

Arrumei o código do boleto e de testes do UNICRED, esse já ficou OK.

[mbcosta]

@kivanio sim da minha parte a ideia é ter sempre os PRs com testes passando, infelizmente esse PR acabou ficando em aberto por bastante tempo, alguns do motivos são porque eu não domino o Ruby e durante o mesmo período em paralelo eu precisei implementar o CNAB usando o BRCobranca no projeto de Localização do Odoo no Brasil( https://github.com/OCA/l10n-brazil/tree/12.0/l10n_br_account_payment_brcobranca ), o foco acabou ficando lá e acabei deixando aqui apenas funcional, bom espero que essa integração traga novos colaboradores ao projeto BRCobranca.

Os commits recentes passaram a causar erro https://github.com/kivanio/brcobranca/pull/228 e sobre o UNICRED 400 ele esta funcional, esse caso foi homologado junto a uma empresa e parece estar tudo de acordo, aqui no projeto estava pendente apenas atualizar os testes, os arquivos de Remessa https://github.com/kivanio/brcobranca/blob/master/spec/fixtures/remessa/remessa-unicred-cnab400.rem e Retorno https://github.com/kivanio/brcobranca/blob/master/spec/arquivos/CNAB400UNICRED.RET foram atualizados de acordo com a especificação dos arquivos na pasta https://github.com/kivanio/brcobranca/tree/master/docs/unicred, as alterações recentes que foram feitas nesse caso do UNICRED 400 passaram a gerar erro: