Re: [jakartaee-tck-dev] [glassfish-dev] A test added in CDI TCK 4.0.5 is

[Ondro Mihályi <mihalyi@xxxxxxxxxxx>]

Hi CDI team,

I've modified the GlassFish CDI Lang Model TCK runner and got a bit further with the LangModelTckTest test. The LangModelVerifier extension now succeeds but the deployment fails further because of

IllegalAccessException: class org.jboss.weld.resources.DefaultReflectionCache$AnnotationClassImpl cannot access a member of interface org.jboss.cdi.lang.model.tck.AnnRepeatableContainer with modifiers "public abstract"

See https://github.com/eclipse-ee4j/jakartaee-tck/pull/1094 for more details. Any idea why is that? The generated class AnnRepeatableContainer is package private, why is that? Why not public?

All the best,,

Ondro Mihalyi

Director, Jakarta EE expert

OmniFish - Jakarta EE Consulting & Support | www.omnifish.ee

Omnifish OÜ, Narva mnt 5, 10117 Tallinn, Estonia | VAT: EE102487932

On Wed, Jul 6, 2022 at 1:57 PM Scott Marlow <smarlow@xxxxxxxxxx> wrote:

Thanks Ondro for letting us know!

There may be a delay when subscribing to cdi-dev mailing list.  Hopefully you will be accepted soon!  

Scott

On Wed, Jul 6, 2022, 4:53 AM Ondro Mihályi <mihalyi@xxxxxxxxxxx> wrote:

The issue is resolved now. We found out that the class with the defect is actually not part of the CDI TCK, it's just in an example runner. So we can safely modify it and fix it for GlassFish. GlassFish still doesn't pass the test but at least deploys the test deployment. We're investigating how to fix it in GlassFish.

I've closed the issue on the cdi-tck repo.

Ondro

P.S. I cannot post to the cdi-dev mailing list as I'm not subscribed. I tried to subscribe in my Eclipse account but it didn't work. Scott or anybody else subscribed to the cdi-dev ML, please reply with a short message to get my email there and conclude the email thread.

On Tue, Jul 5, 2022 at 5:48 PM Scott Marlow <smarlow@xxxxxxxxxx> wrote:

On 7/5/22 11:25 AM, Scott Marlow wrote:

Please create a CDI issue for this.  Thank you!

Never mind, I now see the below issue (as everyone else does :-)

On 7/5/22 9:04 AM, Ondro Mihályi wrote:

Hi,

We've found out that the new test LangModelTckTest added in CDI TCK 4.0.5 doesn't pass with GlassFish and it seems that it's because of the test and not because of GlassFish. I've raise an issue in the cdi-tck repo to fix it: https://github.com/jakartaee/cdi-tck/issues/392

Can somebody from the CDI team have a look? This is now blocking Jakarta EE 10 release because this is AFAIK the last test where GlassFish fails against the Full Profile TCK.

All the best,,

Ondro Mihalyi

Director, Jakarta EE expert

OmniFish - Jakarta EE Consulting & Support | www.omnifish.ee

Omnifish OÜ, Narva mnt 5, 10117 Tallinn, Estonia | VAT: EE102487932

_______________________________________________
glassfish-dev mailing list
glassfish-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/glassfish-dev