search

eclipse-ee4j / mojarra

Mojarra, a Jakarta Faces implementation
Other
159 stars 109 forks source link

Version 4.0.1 breaks the TCK Signature tests #5215

Closed dmatej closed 1 year ago

dmatej commented 1 year ago

Describe the bug

Unfortunately despite I completely support the change in d7a916bd38fc64c49bc98551d6e0c4945cfd1fab, it fails in tests (with current GlassFish 7.0.3-SNAPSHOT).

[runcts] OUT => [javatest.batch] Missing Superclasses or Superinterfaces
[runcts] OUT => [javatest.batch] ---------------------------------------
[runcts] OUT => [javatest.batch] 
[runcts] OUT => [javatest.batch] jakarta.faces.application.FacesMessage$Severity:            interface @ java.lang.Comparable
[runcts] OUT => [javatest.batch] 
[runcts] OUT => [javatest.batch] Missing Methods
[runcts] OUT => [javatest.batch] ---------------
[runcts] OUT => [javatest.batch] 
[runcts] OUT => [javatest.batch] jakarta.faces.application.FacesMessage$Severity:            method public int jakarta.faces.application.FacesMessage$Severity.compareTo(java.lang.Object)
[runcts] OUT => [javatest.batch] 
[runcts] OUT => [javatest.batch] Added Superclasses or Superinterfaces
[runcts] OUT => [javatest.batch] -------------------------------------
[runcts] OUT => [javatest.batch] 
[runcts] OUT => [javatest.batch] jakarta.faces.application.FacesMessage$Severity:            interface @ java.lang.Comparable<jakarta.faces.application.FacesMessage$Severity>
[runcts] OUT => [javatest.batch] 
[runcts] OUT => [javatest.batch] Added Methods
[runcts] OUT => [javatest.batch] -------------
[runcts] OUT => [javatest.batch] 
[runcts] OUT => [javatest.batch] jakarta.faces.application.FacesMessage$Severity:            method public int jakarta.faces.application.FacesMessage$Severity.compareTo(jakarta.faces.application.FacesMessage$Severity)

To Reproduce

In the GlassFish project run (You need docker installed):

mvn clean install -Ptck -pl :jakarta-ant-based-tck -Dtck.test.jee.file=jakarta-jakartaeetck-10.0.2.zip
mvn clean install -Ptck -pl :tck-runner -Dit.test=CustomITest -Dtck.module=signaturetest/javaee

Or take a look to the Jenkins (doesn't print details): https://ci.eclipse.org/jakartaee-tck/job/jakartaee-tck/job/10.0.x/49/testReport/junit/com.sun.ts.tests.signaturetest.javaee/JavaEESigTest/

Expected behavior

Passed test.

Screenshots

Left: jakartaee-api-10.0.0, right: jakarta.faces-4.0.1 image