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

TOMEE-4165 JSON-P TCK setup #1003

Merged
merged 11 commits into from
Jan 19, 2024
Merged

TOMEE-4165 JSON-P TCK setup #1003

merged 11 commits into from
Jan 19, 2024

Conversation

jeanouii
Copy link
Contributor

No description provided.

@jeanouii
Copy link
Contributor Author

Closing this one because it's merged into #1005

@jeanouii jeanouii closed this Jan 19, 2023
@jeanouii jeanouii reopened this Feb 10, 2023
@rzo1
Copy link
Contributor

rzo1 commented Nov 8, 2023

@jeanouii Guess it is time to rebase? :)

@rzo1
Copy link
Contributor

rzo1 commented Nov 21, 2023

The standalone TCK looks good, but the signature tests are faile with our jakartaee-api uber jar:

Some signatures failed.
	Failed packages listed below: 
		jakarta.json(static mode)
		jakarta.json(reflection mode)
	Passed packages listed below: 
		jakarta.json.spi(static mode)
		jakarta.json.spi(reflection mode)
		jakarta.json.stream(static mode)
		jakarta.json.stream(reflection mode)

Edit: These signature tests fail because our uber-jar contains jakarta.json.bind.* classes (confirmed).

@rzo1
Copy link
Contributor

rzo1 commented Nov 21, 2023

Should be ok now (everything passes). Please check @jeanouii

@rzo1 rzo1 merged commit d6ec33f into main Jan 19, 2024
1 check passed
@rzo1 rzo1 deleted the TOMEE-4165_JSONP-TCK branch January 19, 2024 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants