Simplify AssertJ assertions flagged by Sonar (java:S5838)
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
Línea de trabajo
Elige un módulo y comenta aquí para reclamar ese alcance. En ese módulo, revisa los hallazgos de SonarCloud java:S5838; el issue menciona ComputeTest en las líneas 708 y 767 como ejemplos en reader. Reemplaza solo las aserciones por formas más específicas de AssertJ, conserva la estructura Given/When/Then y evita containsExactly para arrays grandes. Ejecuta ./mvnw test -pl <module> -am; está terminado cuando las pruebas pasen y el cambio siga siendo solo de pruebas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Java
- Estrellas
- 3
- Forks
- 2
- Merge medio
- 9 h 19 min
- PR fusionados (30 d)
- 154
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de dfa1/vortex-java
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
dfa1/vortex-java#738 ·
Los mantenedores suelen responder en 1 día
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
dfa1/vortex-java#737 ·
Los mantenedores suelen responder en 1 día
-
explore ADBC driver for vortexAbiertohelp wanted project
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
dfa1/vortex-java#729 ·
Los mantenedores suelen responder en 1 día
-
enhancement good first issue
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
dfa1/vortex-java#728 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
dfa1/vortex-java#723 ·
Los mantenedores suelen responder en 1 día
Todos los issues de dfa1/vortex-java
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
Dificultad 2/5 Menos de una hora Aptitud para principiantes 84/100
cpp-best-practices/project_validator#251 ·
Los mantenedores suelen responder en 1 día
-
backend enhancement
Dificultad 2/5 Medio día Aptitud para principiantes 70/100
damodharg6/payledger#16 ·
-
[Bug]: An initiative list with no current entry logs an error on every changePosiblemente ocupada @dwizard0 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día