Simplify AssertJ assertions flagged by Sonar (java:S5838)
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 78/100
Piste de recherche
Choisis un module et commente ici pour revendiquer ce périmètre. Dans ce module, examine les constats SonarCloud java:S5838 ; l'issue cite ComputeTest lignes 708 et 767 comme exemples dans reader. Remplace uniquement les assertions par des formes AssertJ plus précises, conserve la structure Given/When/Then et évite containsExactly pour les grands tableaux. Lance ./mvnw test -pl <module> -am ; c'est terminé lorsque les tests passent et que la modification reste limitée aux tests.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Java
- Étoiles
- 3
- Forks
- 2
- Merge moyen
- 9 h 19 min
- PR mergées (30 j)
- 154
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 dfa1/vortex-java
-
good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
dfa1/vortex-java#738 ·
Les mainteneurs répondent en général sous 1 jour
-
good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
dfa1/vortex-java#737 ·
Les mainteneurs répondent en général sous 1 jour
-
explore ADBC driver for vortexOuvertehelp wanted project
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
dfa1/vortex-java#729 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement good first issue
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
dfa1/vortex-java#728 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
dfa1/vortex-java#723 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de dfa1/vortex-java
Issues similaires
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
-
Difficulté 2/5 Moins d'une heure Accessibilité débutants 84/100
cpp-best-practices/project_validator#251 ·
Les mainteneurs répondent en général sous 1 jour
-
backend enhancement
Difficulté 2/5 Une demi-journée Accessibilité débutants 70/100
damodharg6/payledger#16 ·
-
[Bug]: An initiative list with no current entry logs an error on every changePeut-être pris @dwizard0 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour