elastic/elasticsearch

Share number based field mapper test code with other modules

Aberta

#92.947 aberto em 16 de jan. de 2023

Ver no GitHubGuia do colaborador
 (18 comentários) (2 reações) (0 responsável)Java (25.882 forks)batch import
:Search Foundations/Mapping>testTeam:Search Foundationsgood first issue

Métricas do repositório

Stars
 (76.700 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Move number based field mapper test infrastructure to the test framework module. This would allow other number based field mapper to use the same test infrastructure that the core number field mappers are using.

Currently UnsignedLongFieldMapper and ScaledFloatFieldMapper have their own concrete tests, but could instead make use of NumberFieldMapperTests or WholeNumberFieldMapperTests. This avoids duplicates tests and testing logic.

Furthermore this can avoid that non core field mappers don't get forgotten when new features / behaviour is added to core field mappers.

Originates from #92917

Guia do colaborador

Direção de pesquisa
Examine os testes existentes NumberFieldMapperTests e WholeNumberFieldMapperTests para identificar métodos de teste comuns. Em seguida, refatore os em uma classe base compartilhada no módulo de framework de teste.
Pilha de tecnologia
java
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaJUnit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70