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

[341] Upgrade Wiremock, which also upgrades Jetty. Also upgrade TestN… #370

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

jamezp
Copy link
Contributor

@jamezp jamezp commented Jan 23, 2024

…G. All these upgrades required some additional changes.

resolves #341
resolves #346

Locally I ran this against resteasy-microprofile. While it required some dependencies changes there too, this passed the TCK there.

[INFO] Results:
[INFO] 
[WARNING] Tests run: 226, Failures: 0, Errors: 0, Skipped: 13
[INFO] 
[INFO] 
[INFO] --- jar:3.1.2:jar (default-jar) @ microprofile-rest-client-tck ---
[INFO] Building jar: /home/jperkins/projects/resteasy/resteasy-microprofile/testsuite/microprofile-rest-client-tck/target/microprofile-rest-client-tck-2.1.6.Final-SNAPSHOT.jar
[INFO] 
[INFO] --- source:3.1.0:jar-no-fork (attach-sources) @ microprofile-rest-client-tck ---
[INFO] No sources in project. Archive not created.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  05:00 min
[INFO] Finished at: 2024-01-22T16:19:03-08:00
[INFO] ------------------------------------------------------------------------

…G. All these upgrades required some additional changes.

Signed-off-by: James R. Perkins <[email protected]>
Copy link
Member

@Emily-Jiang Emily-Jiang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Emily-Jiang Emily-Jiang merged commit 40afc49 into microprofile:main Feb 5, 2024
8 checks passed
@jamezp jamezp deleted the issue341 branch February 6, 2024 17:57
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.

Replace javax namespace usage with jakarta The 3.0 TCK Relies on old Jakarta/Java EE Servlet Specification
2 participants