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.

Sign up for GitHub

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

Jump to bottom

Get the TCK tests to pass #432

Merged

sdelamo merged 3 commits into master from tck

Apr 27, 2023

Contributor

timyates commented Apr 21, 2023

Since we've updated to latest versions, we should get more TCK tests passing.


          Get the TCK tests to pass

e1a6505

Since we've updated to latest versions, we should get more TCK tests passing.

timyates self-assigned this

timyates added 2 commits

April 21, 2023 17:39


          Move to unique paths in a vain attempt to fix graal

275aeab


          Exclude TCK tests that fail with Graal

adc1777

Contributor Author

timyates commented Apr 21, 2023

I have had to exclude the two TCK tests that fail.

Failing TCK Class	Cause
`StaticResourceTest`	The `assets` resource (from the server-tck in core) is not getting used by Graal despite it being in resource-config.json
`ClientResponseFilterTest`	No idea. `responseFilterThrowableParameter` just seems to fail under native (but works otherwise)

sonarqubecloud bot commented Apr 21, 2023

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

sdelamo marked this pull request as ready for review

April 27, 2023 13:35

sdelamo approved these changes

View reviewed changes

sdelamo merged commit fd4ca9f into master

sdelamo deleted the tck branch

April 27, 2023 13:37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet