googleapis/google-cloud-java

[java-spanner] test: OptionsTest checks for fixed values being returned from hashCode()

Open

#12.268 aperta il 3 nov 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Java (1082 fork)batch import
api: spannergood first issuehelp wantedtype: cleanup

Metriche repository

Star
 (1763 star)
Metriche merge PR
 (Merge medio 2g 21h) (145 PR mergiate in 30 g)

Descrizione

The Options#hashCode() method does not guarantee to return a fixed value that is equal on all JVM's (at least; the documentation does not guarantee it). In reality it does, because the current implementation only uses hash codes that are guaranteed to be equal on all JVM's. The method should either be documented to have this behavior, or these tests should be replaced with tests that assert that the hashCode and equals methods comply with the general contract regarding these methods.

Guida contributor

Direzione di ricerca
Esamina Options.java e OptionsTest.java nel modulo java spanner del repository google cloud java. Il problema suggerisce che il comportamento del metodo hashCode() dovrebbe essere documentato o i test dovrebbero essere aggiornati per asserire solo il contratto generale. Osserva l'implementazione attuale per vedere se utilizza codici hash coerenti tra JVM. Considera di aggiornare il test per utilizzare assertThat(options).isEqualTo(anotherOptions) o controlli simili che si basano su equals, poiché questo è il modo standard per testare la conformità di hashCode. Se si opta per la documentazione, aggiungi una nota nel Javadoc della classe che indica che il codice hash è deterministico e uguale tra JVM.
Tech stack
java
Dominio
testingbackend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Understanding of equals/hashCode contractFamiliarity with Java testing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60