google / google/s2-geometry-library-java
S2CellUnionTests Line 258-260 should be removed.
- 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
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