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

Enable build cache by default in Gradle integration tests #3716

Merged
merged 6 commits into from
Aug 1, 2024

Conversation

adam-enko
Copy link
Member

@adam-enko adam-enko commented Jul 29, 2024

Enabling build cache by default will improve performance when re-running the tests locally.

Update the test assertions to check that tasks either succeed or are loaded from cache (which does not make a difference in terms of integration tests).

Additionally, use GRADLE_RO_DEP_CACHE to re-use the host machine's Gradle dependencies cache. This will improve test performance locally and on CI.

Result

When running the tests locally I see an improvement of ~5 minutes when re-running the tests after a minor change in DGP.

image

https://ge.jetbrains.com/c/ihppwm4njsxpa/wv56vbdme3ais/task-inputs

^OSIP-355

@adam-enko adam-enko added the runner: Gradle plugin An issue/PR related to Dokka's Gradle plugin label Jul 29, 2024
Enabling build cache by default will improve performance when re-running the tests locally.

Update the test assertions to check that tasks either succeed or are loaded from cache (which does not make a difference in terms of integration tests).

Additionally, use `GRADLE_RO_DEP_CACHE` to re-use the host machine's Gradle dependencies cache. This will improve test performance locally and on CI.
@adam-enko adam-enko force-pushed the adam/feat/enable-build-cache-in-integration-tests branch from cdad967 to bbb4185 Compare July 29, 2024 18:57
@adam-enko adam-enko marked this pull request as ready for review July 30, 2024 07:07
@dkrasnoff
Copy link

Good job 👍

@adam-enko adam-enko merged commit 89c5200 into master Aug 1, 2024
10 of 13 checks passed
@adam-enko adam-enko deleted the adam/feat/enable-build-cache-in-integration-tests branch August 1, 2024 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
runner: Gradle plugin An issue/PR related to Dokka's Gradle plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants