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.

Sign up for GitHub

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

Jump to bottom

how to debug tests #12224

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

how to debug tests #12224

wants to merge 1 commit into from

Conversation

zeitlinger
Copy link
Member

No description provided.

@zeitlinger zeitlinger self-assigned this Sep 11, 2024
@zeitlinger zeitlinger requested a review from a team September 11, 2024 16:27
breedx-splk
breedx-splk approved these changes Sep 11, 2024
View reviewed changes
Copy link
Contributor

@breedx-splk breedx-splk left a comment

Choose a reason for hiding this comment

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

Thanks, it's a helpful tip!

@laurit
Copy link
Contributor

laurit commented Sep 11, 2024

I'm not sure about the value of this. As far as I know Gregor has something broken in his setup that prevents him from debugging by just hitting the debug button in Intellij. If we are going to add it I think we should at least make clear that this isn't the preferred way for debugging.

@zeitlinger
Copy link
Member Author

I'm not sure about the value of this. As far as I know Gregor has something broken in his setup that prevents him from debugging by just hitting the debug button in Intellij. If we are going to add it I think we should at least make clear that this isn't the preferred way for debugging.

agree - that's why I added "If you have trouble debugging tests in IntelliJ IDEA"
even better would be a way to fix my setup - I've tried very long...

SylvainJuge
SylvainJuge reviewed Sep 12, 2024
View reviewed changes
docs/contributing/running-tests.md
If you have trouble debugging tests in IntelliJ IDEA (because the classpath is not correct),
you can use remote debugging:

1. Run the tests with the `--debug-jvm` flag.
Copy link
Contributor

Choose a reason for hiding this comment

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

[minor] maybe link to gradle documentation about this.

Copy link
Contributor

Choose a reason for hiding this comment

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

is this https://docs.gradle.org/current/userguide/java_testing.html#sec:debugging_java_tests?

@heyams
Copy link
Contributor

heyams commented Sep 12, 2024

I'm not sure about the value of this. As far as I know Gregor has something broken in his setup that prevents him from debugging by just hitting the debug button in Intellij. If we are going to add it I think we should at least make clear that this isn't the preferred way for debugging.

agree - that's why I added "If you have trouble debugging tests in IntelliJ IDEA" even better would be a way to fix my setup - I've tried very long...

have you tried reinstalling or upgrade to the latest?

@laurit
Copy link
Contributor

laurit commented Sep 17, 2024

agree - that's why I added "If you have trouble debugging tests in IntelliJ IDEA"
even better would be a way to fix my setup - I've tried very long...

@zeitlinger could you describe what fails when you try to debug?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@heyams heyams heyams left review comments

@SylvainJuge SylvainJuge SylvainJuge left review comments

@breedx-splk breedx-splk breedx-splk approved these changes

@theletterf theletterf Awaiting requested review from theletterf

Assignees

@zeitlinger zeitlinger

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@zeitlinger @laurit @heyams @SylvainJuge @breedx-splk