Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add documentation for @WithTestResource to getting started testing #44685

Merged

Conversation

HerrDerb
Copy link
Contributor

@HerrDerb HerrDerb commented Nov 25, 2024

Small draft for the usage of @WithTestResource as we got confused by how it works and finally switched back to @QuarkusTestResource
I just wrote down of what I think I know of reading the code 🙂I hope I did not get it completely wrong.

Please feel free to alter and add information, especially for the stuff of BELOW TO EDIT:

This comment has been minimized.

Copy link

github-actions bot commented Nov 25, 2024

🙈 The PR is closed and the preview is expired.

@geoand
Copy link
Contributor

geoand commented Nov 25, 2024

Thanks! I'll have a look tomorrow

@geoand
Copy link
Contributor

geoand commented Nov 26, 2024

May I ask that the changes be limited to only what related to test resources?

It makes reviewing stuff much easier.

Thanks

@HerrDerb
Copy link
Contributor Author

@geoand Sorry, that must have been some auto formatting. I'll revert that

@geoand
Copy link
Contributor

geoand commented Nov 27, 2024

🙏🏽

@HerrDerb HerrDerb force-pushed the add-with-test-resource-documentation branch from a250342 to 30f019b Compare November 27, 2024 08:26
Copy link
Contributor

@geoand geoand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice, thanks!

@geoand geoand added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Nov 27, 2024
Copy link

quarkus-bot bot commented Nov 27, 2024

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 30f019b.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@geoand geoand merged commit 014bf4b into quarkusio:main Nov 27, 2024
5 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.18 - main milestone Nov 27, 2024
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Nov 27, 2024
@geoand geoand added triage/waiting-for-ci Ready to merge when CI successfully finishes triage/backport labels Nov 27, 2024
@HerrDerb
Copy link
Contributor Author

@geoand
Copy link
Contributor

geoand commented Nov 27, 2024

Oops, I missed that! I'll need to fix it

geoand added a commit to geoand/quarkus that referenced this pull request Nov 27, 2024
@geoand
Copy link
Contributor

geoand commented Nov 27, 2024

#44768

gsmet added a commit that referenced this pull request Nov 27, 2024
Address leftovers in testing docs from #44685
@gsmet gsmet modified the milestones: 3.18 - main, 3.17.1 Nov 27, 2024
gsmet pushed a commit to gsmet/quarkus that referenced this pull request Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation triage/waiting-for-ci Ready to merge when CI successfully finishes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants