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

CXF-7996 Jakarta EE TCKs and compatibility #783

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

dblevins
Copy link
Contributor

Tests are still being aggressively fixed, so this PR won't likely go "quiet" till Sunday or so. However, if there's appetite to merge it to master or a new branch where more of us can collaborate, that's totally great too.

Opening this PR now so if that's what we want to do, we can just click a button vs sending an email to say "go create that PR so I can click a button" :) No expectation it will actually be merged at this point in time.

I think even though I opened this for master that the branch can be changed. If not and we want to use a different branch -- maybe a new one for TCK-work-in-progress -- let me know and I'll update the PR.

reta and others added 16 commits April 25, 2021 22:31
…ee.rs.get.JAXRSClient#optionSubTest

 CXF-8486   jaxrs.ee.rs.pathparam.locator pathParamEntityWithConstructorTest                                        Open
 CXF-8487   jaxrs.ee.rs.pathparam.locator pathParamEntityWithFromStringTest                                         Open
 CXF-8488   jaxrs.ee.rs.pathparam.locator pathParamEntityWithValueOfTest                                            Open
 CXF-8489   jaxrs.ee.rs.pathparam.locator pathParamListEntityWithFromStringTest                                     Open
 CXF-8490   jaxrs.ee.rs.pathparam.locator pathParamSetEntityWithFromStringTest                                      Open
 CXF-8491   jaxrs.ee.rs.pathparam.locator pathParamThrowingIllegalArgumentExceptionTest                             Open
 CXF-8492   jaxrs.ee.rs.pathparam.locator pathParamThrowingWebApplicationExceptionTest                              Open
 CXF-8493   jaxrs.ee.rs.pathparam.locator test1                                                                     Open
 CXF-8494   jaxrs.ee.rs.pathparam.locator test2                                                                     Open
 CXF-8495   jaxrs.ee.rs.pathparam.locator test3                                                                     Open
 CXF-8496   jaxrs.ee.rs.pathparam.locator test4                                                                     Open
 CXF-8497   jaxrs.ee.rs.pathparam.locator test5                                                                     Open
Also fixes test which will intermittently pass (dumb luck):

 - com.sun.ts.tests.jaxrs.jaxrs21.ee.priority.JAXRSClient	paramConverterPriorityTest_from_standalone
CXF-8515   jaxrs.spec.context.client clientWriterTest
Maybe not the most elegant fix, but does work
@reta
Copy link
Member

reta commented Jun 9, 2022

@dblevins do you mind if I close this one? TCK tests are all passing now, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants