Skip to content

Commit 8f33083

Browse files
authored
Merge pull request #48725 from holly-cummins/document-integration-tests
Document current behaviour for integration tests
2 parents 25e9472 + a115721 commit 8f33083

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

CONTRIBUTING.md

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -250,7 +250,7 @@ javadoc...).
250250
Be sure to test your pull request in:
251251

252252
1. Java mode
253-
2. Native mode
253+
2. Native mode (with `-Dnative`)
254254

255255
Also, make sure that any native tests you add will actually get executed on CI.
256256
In the interest of speeding up CI, the native build job `native-tests` have been split into multiple categories which
@@ -558,6 +558,17 @@ this is by executing the following command:
558558
./mvnw test -f integration-tests/resteasy-jackson/ -Dtest=GreetingResourceTest
559559
```
560560

561+
#### Running integration tests
562+
563+
By default, in the Quarkus codebase, all tests ending in *IT are skipped, even with the verify goal.
564+
This means most `@QuarkusIntegrationTest` tests in the Quarkus codebase only run when `-Dnative` is added to the command line.
565+
Even using the `verify` goal and adding `-DskipITs=false` is not enough to run these tests.
566+
Only `-Dnative` will trigger them.
567+
Note that this is a behaviour difference to what's [documented externally](https://quarkus.io/guides/getting-started-testing#quarkus-integration-test).
568+
569+
Why the different behaviour? Quarkus builds are _long_, and running every integration test twice would make them even longer.
570+
In most cases, coverage in the native build is sufficient.
571+
561572
##### Maven Invoker tests
562573

563574
For testing some scenarios, Quarkus uses the [Maven Invoker](https://maven.apache.org/shared/maven-invoker/) to run

0 commit comments

Comments
 (0)