apache/gravitino

[Improvement] clarify docker test failure messages

Aperta

#4170 aperta il 16 lug 2024

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (2 assegnatari)Java (887 fork)auto 404
good first issueimprovement

Metriche repository

Star
 (3058 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina il task Gradle per i test Docker e modifica il messaggio di errore per includere una nota sull'impostazione di skipDockerTests=false.
Tech stack
java
Dominio
testing
Tipo issue
Documentazione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GradleDocker
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90