Skip to content

Commit e546052

Browse files
authored
Improve running integration tests and examples in CI (#437)
- Re-enable output capturing in build scans For the same reasons stated in #435, output capturing had to be disabled to avoid leaking information about the Develocity server the library was tested with locally. With the current Develocity server it's being tested with, that's no longer a concern. Having console and tests output visible in build scans will make troubleshooting CI failures significantly easier. - Allow running examples in parallel, which should make CI faster - Enforce cache is disabled when running `:examples` tasks to guarantee that it is making the expected requests to Develocity (was already disabled for `examplesTest` test suite). There are pros and cons to this, but currently safety is preferred. All aimed at making CI faster and easier to troubleshoot, after #435. Part of work done to prevent regressions such as #426.
1 parent 86f2599 commit e546052

File tree

4 files changed

+1
-12
lines changed

4 files changed

+1
-12
lines changed

.github/workflows/pr.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ jobs:
1414
env:
1515
DEVELOCITY_API_URL: "${{ vars.DEVELOCITY_API_URL }}"
1616
DEVELOCITY_API_TOKEN: "${{ secrets.DEVELOCITY_API_TOKEN }}"
17+
DEVELOCITY_API_CACHE_ENABLED: "false"
1718
steps:
1819
- name: Checkout
1920
uses: actions/checkout@v4

examples/build.gradle.kts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -36,13 +36,6 @@ val runAll = tasks.register("runAll") {
3636
dependsOn(exampleTestTasks)
3737
}
3838

39-
val parallelExamples: String? by project
40-
if (!parallelExamples.toBoolean()) {
41-
exampleTestTasks.windowed(size = 2, step = 1).forEach { (a, b) ->
42-
b.configure { mustRunAfter(a) }
43-
}
44-
}
45-
4639
tasks.named("check") {
4740
dependsOn(runAll)
4841
}

gradle.properties

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ artifact=develocity-api-kotlin
33
version=2025.1.0
44
develocity.version=2025.1
55
repo.url=https://github.com/gabrielfeo/develocity-api-kotlin
6-
parallelExamples=false
76
org.gradle.parallel=true
87
org.gradle.jvmargs=-Xmx5g
98
org.gradle.caching=true

settings.gradle.kts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,6 @@ develocity {
2424
buildScan {
2525
termsOfUseUrl = "https://gradle.com/help/legal-terms-of-use"
2626
termsOfUseAgree = "yes"
27-
capture {
28-
buildLogging = false
29-
testLogging = false
30-
}
3127
obfuscation {
3228
ipAddresses { addresses -> addresses.map { _ -> "0.0.0.0" } }
3329
hostname { "-redacted-" }

0 commit comments

Comments
 (0)