Skip to content

Conversation

@duanemay
Copy link
Member

Update unit test settings

making them closer to the updated integration_tests.sh settings

Add additional logging for test timing

Display for each gradle subproject and slow tests
e.g.

[cloudfoundry-identity-metrics-data] Unit tests completed: 42 tests, 42 passed, 0 failed, 0 skipped in 6978ms

SLOW TEST: org.cloudfoundry.identity.uaa.performance.LoginPagePerformanceMockMvcTest.idpDiscoveryRedirectsToOIDCProvider(JdbcIdentityProviderProvisioning) took 32292ms

Adjust metaspace size

This gradle default is 384m, as shown in the doc:
https://docs.gradle.org/9.2.0/userguide/config_gradle.html#sec:configuring_jvm_memory

Was seeing the following message:

The Daemon will expire after the build after running out of JVM Metaspace.
The project memory settings are likely not configured or are configured to an insufficient value.
The daemon will restart for the next build, which may increase subsequent build times.
These settings can be adjusted by setting 'org.gradle.jvmargs' in 'gradle.properties'.
The currently configured max heap space is '1 GiB' and the configured max metaspace is '128 MiB'.
For more information on how to set these values, please refer to https://docs.gradle.org/9.2.0/userguide/build_environment.html#sec:configuring_jvm_memory in the Gradle documentation.
To disable this warning, set 'org.gradle.daemon.performance.disable-logging=true'.

make it closer to the updated integration_tests.sh settings
e.g.
[cloudfoundry-identity-metrics-data] Unit tests completed: 42 tests, 42 passed, 0 failed, 0 skipped in 6978ms

SLOW TEST: org.cloudfoundry.identity.uaa.performance.LoginPagePerformanceMockMvcTest.idpDiscoveryRedirectsToOIDCProvider(JdbcIdentityProviderProvisioning) took 32292ms
This doc shows the default is 384m
https://docs.gradle.org/9.2.0/userguide/config_gradle.html#sec:configuring_jvm_memory

Seeing the following message:

The Daemon will expire after the build after running out of JVM Metaspace.
The project memory settings are likely not configured or are configured to an insufficient value.
The daemon will restart for the next build, which may increase subsequent build times.
These settings can be adjusted by setting 'org.gradle.jvmargs' in 'gradle.properties'.
The currently configured max heap space is '1 GiB' and the configured max metaspace is '128 MiB'.
For more information on how to set these values, please refer to https://docs.gradle.org/9.2.0/userguide/build_environment.html#sec:configuring_jvm_memory in the Gradle documentation.
To disable this warning, set 'org.gradle.daemon.performance.disable-logging=true'.
@github-project-automation github-project-automation bot moved this from Inbox to Pending Merge | Prioritized in Foundational Infrastructure Working Group Nov 26, 2025
@fhanik
Copy link
Contributor

fhanik commented Nov 26, 2025

Only feedback would be to squash the commits into 1. It's all part of build and test runs

@duanemay duanemay merged commit 5a77e8e into develop Nov 27, 2025
32 of 33 checks passed
@duanemay duanemay deleted the unit_test_timing branch November 27, 2025 02:40
@github-project-automation github-project-automation bot moved this from Pending Merge | Prioritized to Done in Foundational Infrastructure Working Group Nov 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants