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

Two ClasspathResolutionTest failures since I20240317-1800 #1205

Closed
iloveeclipse opened this issue Mar 18, 2024 · 4 comments · Fixed by #1207
Closed

Two ClasspathResolutionTest failures since I20240317-1800 #1205

iloveeclipse opened this issue Mar 18, 2024 · 4 comments · Fixed by #1207
Labels
bug Something isn't working regression Regression defect

Comments

@iloveeclipse
Copy link
Member

We have two new ClasspathResolutionTest failures since https://download.eclipse.org/eclipse/downloads/drops4/I20240317-1800/

See for example https://download.eclipse.org/eclipse/downloads/drops4/I20240317-1800/testresults/html/org.eclipse.pde.ui.tests_ep432I-unit-cen64-gtk3-java17_linux.gtk.x86_64_17.html

Most likely regression from ab28042 that added new dependencies.

org.eclipse.pde.core.tests.internal.classpath.ClasspathResolutionTest	testImportSystemPackageDoesntAddExtraBundleJava8	Failure	Expecting empty but was: ["org.osgi.annotation.versioning_1.1.2.202109301733.jar", "org.osgi.annotation.bundle_2.0.0.202202082230.jar", "org.osgi.service.component.annotations_1.5.1.202212101352.jar", "org.osgi.service.metatype.annotations_1.4.1.202109301733.jar"]

java.lang.AssertionError:
Expecting empty but was: ["org.osgi.annotation.versioning_1.1.2.202109301733.jar",
"org.osgi.annotation.bundle_2.0.0.202202082230.jar",
"org.osgi.service.component.annotations_1.5.1.202212101352.jar",
"org.osgi.service.metatype.annotations_1.4.1.202109301733.jar"]
at org.eclipse.pde.core.tests.internal.classpath.ClasspathResolutionTest.testImportSystemPackageDoesntAddExtraBundleJava8(ClasspathResolutionTest.java:132)
at java.base/java.lang.reflect.Method.invoke(Method.java:568)
at org.eclipse.pde.ui.tests.runtime.TestUtils$1.evaluate(TestUtils.java:276)
at org.eclipse.pde.ui.tests.runtime.TestUtils$1.evaluate(TestUtils.java:276)
at org.eclipse.pde.ui.tests.runtime.TestUtils$1.evaluate(TestUtils.java:276)
org.eclipse.pde.core.tests.internal.classpath.ClasspathResolutionTest	testImportSystemPackageDoesntAddExtraBundleJava8_osgiEERequirement	Failure	Expecting empty but was: ["org.osgi.annotation.versioning_1.1.2.202109301733.jar", "org.osgi.annotation.bundle_2.0.0.202202082230.jar", "org.osgi.service.component.annotations_1.5.1.202212101352.jar", "org.osgi.service.metatype.annotations_1.4.1.202109301733.jar"]

java.lang.AssertionError:
Expecting empty but was: ["org.osgi.annotation.versioning_1.1.2.202109301733.jar",
"org.osgi.annotation.bundle_2.0.0.202202082230.jar",
"org.osgi.service.component.annotations_1.5.1.202212101352.jar",
"org.osgi.service.metatype.annotations_1.4.1.202109301733.jar"]
at org.eclipse.pde.core.tests.internal.classpath.ClasspathResolutionTest.testImportSystemPackageDoesntAddExtraBundleJava8_osgiEERequirement(ClasspathResolutionTest.java:146)
at java.base/java.lang.reflect.Method.invoke(Method.java:568)
at org.eclipse.pde.ui.tests.runtime.TestUtils$1.evaluate(TestUtils.java:276)
at org.eclipse.pde.ui.tests.runtime.TestUtils$1.evaluate(TestUtils.java:276)
at org.eclipse.pde.ui.tests.runtime.TestUtils$1.evaluate(TestUtils.java:276)
@iloveeclipse iloveeclipse added bug Something isn't working regression Regression defect labels Mar 18, 2024
@laeubi
Copy link
Contributor

laeubi commented Mar 18, 2024

Any reason these tests are not executed as part of the verification?

@iloveeclipse
Copy link
Member Author

Any reason these tests are not executed as part of the verification?

Not really, just matter of resources.
Many PDE tests expect a "special" setup which was probably not easy to replicate in maven/tycho, so that was original issue with them. There was a task to enable all PDE tests for jenkins, but the problem was that nobody stepped in :-(

@laeubi
Copy link
Contributor

laeubi commented Mar 19, 2024

I now checked this and the test actually is run but succeeds, this is likely because it is not running in a full IDE, I'll try to adjust the test setup.

@laeubi
Copy link
Contributor

laeubi commented Mar 19, 2024

Fixed the test setup to show the problem here:

this also includes a fix for the assertions now so they don't assert empty (what was just a side effect) but absence of the item.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working regression Regression defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants