Skip to content

Conversation

@tommaso-borgato
Copy link
Contributor

*** back-port of #4651 to branch 6.2 ***

In other CI environments other than Github, we need to be able to exclude additional tests without overriding existing excludes;

@tommaso-borgato tommaso-borgato requested a review from a team as a code owner September 4, 2025 13:01
@tommaso-borgato tommaso-borgato changed the base branch from main to 6.2 September 4, 2025 13:02
@tommaso-borgato tommaso-borgato changed the title 6.2 additional.surefire.excludes [6.2] Exclude additional tests in resteasy-integration-tests Sep 4, 2025
@tommaso-borgato
Copy link
Contributor Author

/test

@jamezp
Copy link
Member

jamezp commented Sep 4, 2025

What is the goal here? I assume to allow tests to be skipped, but I'd like to better understand why. If it's for tests we regularly want skipped, we should tag them and allow them to excluded that way.

As far as CI not running, I'll have to look at that. I looks like every change to a pom.xml should kick off the workflow.

@tommaso-borgato
Copy link
Contributor Author

tommaso-borgato commented Sep 4, 2025

We are skipping JettyClientHttpEngineTest and VertxClientHttpEngineTest in our internal CI ... we need a way to do that without overriding excludes already in the TS

@tommaso-borgato tommaso-borgato force-pushed the 6.2_additional.surefire.excludes branch from 1aa7d8c to 94002b6 Compare September 8, 2025 08:00
@tommaso-borgato
Copy link
Contributor Author

@jamezp I see there are errors in resteasy-vertx ... they seem unrelated to this PR

@jamezp jamezp merged commit 351c5f1 into resteasy:6.2 Sep 17, 2025
11 of 13 checks passed
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