google / google/s2-geometry-library-java

S2CellUnionTests Line 258-260 should be removed.

Abierto
#1 0 comentarios 0 reacciones 0 asignados Ver en GitHub
auto-migrated Priority-Medium Type-Defect
Lenguaje dominante
Java
Estrellas
587
Forks
231
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

```
Line 258-260 as below should be removed:

if (!u.contains(xId)) {
u.getIntersection(xCells, yId);
}

It is extraneous and guarantees the subsequent assertTrue(u.contains(xId)); on
line 261 will always work even though this is the line that is being ultimately
tested.
```

Original issue reported on code.google.com by `uvico...@gmail.com` on 9 Jul 2013 at 1:39

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Abre S2CellUnionTests e inspecciona las líneas 258-261, centrándote en cómo se hace alcanzable actualmente la assertion. Confirma que la prueba sigue ejercitando el comportamiento previsto sin el bloque superfluo, luego ejecuta las pruebas relevantes de S2CellUnion y verifica que la assertion siga siendo significativa.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
testing
Tipo de issue
Refactorización
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.