Do extra work to integrate Weld 5

[arjantijms]

Weld 5 has changed things so that extra work is needed to integrate it.

See http://weld.cdi-spec.org/news/2022/01/24/weld-500Beta1/ and

https://www.eclipse.org/lists/cdi-dev/msg00573.html

[gurunrao]

CDI TCK with GF7 has 102 failures CI run link .
BV TCK with GF 7 has 165 failures CI run link.

[arjantijms]

@gurunrao

The latest numbers with updated GF and updated CDI API are:

Tests run: 1863, Failures: 54, Errors: 0, Skipped: 32

Additionally, the existing TCK porting kit is very large. It's possible to run the CDI TCK with much less code. See https://github.com/eclipse-ee4j/glassfish/tree/master/appserver/tests/tck/cdi

[dmatej]

Example:
mvn clean install -Pstaging,tck -pl :glassfish-external-tck-cdi -Dtest=**/full/extensions/lifecycle/**
This passed.

Or another:
mvn clean install -Pstaging,tck -pl :glassfish-external-tck-cdi -Dtest=**/full/extensions/**
This has one failure, but what is the cause and how to fix it?

[ERROR]   BuildCompatibleExtensionSmokeTest>Arquillian.run:138->testExtensionsCanCoexist:41 expected [0] but found [5]
[INFO]
[ERROR] Tests run: 281, Failures: 1, Errors: 0, Skipped: 0

Seems extensions now cannot coexist. Is it a bug in GF or in Weld?

[dmatej]

Can this be already closed, or everything is done now?

[arjantijms]

I think it’s all done now done ✅

…

On Monday, August 1, 2022, David Matějček ***@***.***> wrote: Can this be already closed, or everything is done now? — Reply to this email directly, view it on GitHub <#23872 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAXFOTQNPXSR4N5I26VZY5TVW6Q3XANCNFSM5R6AASXA> . You are receiving this because you authored the thread.Message ID: ***@***.***>