google / google/s2-geometry-library-java

S2CellUnionTests Line 258-260 should be removed.

Ouverte
#1 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
auto-migrated Priority-Medium Type-Defect
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.