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

Enable LeakTestSuite in org.eclipse.jdt.ui.tests pom.xml file again (#2) #23

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

Conversation

iloveeclipse
Copy link
Member

Fixes issue #2

@iloveeclipse
Copy link
Member Author

Some maven tweaking is needed, tests are still failing:

java.lang.RuntimeException: 
JVM settings for --add-modules, --add-opens, and --illegal-access are probably missing.
Exception : 
Unable to make field private static final java.util.Set jdk.nio.zipfs.ZipFileSystem.supportedFileAttributeViews accessible: module jdk.zipfs does not "opens jdk.nio.zipfs" to unnamed module @543315dc
	at org.eclipse.jdt.ui.tests.leaks.JavaLeakTest.testTextEditorCloseOneOfTwo(JavaLeakTest.java:99)
Caused by: java.lang.reflect.InaccessibleObjectException: Unable to make field private static final java.util.Set jdk.nio.zipfs.ZipFileSystem.supportedFileAttributeViews accessible: module jdk.zipfs does not "opens jdk.nio.zipfs" to unnamed module @543315dc
	at org.eclipse.jdt.ui.tests.leaks.JavaLeakTest.testTextEditorCloseOneOfTwo(JavaLeakTest.java:99)

@iloveeclipse
Copy link
Member Author

@ktatavarthi : I have no idea why the test in tycho / jenkins doesn't work now. Any idea?

@ktatavarthi
Copy link
Member

@ktatavarthi : I have no idea why the test in tycho / jenkins doesn't work now. Any idea?

I donot have any idea. I will try and check if, I can reproduce this problem locally.

@mickaelistria : Do you have any idea. In one of your previous comment , you said that you were able to run the LeakTestSuite locally using maven successfully.

@mickaelistria
Copy link
Contributor

Indeed, I can run those tests locally with a mvn clean verify from the module (removing the AutomatedTests line to speed up tests) without changing anything. I suspect this could be a bug in the used JVM? Can you reproduce it locally?

@ktatavarthi
Copy link
Member

Indeed, I can run those tests locally with a mvn clean verify from the module (removing the AutomatedTests line to speed up tests) without changing anything. I suspect this could be a bug in the used JVM? Can you reproduce it locally?

I also tried to run locally with mvn clean verify from the module (removing the AutomatedTests line to speed up tests) without changing anything. The tests fail for me locally with the below error
java.lang.NoClassDefFoundError: Lorg/apache/maven/surefire/spi/MasterProcessChannelProcessorFactory;

@mickaelistria
Copy link
Contributor

FWIW, I'm using latest Java release.

@jukzi
Copy link
Contributor

jukzi commented Jan 31, 2024

this PR duplicates #7 please close one of them and resolve the remaining

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.

None yet

4 participants