search

BoletoNet / BoletoNetCore

Versão do BoletoNet para .NET Core
MIT License
186 stars 117 forks source link

Testes Unitários no SICRED não estão passando #4

Closed olavorn closed 5 years ago

olavorn commented 5 years ago

Algum interessado que trabalhe com o SICRED?

Verificar porque os testes no SICRED não estão passando.

rafd75 commented 5 years ago

Vixe, esse é o único banco que não utilizo aqui...

Não tenho nenhum exemplo válido para fazer o teste fechar.

carloscds commented 5 years ago

@olavorn Pode fixar o teste?

olavorn commented 5 years ago 
D:\Dev\BoletoNetCore>dotnet test
Build iniciada, aguarde...
Build iniciada, aguarde...
Build iniciada, aguarde...
Build concluído.

Skipping running test for project D:\Dev\BoletoNetCore\Boleto2.Net\Boleto2.Net.csproj. To run tests with dotnet test add "<IsTestProject>true<IsTestProject>" property to project file.
Skipping running test for project D:\Dev\BoletoNetCore\Boleto2.Net\Boleto2.Net.csproj. To run tests with dotnet test add "<IsTestProject>true<IsTestProject>" property to project file.
Build concluído.

Execução de teste para D:\Dev\BoletoNetCore\Boleto2.Net.Testes\bin\Debug\netcoreapp2.2\Boleto2.Net.Testes.dll(.NETCoreApp,Version=v2.2)
Microsoft (R) Test Execution Command Line Tool Version 15.9.0
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
Build concluído.

Skipping running test for project D:\Dev\BoletoNetCore\Boleto2.Net.Pdf\Boleto2.Net.Pdf.csproj. To run tests with dotnet test add "<IsTestProject>true<IsTestProject>" property to project file.
Skipping running test for project D:\Dev\BoletoNetCore\Boleto2.Net.Pdf\Boleto2.Net.Pdf.csproj. To run tests with dotnet test add "<IsTestProject>true<IsTestProject>" property to project file.
Failed   Sicredi_1_01_BoletoOK(500,"4","4","4","19/200004-8","74894629800000500001119200004801560585305103","74891.11927 00004.801569 05853.051034 4 62980000050000",2015,1,4)
Error Message:
 System.Exception : Nosso número (4) deve conter 9 dígitos.
Stack Trace:
   at Boleto2Net.BancoSicrediCarteira1.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\Carteiras\BancoSicredi\BancoSicrediCarteira1.cs:line 48
   at Boleto2Net.BancoSicredi.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\BancoSicredi.cs:line 39
   at Boleto2Net.Boleto.ValidarDados() in D:\Dev\BoletoNetCore\Boleto2.Net\Boleto\Boleto.cs:line 155
   at Boleto2Net.Testes.BancoSicrediCarteira1Tests.Sicredi_1_01_BoletoOK(Decimal valorTitulo, String nossoNumero, String numeroDocumento, String digitoVerificador, String nossoNumeroFormatado, String codigoDeBarras, String linhaDigitavel, Int32[] anoMesDia) in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\BancoSicrediCarteira1Tests.cs:line 60
Failed   Sicredi_1_01_BoletoOK(400,"3","4","4","19/200003-0","74894740700000400001119200003001560585305101","74891.11927 00003.001567 05853.051018 4 74070000040000",2018,1,17)
Error Message:
 System.Exception : Nosso número (3) deve conter 9 dígitos.
Stack Trace:
   at Boleto2Net.BancoSicrediCarteira1.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\Carteiras\BancoSicredi\BancoSicrediCarteira1.cs:line 48
   at Boleto2Net.BancoSicredi.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\BancoSicredi.cs:line 39
   at Boleto2Net.Boleto.ValidarDados() in D:\Dev\BoletoNetCore\Boleto2.Net\Boleto\Boleto.cs:line 155
   at Boleto2Net.Testes.BancoSicrediCarteira1Tests.Sicredi_1_01_BoletoOK(Decimal valorTitulo, String nossoNumero, String numeroDocumento, String digitoVerificador, String nossoNumeroFormatado, String codigoDeBarras, String linhaDigitavel, Int32[] anoMesDia) in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\BancoSicrediCarteira1Tests.cs:line 60
Failed   Sicredi_1_01_BoletoOK(800,"10","10","3","19/200010-2","74893787000000800001119200010201560585305102","74891.11927 00010.201564 05853.051026 3 78700000080000",2019,4,25)
Error Message:
 System.Exception : Nosso número (10) deve conter 9 dígitos.
Stack Trace:
   at Boleto2Net.BancoSicrediCarteira1.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\Carteiras\BancoSicredi\BancoSicrediCarteira1.cs:line 48
   at Boleto2Net.BancoSicredi.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\BancoSicredi.cs:line 39
   at Boleto2Net.Boleto.ValidarDados() in D:\Dev\BoletoNetCore\Boleto2.Net\Boleto\Boleto.cs:line 155
   at Boleto2Net.Testes.BancoSicrediCarteira1Tests.Sicredi_1_01_BoletoOK(Decimal valorTitulo, String nossoNumero, String numeroDocumento, String digitoVerificador, String nossoNumeroFormatado, String codigoDeBarras, String linhaDigitavel, Int32[] anoMesDia) in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\BancoSicrediCarteira1Tests.cs:line 60
Failed   Sicredi_1_01_BoletoOK(900,"9","9","6","19/200009-9","74896787700000900001119200009901560585305100","74891.11927 00009.901562 05853.051000 6 78770000090000",2019,5,2)
Error Message:
 System.Exception : Nosso número (9) deve conter 9 dígitos.
Stack Trace:
   at Boleto2Net.BancoSicrediCarteira1.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\Carteiras\BancoSicredi\BancoSicrediCarteira1.cs:line 48
   at Boleto2Net.BancoSicredi.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\BancoSicredi.cs:line 39
   at Boleto2Net.Boleto.ValidarDados() in D:\Dev\BoletoNetCore\Boleto2.Net\Boleto\Boleto.cs:line 155
   at Boleto2Net.Testes.BancoSicrediCarteira1Tests.Sicredi_1_01_BoletoOK(Decimal valorTitulo, String nossoNumero, String numeroDocumento, String digitoVerificador, String nossoNumeroFormatado, String codigoDeBarras, String linhaDigitavel, Int32[] anoMesDia) in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\BancoSicrediCarteira1Tests.cs:line 60
Failed   Sicredi_1_01_BoletoOK(603.56d,"5","5","6","19/200005-6","74896790500000603561119200005601560585305109","74891.11927 00005.601562 05853.051091 6 79050000060356",2019,5,30)
Error Message:
 System.Exception : Nosso número (5) deve conter 9 dígitos.
Stack Trace:
   at Boleto2Net.BancoSicrediCarteira1.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\Carteiras\BancoSicredi\BancoSicrediCarteira1.cs:line 48
   at Boleto2Net.BancoSicredi.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\BancoSicredi.cs:line 39
   at Boleto2Net.Boleto.ValidarDados() in D:\Dev\BoletoNetCore\Boleto2.Net\Boleto\Boleto.cs:line 155
   at Boleto2Net.Testes.BancoSicrediCarteira1Tests.Sicredi_1_01_BoletoOK(Decimal valorTitulo, String nossoNumero, String numeroDocumento, String digitoVerificador, String nossoNumeroFormatado, String codigoDeBarras, String linhaDigitavel, Int32[] anoMesDia) in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\BancoSicrediCarteira1Tests.cs:line 60
Failed   Sicredi_1_01_BoletoOK(300,"2","2","7","19/200002-1","74897790500000300001119200002101560585305102","74891.11927 00002.101566 05853.051026 7 79050000030000",2019,5,30)
Error Message:
 System.Exception : Nosso número (2) deve conter 9 dígitos.
Stack Trace:
   at Boleto2Net.BancoSicrediCarteira1.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\Carteiras\BancoSicredi\BancoSicrediCarteira1.cs:line 48
   at Boleto2Net.BancoSicredi.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\BancoSicredi.cs:line 39
   at Boleto2Net.Boleto.ValidarDados() in D:\Dev\BoletoNetCore\Boleto2.Net\Boleto\Boleto.cs:line 155
   at Boleto2Net.Testes.BancoSicrediCarteira1Tests.Sicredi_1_01_BoletoOK(Decimal valorTitulo, String nossoNumero, String numeroDocumento, String digitoVerificador, String nossoNumeroFormatado, String codigoDeBarras, String linhaDigitavel, Int32[] anoMesDia) in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\BancoSicrediCarteira1Tests.cs:line 60
Failed   Sicredi_1_01_BoletoOK(200,"6","6-1/3","5","19/200006-4","74895810800000200001119200006401560585305104","74891.11927 00006.401566 05853.051042 5 81080000020000",2019,12,19)
Error Message:
 System.Exception : Nosso número (6) deve conter 9 dígitos.
Stack Trace:
   at Boleto2Net.BancoSicrediCarteira1.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\Carteiras\BancoSicredi\BancoSicrediCarteira1.cs:line 48
   at Boleto2Net.BancoSicredi.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\BancoSicredi.cs:line 39
   at Boleto2Net.Boleto.ValidarDados() in D:\Dev\BoletoNetCore\Boleto2.Net\Boleto\Boleto.cs:line 155
   at Boleto2Net.Testes.BancoSicrediCarteira1Tests.Sicredi_1_01_BoletoOK(Decimal valorTitulo, String nossoNumero, String numeroDocumento, String digitoVerificador, String nossoNumeroFormatado, String codigoDeBarras, String linhaDigitavel, Int32[] anoMesDia) in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\BancoSicrediCarteira1Tests.cs:line 60
Failed   Sicredi_1_01_BoletoOK(200,"7","6-2/3","8","19/200007-2","74898813900000200001119200007201560585305100","74891.11927 00007.201569 05853.051000 8 81390000020000",2020,1,19)
Error Message:
 System.Exception : Nosso número (7) deve conter 9 dígitos.
Stack Trace:
   at Boleto2Net.BancoSicrediCarteira1.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\Carteiras\BancoSicredi\BancoSicrediCarteira1.cs:line 48
   at Boleto2Net.BancoSicredi.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\BancoSicredi.cs:line 39
   at Boleto2Net.Boleto.ValidarDados() in D:\Dev\BoletoNetCore\Boleto2.Net\Boleto\Boleto.cs:line 155
   at Boleto2Net.Testes.BancoSicrediCarteira1Tests.Sicredi_1_01_BoletoOK(Decimal valorTitulo, String nossoNumero, String numeroDocumento, String digitoVerificador, String nossoNumeroFormatado, String codigoDeBarras, String linhaDigitavel, Int32[] anoMesDia) in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\BancoSicrediCarteira1Tests.cs:line 60
Failed   Sicredi_1_01_BoletoOK(200,"8","6-3/3","4","19/200008-0","74894817000000200001119200008001560585305105","74891.11927 00008.001562 05853.051059 4 81700000020000",2020,2,19)
Error Message:
 System.Exception : Nosso número (8) deve conter 9 dígitos.
Stack Trace:
   at Boleto2Net.BancoSicrediCarteira1.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\Carteiras\BancoSicredi\BancoSicrediCarteira1.cs:line 48
   at Boleto2Net.BancoSicredi.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\BancoSicredi.cs:line 39
   at Boleto2Net.Boleto.ValidarDados() in D:\Dev\BoletoNetCore\Boleto2.Net\Boleto\Boleto.cs:line 155
   at Boleto2Net.Testes.BancoSicrediCarteira1Tests.Sicredi_1_01_BoletoOK(Decimal valorTitulo, String nossoNumero, String numeroDocumento, String digitoVerificador, String nossoNumeroFormatado, String codigoDeBarras, String linhaDigitavel, Int32[] anoMesDia) in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\BancoSicrediCarteira1Tests.cs:line 60
Failed   Sicredi_1_REM400
Error Message:
 System.Exception : Nosso número (92) deve conter 9 dígitos.
Stack Trace:
   at Boleto2Net.BancoSicrediCarteira1.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\Carteiras\BancoSicredi\BancoSicrediCarteira1.cs:line 48
   at Boleto2Net.BancoSicredi.FormataNossoNumero(Boleto boleto) in D:\Dev\BoletoNetCore\Boleto2.Net\Banco\BancoSicredi.cs:line 39
   at Boleto2Net.Boleto.ValidarDados() in D:\Dev\BoletoNetCore\Boleto2.Net\Boleto\Boleto.cs:line 155
   at Boleto2Net.Testes.Utils.GerarBoleto(IBanco banco, Int32 i, String aceite, Int32 NossoNumeroInicial) in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\Utils.cs:line 142
   at Boleto2Net.Testes.Utils.GerarBoletos(IBanco banco, Int32 quantidadeBoletos, String aceite, Int32 NossoNumeroInicial) in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\Utils.cs:line 79
   at Boleto2Net.Testes.Utils.TestarHomologacao(IBanco banco, TipoArquivo tipoArquivo, String nomeCarteira, Int32 quantidadeBoletos, Boolean gerarBoletoPdfHtml, String aceite, Int32 NossoNumeroInicial) in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\Utils.cs:line 150
   at Boleto2Net.Testes.BancoSicrediCarteira1Tests.Sicredi_1_REM400() in D:\Dev\BoletoNetCore\Boleto2.Net.Testes\BancoSicrediCarteira1Tests.cs:line 34

Total tests: 130. Passed: 120. Failed: 10. Skipped: 0.
Test Run Failed.
Test execution time: 1,4993 Minutes

Ele passava na versão original?

carloscds commented 5 years ago

@olavorn Nao esta quebrando no build.

olavorn commented 5 years ago

@carloscds não mesmo... buildando normal. E os outros testes passam todos. Ele só não tá passando os testes do Sicredi mesmo, mas vi que a construção do Nosso número lá tá meio zoada..

Total tests: 130. Passed: 120. Failed: 10. Skipped: 0.
olavorn commented 5 years ago

Seria bom marcar adicionar uma tag "Help Wanted" nessa issue...

carloscds commented 5 years ago

@rafd75 Poderia verificar ?

carloscds commented 5 years ago

@olavorn @rafd75 A ultima alteração no método que está quebrando o teste foi feita pelo @denerkrebs

olavorn commented 5 years ago

Mas teria como confirmar se na branch original esses testes passavam?

carloscds commented 5 years ago

@olavorn Nao esta passando.

rafd75 commented 5 years ago

@olavorn @rafd75 A ultima alteração no método que está quebrando o teste foi feita pelo @denerkrebs

@carloscds - Como eu não tenho sicred aqui (em produção), não tenho como garantir que o teste é válido.

Normalmente, para gerar um teste, eu primeiro faço a homologação com o banco, e considerando que está correto, gero 9 boletos com diferentes dígitos da linha digitavel (de 1 a 9) e vou alterando os valores até bater.

Melhor que alguém que tenha em produção faça isso.

Talvez o @marcosgerene (não sei como linkar ele aqui nesse projeto) pode ajudar... Ele está fazendo ajustes no Sicred do Boleto2Net Vou mandar um email para ele...

marcosgerene commented 5 years ago

@rafd75 estamos validando o boleto, o único erro que tivemos foi por causa da duplicata mercantil, a qual fiz um PR...

Não sei se é ultil a informação, mas só está implementado o CNAB400 do Sicredi.

Vou tentar entender o problema teste unitário e vou validar meus boletos no banco, posto os resultados aqui

marcosgerene commented 5 years ago

@rafd75 boletos validados.

Me perdoem a ignorancia, mas não sou um profundo conhecedor de testes... como faço para executar os testes unitário?

Segue como fiz:

tentando rodar testes

Das duas uma: ou ta tudo ok ou não rodou, pois foi MUITO rápido.

olavorn commented 5 years ago

Eu uso dotnet test a partir da linha de comando.

carloscds commented 5 years ago

@marcosgerene Faz o Build all da Solution e depois rode os testes. Testes executados com sucesso ficam em verde e com erro em vermelho. NA sua imagem parece nao terem rodado.

olavorn commented 5 years ago

image (Na branch do projeto renomeado) @marcosgerene aguardando seu PR pra eu fazer um merge com a minha branch local.

marcosgerene commented 5 years ago

Caros,

Quanto ao teste (Core), segue o motivo:

testes_core_motivo

O problema está aqui:

image

O erro foi gerado nesse PR: https://github.com/BoletoNet/boleto2net/commit/474ea97e591a7972a6353805649399c74a5e2e35