Adopt shared mapper test data from engine-test-data
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 38/100
- Tipo de issue
- Funcionalidade
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- python
- Domínio
- backend, testing-qa
Direção de pesquisa
Comece por flagsmith/mappers.py por volta da linha 182 e inspecione os testes existentes do mapper. Revise o pull request #46 de engine-test-data e os mapper_test_cases, depois compare-os com as implementações vinculadas em PHP e .NET. O trabalho estará concluído quando os casos compartilhados passarem e as chaves de segmento virtual de identity override seguirem o padrão portátil de JSON mais SHA256.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Mapper tests use inline expected context data rather than shared test cases from engine-test-data repository.
Acceptance criteria
- mapper_test_cases pass (see Flagsmith/engine-test-data#46)
- Identity override virtual segment key generation matches cross-SDK portable standard (JSON + SHA256)
Technical context
Current implementation uses non-portable hash:
https://github.com/Flagsmith/flagsmith-python-client/blob/c0d57ec4cd208b79908298e6cf9a3504e5d42fda/flagsmith/mappers.py#L182
Portable standard (JSON + SHA256):
- PHP: https://github.com/Flagsmith/flagsmith-php-client/pull/124/files#diff-5872f438d0e149e75ba3660c6b8e7ec8af446b87ea6a4e70f45f7e8e9e7b9c4dR208
- .NET: https://github.com/Flagsmith/flagsmith-dotnet-client/pull/177/files#diff-f851c822fa0e162b75312b6b9ab5dc31d3631179a33360ec77bbb83f4ae7c50eR249
Shared test data: https://github.com/Flagsmith/engine-test-data/pull/46
Related issues:
- Flagsmith/flagsmith-ruby-client#93
- Flagsmith/flagsmith-java-client#197
- Flagsmith/flagsmith-nodejs-client#228
- Flagsmith/flagsmith-go-client#196
- Flagsmith/flagsmith-rust-flag-engine#26
- Flagsmith/flagsmith-dotnet-client#180
- Linguagem predominante
- Python
- Estrelas
- 24
- Forks
- 12
- Merge médio
- 4d 18h
- PRs com merge (30d)
- 7
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de Flagsmith/flagsmith-python-client
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 68/100
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 65/100
-
Dependency DashboardAberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 68/100
-
Performance regression: v5 is 2–7× slower than v3.8.0 for local evaluationTalvez já em andamento @khvn26 assumiu há 159 dias. Aberta
Flagsmith/flagsmith-python-client#198 · 1 comentário · 2 responsáveis ·
Todas as issues de Flagsmith/flagsmith-python-client
Issues semelhantes
-
needs triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
Mantenedores costumam responder em até 1 dia
-
json_params_matcher fails on falsy top-level JSON primitives (0, False, "")Talvez já em andamento @mayureshsonawane17 assumiu hoje. AbertaWaiting for: Product Owner
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
Mantenedores costumam responder em até 5 dias
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Mantenedores costumam responder em até 1 dia
-
Add .devin pluginAberta
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
ayghri/i-have-adhd#249 ·
Mantenedores costumam responder em até 2 dias
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
modelscope/FunASR#3762 ·
Mantenedores costumam responder em até 1 dia