Repository navigation
Teste do pattern de Decimal não depende da versão do pydantic - #216
Merged
Merged
Conversation
O teste tirava o schema de um modelo com campo Decimal e esperava a recusa do lookahead. Da 2.14 em diante o pydantic não emite pattern para Decimal, o schema passa e o teste falha em todo branch, porque o CI resolve as dependências a cada execução. O schema do teste passa a trazer escrito o pattern que o pydantic emite até a 2.13, que é o caso que a recusa protege.
O CHANGELOG e o guia de providers citavam o pattern de Decimal como exemplo do que o preflight do codex recusa. O pydantic 2.14 deixou de emitir pattern para o tipo, e um campo Decimal passa pelo preflight; a frase passa a nomear as versões em que o exemplo vale.
Owner
Author
Revisão independente antes do mergeUm subagente novo, sem acesso ao raciocínio da implementação, revisou o commit O que ele conferiu por execução
Achados fora do diff e o que foi feito
Recusado
O revisor listou outros testes de |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
O quê
tests/test_codex.py::TestStrictPydanticSchema::test_pattern_do_decimal_e_recusado_pelo_lookaheadfalha em todo branch desde que o CI passou a instalar o pydantic 2.14. O teste tirava o schema de um modelo com campoDecimale esperava que o preflight do codex recusasse opatterncom lookahead. Em toda a série 2.13 (conferido até a 2.13.5) o schema deDecimaltraz^(?!^[-+.]*$)[+-]?0*\d*\.?\d*$; do 2.14.0 em diante não trazpattern, o schema passa e opytest.raisesnão vê a exceção. Como ouv.locknão é versionado, o CI resolve as dependências a cada execução.Como
O schema do teste passa a trazer escrito o
patternque o pydantic emite até a 2.13, que é o caso que a recusa protege. Nenhum código desrc/muda.O
CHANGELOG.mde o guia de providers citavam opatterndeDecimalcomo exemplo do que o preflight recusa. Com o pydantic 2.14 um campoDecimalpassa pelo preflight, e a frase passa a nomear as versões em que o exemplo vale.Testes
codex: 1248 passam, cobertura de ramos em 100%.(?!removida decodex.py, o teste falha.ruff format --check,ruff checkety checkpassam.