apache/gravitino

[Improvement] clarify docker test failure messages

Ouverte

#4 170 ouverte le 16 juil. 2024

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (2 personnes assignées)Java (887 forks)auto 404
good first issueimprovement

Métriques du dépôt

Stars
 (3 058 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

What would you like to be improved?

Since the default value of skipDockerTests is true. When user runs the docker tests task without setting skipDockerTests=false, the test will be failed but the error message is confused:

  • What went wrong: Execution failed for task ':integration-test:test'.

No tests found for given includes: [org.apache.gravitino.integration.test.client.AuditIT](--tests filter)

note: use ./gradlew :integration-test:test --tests "org.apache.gravitino.integration.test.client.AuditIT" to reproduce

How should we improve?

We should add configuration modification instructions in the error message to help users locate the cause of the error more easily.

Guide contributeur

Direction de recherche
Examinez la tâche Gradle pour les tests Docker et modifiez le message d'erreur pour inclure une note sur la définition de skipDockerTests=false.
Stack technique
java
Domaine
testing
Type d'issue
Documentation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GradleDocker
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90