google / google/s2-geometry-library-java
S2CellUnionTests Line 258-260 should be removed.
- Langage dominant
- Java
- Étoiles
- 587
- Forks
- 231
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
```
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
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Ouvrez S2CellUnionTests et examinez les lignes 258-261, en vous concentrant sur la manière dont l’assertion est actuellement rendue atteignable. Confirmez que le test vérifie toujours le comportement attendu sans le bloc superflu, puis exécutez les tests S2CellUnion pertinents et vérifiez que l’assertion reste pertinente.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java
- Domaine
- testing
- Type d'issue
- Refactorisation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100