-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
dblevins
wants to merge
16
commits into
apache:main
Choose a base branch
from
dblevins:tckfixes
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…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
…AndThrowsErrorTest
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
@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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.