Skip to content

Commit 64736f0

Browse files
authored
More Readable Gradle test config. (#18890)
The integration testing logs are extremely noisy today. The logs contain the entire test STDOUT and are a pain to read. This PR contains a better test setup config to reduce logging output. The main change is to comment out ShowStandardStreams in the integration tests plugin. Then we add the started event so Gradle shows us when a test is started. Some tests take a longer time. It is useful to know they have begun so it's clear if the job is stuck or not. Standardise the above setting across all of our custom test tasks.
1 parent 188612c commit 64736f0

File tree

2 files changed

+14
-6
lines changed

2 files changed

+14
-6
lines changed

build.gradle

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -357,8 +357,9 @@ subprojects { subproj ->
357357
excludeTags(integrationTagName, slowIntegrationTagName, cloudStorageTestTagName)
358358
}
359359
testLogging() {
360-
events 'failed'
360+
events "passed", "skipped", "failed"
361361
exceptionFormat 'full'
362+
// uncomment to get the full log output
362363
// showStandardStreams = true
363364
}
364365
finalizedBy jacocoTestReport
@@ -369,8 +370,10 @@ subprojects { subproj ->
369370
includeTags integrationTagName
370371
}
371372
testLogging() {
372-
events 'failed'
373+
events "passed", "failed", "started"
373374
exceptionFormat 'full'
375+
// uncomment to get the full log output
376+
// showStandardStreams = true
374377
}
375378
finalizedBy jacocoTestReport
376379
}
@@ -380,17 +383,21 @@ subprojects { subproj ->
380383
includeTags slowIntegrationTagName
381384
}
382385
testLogging() {
383-
events 'failed'
386+
events "passed", "failed", "started"
384387
exceptionFormat 'full'
388+
// uncomment to get the full log output
389+
// showStandardStreams = true
385390
}
386391
finalizedBy jacocoTestReport
387392
}
388393

389394
task allTests(type: Test) {
390395
useJUnitPlatform()
391396
testLogging() {
392-
events 'failed'
397+
events "passed", "failed", "started"
393398
exceptionFormat 'full'
399+
// uncomment to get the full log output
400+
// showStandardStreams = true
394401
}
395402
finalizedBy jacocoTestReport
396403
}

buildSrc/src/main/groovy/airbyte-integration-test-java.gradle

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,10 @@ class AirbyteIntegrationTestJavaPlugin implements Plugin<Project> {
3131
}
3232

3333
testLogging() {
34-
events "passed", "failed"
34+
events "passed", "failed", "started"
3535
exceptionFormat "full"
36-
showStandardStreams = true
36+
// uncomment to get the full log output
37+
// showStandardStreams = true
3738
}
3839

3940
outputs.upToDateWhen { false }

0 commit comments

Comments
 (0)