elastic/elasticsearch

Share number based field mapper test code with other modules

Open

#92.947 aperta il 16 gen 2023

Vedi su GitHub
 (17 commenti) (2 reazioni) (0 assegnatari)Java (25.882 fork)batch import
:Search Foundations/Mapping>testTeam:Search Foundationsgood first issue

Metriche repository

Star
 (76.700 star)
Metriche merge PR
 (Merge medio 2g) (1000 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina gli esistenti NumberFieldMapperTests e WholeNumberFieldMapperTests per identificare metodi di test comuni. Quindi rifattorizzali in una classe base condivisa nel modulo del framework di test.
Tech stack
java
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaJUnit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70