search

apache / gravitino

World's most powerful open data catalog for building a high-performance, geo-distributed and federated metadata lake.
https://gravitino.apache.org
Apache License 2.0
915 stars 295 forks source link

[Improvement] clarify docker test failure messages #4170

Open mchades opened 1 month ago

mchades commented 1 month ago

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.

four1er commented 1 month ago

Assign it to me, thx!

justinmclean commented 1 month ago

@four1er do you still want to work on this?

Naresh-kumar-Thodupunoori commented 4 weeks ago

Are you still working on it @four1er ? If not I would like to work on it @justinmclean

mchades commented 3 weeks ago

Thank you for your enthusiasm! I think anyone can submit a PR at any time as long as the issue has not been resolved and there are no related PRs submitted. @Naresh-kumar-Thodupunoori