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

Do not test for link in Bulkhead property #487

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

Conversation

datho7561
Copy link
Contributor

The link is different on every computer. There is no trivial way to only check for the link's presence. With the current test setup, jdt-ls will not resolve the link, since we aren't enabling the extended capability properly.

Fixes a part of #450

The link is different on every computer.
There is no trivial way to only check for the link's presence.
With the current test setup, jdt-ls will not resolve the link,
since we aren't enabling the extended capability properly.

Fixes a part of eclipse-lsp4mp#450

Signed-off-by: David Thompson <davthomp@redhat.com>
@datho7561
Copy link
Contributor Author

Expecting ~3 tests failing since they are flakey (usually related to validation)

@datho7561
Copy link
Contributor Author

datho7561 commented Feb 24, 2025

I think I figured out how to fix the rest of the flaky tests... just need to test to make sure it's actually working...
nope
maybe?
nope, I'm pretty certain now that it's not the property collection that's causing the flakey tests.

After more testing, it seems that sometimes the HEALTH_CHECK annotation is not located using javaProject.findType some times, I don't know if this is related to indexing not being finished or something related to building the Java Model.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants