search

imobanco / bb-wrapper

https://pypi.org/project/bb-wrapper/
GNU General Public License v3.0
8 stars 1 forks source link

o parse deve ser de unicode para alfanumérico e não ascii! #11

Closed rodrigondec closed 3 years ago

rodrigondec commented 3 years ago

Resumo

codecov[bot] commented 3 years ago

Codecov Report

Merging #11 (77c1d42) into main (27acfa8) will increase coverage by 0.22%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
+ Coverage   71.82%   72.04%   +0.22%     
==========================================
  Files           7        7              
  Lines         252      254       +2     
==========================================
+ Hits          181      183       +2     
  Misses         71       71
Impacted Files Coverage Δ
bb_wrapper/utils.py 100.00% <100.00%> (ø)
bb_wrapper/wrapper/cobrancas.py 77.08% <100.00%> (ø)
Impacted Files Coverage Δ
bb_wrapper/utils.py 100.00% <100.00%> (ø)
bb_wrapper/wrapper/cobrancas.py 77.08% <100.00%> (ø)
rodrigondec commented 3 years ago

Aprovado!

Estava achando estranho que fossem ser aceitos os carateres mostrados nos testes do PR anterior, como `'?.

Sim, estão na tabela ASC-II. Mas confirmei, o BB não aceita eles!

Então vamos de duas etapas, tenta transliterar o que consegue e depois capa o que não serve :rofl: