Simplify AssertJ assertions flagged by Sonar (java:S5838)
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 78/100
Rechercherichtung
Wähle ein Modul und kommentiere hier, um diesen Bereich zu beanspruchen. Untersuche in diesem Modul die SonarCloud-Befunde java:S5838; das Issue nennt ComputeTest Zeile 708 und 767 als Beispiele in reader. Ersetze nur Assertions durch spezifischere AssertJ-Formen, behalte die Given/When/Then-Struktur bei und vermeide containsExactly bei großen Arrays. Führe ./mvnw test -pl <module> -am aus; fertig ist es, wenn die Tests bestehen und die Änderung rein testbezogen bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
SonarCloud flags 23 assertions that have a more specific AssertJ form, e.g. assertThat(map.get(k)).isEqualTo(v) should be assertThat(map).containsEntry(k, v). The more specific form gives a better failure message.
Where: reader 17 (e.g. ComputeTest:708, :767), writer 3, and one each in fsst, calcite, integration. List: java:S5838 open issues.
How: pick one module, say so in a comment here, and send one PR. Test-only and behavior-neutral: ./mvnw test -pl <module> -am must stay green. Keep the // Given / // When / // Then structure. See CONTRIBUTING.md.
Beware: AssertJ's containsExactly on big arrays is quadratic; do not use it to replace isEqualTo on arrays.
- Vorherrschende Sprache
- Java
- Sterne
- 3
- Forks
- 2
- Ø Merge
- 9 Std. 19 Min.
- Gemergte PRs (30 T.)
- 154
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus dfa1/vortex-java
-
good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
dfa1/vortex-java#738 ·
Maintainer antworten meist innerhalb von 1 Tag
-
good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
dfa1/vortex-java#737 ·
Maintainer antworten meist innerhalb von 1 Tag
-
help wanted project
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
dfa1/vortex-java#729 ·
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement good first issue
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100
dfa1/vortex-java#728 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100
dfa1/vortex-java#723 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in dfa1/vortex-java
Ähnliche Issues
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
-
Schwierigkeit 2/5 Unter einer Stunde Anfängerfreundlichkeit 84/100
cpp-best-practices/project_validator#251 ·
Maintainer antworten meist innerhalb von 1 Tag
-
backend enhancement
Schwierigkeit 2/5 Ein halber Tag Anfängerfreundlichkeit 70/100
damodharg6/payledger#16 ·
-
[Bug]: An initiative list with no current entry logs an error on every changeEvtl. vergeben @dwizard0 hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100
Maintainer antworten meist innerhalb von 1 Tag