Este PR corrige retorno de SuitableXML.format() para retornar o valor original da entrada, seja de arquivo ou string. Este valor só retornará caso não seja possível fazer o parse do XML.
Onde a revisão poderia começar?
Em src/scielo/bin/xml/prodtools/utils/xml_utils.py
Como este poderia ser testado manualmente?
Execute o XPM com arquivo XML inválido.
O programa não deve quebrar durante a tentativa de adequar o XML (SuitableXML) e conseguirá direcionar para o validador
Execute o primeiro passo com um XML válido e o programa também não deve quebrar.
Algum cenário de contexto que queira dar?
Foi possível identificar este problema por conta de outro erro que já foi corrigido anteriormente. Forçando alguns testes foi possível identificar uma situação em que, sem a correção, o XML declaration e o DOCTYPE eram duplicados caso o conteúdo original os possuísse.
Screenshots
N/A
Quais são tickets relevantes?
3224, apesar do erro relatado no ticket não ocorrer mais.
O que esse PR faz?
Este PR corrige retorno de
SuitableXML.format()
para retornar o valor original da entrada, seja de arquivo ou string. Este valor só retornará caso não seja possível fazer o parse do XML.Onde a revisão poderia começar?
Em
src/scielo/bin/xml/prodtools/utils/xml_utils.py
Como este poderia ser testado manualmente?
SuitableXML
) e conseguirá direcionar para o validadorAlgum cenário de contexto que queira dar?
Foi possível identificar este problema por conta de outro erro que já foi corrigido anteriormente. Forçando alguns testes foi possível identificar uma situação em que, sem a correção, o XML declaration e o DOCTYPE eram duplicados caso o conteúdo original os possuísse.
Screenshots
N/A
Quais são tickets relevantes?
3224, apesar do erro relatado no ticket não ocorrer mais.
Referências
.