googleapis/google-cloud-java

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

Open

#12 268 ouverte le 3 nov. 2020

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Java (1 082 forks)batch import
api: spannergood first issuehelp wantedtype: cleanup

Métriques du dépôt

Stars
 (1 763 stars)
Métriques de merge PR
 (Merge moyen 2j 21h) (145 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez Options.java et OptionsTest.java dans le module java spanner du dépôt google cloud java. Le problème suggère que le comportement de la méthode hashCode() doit être documenté ou que les tests doivent être mis à jour pour n'asserter que le contrat général. Examinez l'implémentation actuelle pour voir si elle utilise des codes de hachage cohérents entre les JVM. Envisagez de mettre à jour le test pour utiliser assertThat(options).isEqualTo(anotherOptions) ou des vérifications similaires qui reposent sur equals, car c'est la manière standard de tester la conformité de hashCode. Si vous optez pour la documentation, ajoutez une note dans le Javadoc de la classe indiquant que le code de hachage est déterministe et égal entre les JVM.
Stack technique
java
Domaine
testingbackend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Understanding of equals/hashCode contractFamiliarity with Java testing
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60