Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
d1150a2
chore: skip unit tests for pure GAPIC modules by default
lqiu96 Jul 7, 2026
b7eaa81
chore: format properties blocks in modified POMs
lqiu96 Jul 7, 2026
175e764
chore: fix indentation for java-bigtable properties block
lqiu96 Jul 7, 2026
89f6801
chore: Trigger all unit tests to run
lqiu96 Jul 8, 2026
e7809ec
chore: optimize kokoro test job to use mvn test
lqiu96 Jul 8, 2026
fe04b82
chore: upgrade flatten-maven-plugin to 1.6.0 and use mvn package for …
lqiu96 Jul 8, 2026
bddf723
chore: optimize global unit test execution in CI
lqiu96 Jul 8, 2026
aa39fe7
chore: use static core exclusions for global unit tests
lqiu96 Jul 8, 2026
7f43a38
build(config): map maven.test.skip to skipUnitTests in jar-parent
lqiu96 Jul 8, 2026
72ae747
docs(config): document maven.test.skip and skipUnitTests mapping
lqiu96 Jul 8, 2026
d46c880
ci(opt): skip client library tests in global run
lqiu96 Jul 8, 2026
be9437b
fix(spanner-jdbc): respect maven.test.skip in doc generation exec
lqiu96 Jul 8, 2026
25ae681
ci(opt): skip test compilation globally in global run
lqiu96 Jul 8, 2026
56103e1
ci(opt): simplify global test run by removing core exclusions and ins…
lqiu96 Jul 8, 2026
36ed633
ci(opt): remove redundant --also-make from targeted test runs
lqiu96 Jul 8, 2026
e6d1f47
ci(opt): remove MVN_ARGS entirely and add comments for skip properties
lqiu96 Jul 8, 2026
1411069
ci(opt): remove bulkTests and EXTRA_PROFILE_OPTS from unit test job
lqiu96 Jul 8, 2026
72dd906
ci(opt): use skipTests instead of maven.test.skip globally in build.sh
lqiu96 Jul 8, 2026
385bc34
chore: Reset changes not needed in the PR
lqiu96 Jul 8, 2026
d2f6f23
Merge branch 'main' of github.com:googleapis/google-cloud-java into s…
lqiu96 Jul 8, 2026
c85b942
Merge branch 'main' into skip-gapic-unit-tests
lqiu96 Jul 8, 2026
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 9 additions & 7 deletions .kokoro/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -43,21 +43,24 @@ case ${JOB_TYPE} in
install_modules "${BUILD_SUBDIR}"
echo "Running in subdir: ${BUILD_SUBDIR}"
pushd "${BUILD_SUBDIR}"
EXTRA_PROFILE_OPTS=()
else
EXTRA_PROFILE_OPTS=("-PbulkTests")
install_modules "sdk-platform-java"
# We do not need to install core modules or exclude them from the reactor.
# We pass -DskipTests=true to skip running tests globally.
# We pass -DskipUnitTests=true which maps to -Dmaven.test.skip=true for client libraries
# (via google-cloud-jar-parent), skipping their test compilation to save time.
# Core modules do not inherit this mapping, so their tests are compiled, which is
# necessary to resolve testlib dependencies (like gax-grpc testlib) in the reactor.
SUREFIRE_JVM_OPT="${SUREFIRE_JVM_OPT} -DskipUnitTests=true -DskipTests=true"
fi
echo "SUREFIRE_JVM_OPT: ${SUREFIRE_JVM_OPT}"
retry_with_backoff 3 10 \
mvn install \
mvn test \
-B -ntp \
-Pquick-build \
-Dorg.slf4j.simpleLogger.showDateTime=true \
-Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \
-Dmaven.wagon.http.retryHandler.count=5 \
--also-make \
${SUREFIRE_JVM_OPT} "${EXTRA_PROFILE_OPTS[@]}"
${SUREFIRE_JVM_OPT}
RETURN_CODE=$?

if [[ -n "${BUILD_SUBDIR}" ]]
Expand Down Expand Up @@ -316,7 +319,6 @@ case ${JOB_TYPE} in
install_modules "${BUILD_SUBDIR}"
echo "Running in subdir: ${BUILD_SUBDIR}"
pushd "${BUILD_SUBDIR}"
EXTRA_PROFILE_OPTS=()
else
install_modules "sdk-platform-java"
fi
Expand Down
10 changes: 9 additions & 1 deletion google-cloud-jar-parent/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,15 @@
<relativePath>../google-cloud-pom-parent/pom.xml</relativePath>
</parent>
<properties>
<skipUnitTests>false</skipUnitTests>
<!-- skipUnitTests controls unit test execution (surefire) -->
<skipUnitTests>true</skipUnitTests>
<!--
We map maven.test.skip to skipUnitTests by default to skip both compilation
and execution of tests for libraries that skip unit tests (like GAPICs).
For integration test jobs that need compilation but skip unit tests,
pass -Dmaven.test.skip=false -DskipUnitTests=true.
-->
<maven.test.skip>${skipUnitTests}</maven.test.skip>
<ignoreNonCompile>true</ignoreNonCompile><!-- maven-dependency-plugin:analyze to skip test scope dependencies -->
</properties>

Expand Down
1 change: 1 addition & 0 deletions java-bigquery-jdbc/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@
<github.global.server>github</github.global.server>
<site.installationModule>google-cloud-bigquery-jdbc</site.installationModule>
<skipShade>false</skipShade>
<skipUnitTests>false</skipUnitTests>
</properties>

<build>
Expand Down
1 change: 1 addition & 0 deletions java-bigquery/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@
<github.global.server>github</github.global.server>
<site.installationModule>google-cloud-bigquery-parent</site.installationModule>
<google-api-services-bigquery.version>v2-rev20251012-2.0.0</google-api-services-bigquery.version>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
1 change: 1 addition & 0 deletions java-bigquerystorage/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@
<project.protobuf-java.version>3.25.4</project.protobuf-java.version>
<github.global.server>github</github.global.server>
<site.installationModule>google-cloud-bigquerystorage-parent</site.installationModule>
<skipUnitTests>false</skipUnitTests>
</properties>

<pluginRepositories>
Expand Down
1 change: 1 addition & 0 deletions java-bigtable/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -146,6 +146,7 @@
<project.javadoc.protobufBaseURL>https://googleapis.dev/java/google-api-grpc/latest</project.javadoc.protobufBaseURL>
<errorprone.version>2.48.0</errorprone.version>
<errorprone.javac.args>-g</errorprone.javac.args>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
1 change: 1 addition & 0 deletions java-common-protos/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
<github.global.server>github</github.global.server>
<site.installationModule>google-iam-parent</site.installationModule>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
1 change: 1 addition & 0 deletions java-datastore/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -146,6 +146,7 @@
<project.javadoc.protobufBaseURL>https://googleapis.dev/java/google-api-grpc/latest</project.javadoc.protobufBaseURL>
<errorprone.version>2.48.0</errorprone.version>
<errorprone.javac.args>-g</errorprone.javac.args>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
1 change: 1 addition & 0 deletions java-dns/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
<github.global.server>github</github.global.server>
<site.installationModule>google-cloud-dns-parent</site.installationModule>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
3 changes: 3 additions & 0 deletions java-firestore/google-cloud-firestore-admin/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@
<artifactId>google-cloud-firestore-parent</artifactId>
<version>3.45.0-SNAPSHOT</version><!-- {x-version-update:google-cloud-firestore:current} -->
</parent>
<properties>
<skipUnitTests>true</skipUnitTests>
</properties>
<dependencies>
<dependency>
<groupId>com.google.api.grpc</groupId>
Expand Down
1 change: 1 addition & 0 deletions java-firestore/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,7 @@
<github.global.server>github</github.global.server>
<site.installationModule>google-cloud-firestore-parent</site.installationModule>
<project.javadoc.protobufBaseURL>https://googleapis.dev/java/google-api-grpc/latest</project.javadoc.protobufBaseURL>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
1 change: 1 addition & 0 deletions java-iam/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
<github.global.server>github</github.global.server>
<site.installationModule>google-iam-parent</site.installationModule>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
1 change: 1 addition & 0 deletions java-logging-logback/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
<logging.version>3.25.0</logging.version>
<slf4j.version>1.7.36</slf4j.version>
<google.api-common.version>1.10.1</google.api-common.version>
<skipUnitTests>false</skipUnitTests>
</properties>

<parent>
Expand Down
1 change: 1 addition & 0 deletions java-logging/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
<github.global.server>github</github.global.server>
<site.installationModule>google-cloud-logging-parent</site.installationModule>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
1 change: 1 addition & 0 deletions java-notification/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
</parent>
<properties>
<site.installationModule>google-cloud-notification</site.installationModule>
<skipUnitTests>false</skipUnitTests>
</properties>
<dependencyManagement>
<dependencies>
Expand Down
1 change: 1 addition & 0 deletions java-pubsub/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
<github.global.server>github</github.global.server>
<site.installationModule>google-cloud-pubsub-parent</site.installationModule>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
1 change: 1 addition & 0 deletions java-showcase/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@
<maven.deploy.skip>true</maven.deploy.skip>
<skipNexusStagingDeployMojo>true</skipNexusStagingDeployMojo>
<mdep.analyze.skip>true</mdep.analyze.skip><!-- The showcase modules are not meant for library customers. -->
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
3 changes: 2 additions & 1 deletion java-spanner-jdbc/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@
<truth.version>1.4.5</truth.version>
<mockito.version>4.11.0</mockito.version>
<opencensus.version>0.31.1</opencensus.version>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down Expand Up @@ -345,7 +346,7 @@
<argument>${project.basedir}/documentation/connection_properties.md</argument>
</arguments>
<classpathScope>test</classpathScope>
<skip>false</skip>
<skip>${maven.test.skip}</skip>
</configuration> </execution>
</executions>
</plugin>
Expand Down
1 change: 1 addition & 0 deletions java-spanner/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
<github.global.server>github</github.global.server>
<site.installationModule>google-cloud-spanner-parent</site.installationModule>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
1 change: 1 addition & 0 deletions java-storage-nio/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
<github.global.server>github</github.global.server>
<site.installationModule>google-cloud-storage-nio-parent</site.installationModule>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
3 changes: 3 additions & 0 deletions java-storage/gapic-google-cloud-storage-v2/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@
<artifactId>google-cloud-storage-parent</artifactId>
<version>2.71.0-SNAPSHOT</version><!-- {x-version-update:google-cloud-storage:current} -->
</parent>
<properties>
<skipUnitTests>true</skipUnitTests>
</properties>
<dependencies>
<dependency>
<groupId>io.grpc</groupId>
Expand Down
1 change: 1 addition & 0 deletions java-storage/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@
<github.global.server>github</github.global.server>
<site.installationModule>google-cloud-storage-parent</site.installationModule>
<google.cloud.shared-dependencies.version>3.31.0</google.cloud.shared-dependencies.version>
<skipUnitTests>false</skipUnitTests>
</properties>

<dependencyManagement>
Expand Down
Loading