S2CellUnionTests Line 258-260 should be removed.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 35/100
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.
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 [email protected] on 9 Jul 2013 at 1:39
- Langage dominant
- Java
- Étoiles
- 587
- Forks
- 231
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de google/s2-geometry-library-java
-
Possible bug in compareEdgesOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
google/s2-geometry-library-java#56 · 1 commentaire · 1 réaction ·
-
Infinite loop in coverer builderOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Build failure when mvn compileOuverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 68/100
google/s2-geometry-library-java#47 · 2 commentaires ·
-
Added new ant target docs: "Generate documentation"Peut-être pris @ThomasThelen l’a pris il y a 204 jours. Ouverteauto-migrated Type-Patch
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
google/s2-geometry-library-java#2 · 1 commentaire ·
-
FR: Add bazel supportOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
Toutes les issues de google/s2-geometry-library-java
Issues similaires
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
open-metadata/OpenMetadata#36463 ·
Les mainteneurs répondent en général sous 1 jour
-
bug web-server
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
jooby-project/jooby#4035 ·
Les mainteneurs répondent en général sous 1 jour
-
afk-ok area:data-quality importer size:S
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
enorm-labs/event-junkie#4140 ·
Les mainteneurs répondent en général sous 1 jour
-
L: javascript
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
dependabot/dependabot-core#16744 ·
Les mainteneurs répondent en général sous 1 jour
-
[Security] NOTICE.md links to hijacked domain kineticjs.com, which now redirects to a betting siteOuverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 83/100
eclipse-ee4j/jersey#6144 ·
Les mainteneurs répondent en général sous 1 jour