googleapis/google-cloud-java

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

Open

#12.268 aberto em 3 de nov. de 2020

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Java (1.082 forks)batch import
api: spannergood first issuehelp wantedtype: cleanup

Métricas do repositório

Stars
 (1.763 stars)
Métricas de merge de PR
 (Mesclagem média 2d 21h) (145 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Examine Options.java e OptionsTest.java no módulo java spanner do repositório google cloud java. O problema sugere que o comportamento do método hashCode() deve ser documentado ou os testes devem ser atualizados para apenas afirmar o contrato geral. Observe a implementação atual para ver se ela usa códigos hash consistentes entre JVMs. Considere atualizar o teste para usar assertThat(options).isEqualTo(anotherOptions) ou verificações semelhantes que dependem de equals, pois essa é a maneira padrão de testar a conformidade de hashCode. Se optar pela documentação, adicione uma nota no Javadoc da classe indicando que o código hash é determinístico e igual entre JVMs.
Pilha de tecnologia
java
Domain
testingbackend
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
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.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Understanding of equals/hashCode contractFamiliarity with Java testing
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60