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

Many tests fail on JDK 11 #4904

Closed
mbien opened this issue Nov 2, 2022 · 15 comments
Closed

Many tests fail on JDK 11 #4904

mbien opened this issue Nov 2, 2022 · 15 comments
Labels
Contribution welcome An issue or feature not currently being worked on, but a contribution would be welcomed! kind:bug Bug report or fix priority:high High priority issue that should, if possible, be fixed in next release tests

Comments

@mbien
Copy link
Member

mbien commented Nov 2, 2022

There are still many tests which can't be run on JDK 11.

NB is currently leaning into the compatibility promise of OpenJDK - it would be good to repair the tests so that the project can say with confidence that things are actually working as intended on JDK 11+.

#4884 is a dummy PR which bumps everything to 11 to generate a current list of failed jobs

feel free to re trigger the workflow as needed when updates happen

remaining list of failing modules:

enterprise/websvc.editor.hints
enterprise/j2ee.dd.webservice

fixed in #5138:

java/classfile
java/whitelist

fixed in #5143:

webcommon/javascript2.jsdoc

fixed in #5144:

ide/db.dataview
ide/schema2beans
ide/xml.wsdl.model

fixed in #5658:

php/php.editor

fixed in #6194:

java/java.openjdk.project 

fixed in #6369:

java/debugger.jpda.truffle

fixed in #6617

platform/openide.filesystems
platform/options.api
platform/openide.awt
platform/core.multiview
platform/favorites

fixed in #7201

apisupport/apisupport.ant

fixed in #7415

groovy/groovy.editor

fixed in #7502

platform/lib.uihandler
platform/netbinox
platform/openide.text
platform/openide.util.ui
@mbien mbien added kind:bug Bug report or fix priority:high High priority issue that should, if possible, be fixed in next release tests labels Nov 2, 2022
@mbien mbien added this to the NB17 milestone Nov 2, 2022
@mbien mbien added the kind:meta High-level information important to the community label Nov 2, 2022
@mbien mbien changed the title Many tests fail in JDK 11 Many tests fail on JDK 11 Nov 2, 2022
@mbien mbien added the Contribution welcome An issue or feature not currently being worked on, but a contribution would be welcomed! label Nov 2, 2022
@ebarboni
Copy link
Contributor

ebarboni commented Nov 2, 2022

@mbien jenkins linux jenkins build on jdk 11 shows junit failure too at the same place (some are from long time ago)
for example,
org.netbeans.core.multiview.MultiViewProcessorTest.testFailsWithoutAnIcon

I'm not sure how to make the GA nice to review, very complicated to get the failure place.

@mbien
Copy link
Member Author

mbien commented Nov 2, 2022

the summary page here (scroll down) also shows a list of test failures (it currently can't show errors, only failures):
https://github.com/apache/netbeans/actions/runs/3761176522

I believe one way to approach this would be to fix the easiest candidates first, to make the situation better observable and reduce the noise.

@mbien mbien removed the kind:meta High-level information important to the community label Nov 2, 2022
@mbien
Copy link
Member Author

mbien commented Dec 23, 2022

java/classfile
java/whitelist

are fixed

@mbien
Copy link
Member Author

mbien commented Dec 23, 2022

webcommon/javascript2.jsdoc and the IDE modules most likely too

@ebresie
Copy link
Contributor

ebresie commented Jan 14, 2023

Silly question, what needs to be called or done to trigger the tests failures locally?

@lkishalmi
Copy link
Contributor

I usually just copy the command from the failed job, and execute that locally. Not necessary all failures are reproducible locally, but it worth to give it a try.

@KacerCZ
Copy link
Contributor

KacerCZ commented Jan 15, 2023

Failing test in php/php.editor is o.n.m.php.editor.csl.FoldingTest in method testPHPTags().

This test should fail also under Linux, because actual result does not match expected result.
Expected result of test matches actual code folding when I open test file in NetBeans.

What I don't understand is, that running only method testPHPTags() fails, but running all tests in FoldingTest class is OK.

@mbien
Copy link
Member Author

mbien commented Jan 15, 2023

@KacerCZ i saw the linux PHP job green a few times in previous test runs on JDK 11:
https://github.com/apache/netbeans/actions/runs/3761176522
https://github.com/apache/netbeans/actions/runs/3623501729

I haven't investigated any further, I only wrote down what I saw back then.

@mbien
Copy link
Member Author

mbien commented Mar 13, 2023

php/php.editor

fixed

@mbien
Copy link
Member Author

mbien commented Sep 20, 2023

java/debugger.jpda.truffle

should be fixed soon. PR is pending. This will move the GraalVM job to JDK 11 baseline and unlock another blocked PR.

@neilcsmith-net neilcsmith-net removed this from the NB20 milestone Oct 14, 2023
This was referenced Oct 20, 2023
@mbien
Copy link
Member Author

mbien commented Oct 25, 2023

PR pending for:

    openide.filesystems
    options.api
    openide.awt
    core.multiview
    platform/favorites

@mbien
Copy link
Member Author

mbien commented May 27, 2024

apisupport/apisupport.ant

fixed in #7201

@mbien
Copy link
Member Author

mbien commented May 28, 2024

PR pending for

groovy/groovy.editor

@mbien
Copy link
Member Author

mbien commented Jun 24, 2024

#7502 pending for

platform/lib.uihandler
platform/netbinox
platform/openide.text
platform/openide.util.ui

@mbien
Copy link
Member Author

mbien commented Oct 11, 2024

closing since CI is now testing on JDK 17+ exclusively. For tests which are disabled via project properties or other means it can be implied that they are probably not working properly - but this doesn't need this issue to track I think.

tests which are currently locked to JDK 17 are tracked via #7871 and TODOs in main.yml

good job everyone who helped with this!

@mbien mbien closed this as completed Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contribution welcome An issue or feature not currently being worked on, but a contribution would be welcomed! kind:bug Bug report or fix priority:high High priority issue that should, if possible, be fixed in next release tests
Projects
None yet
Development

No branches or pull requests

6 participants