From 64f0bfcfc783694e2429fa74d7409303e0aada5f Mon Sep 17 00:00:00 2001 From: Davin Chia Date: Wed, 2 Nov 2022 21:35:05 -0700 Subject: [PATCH 1/4] Better test setup to reduce logging output and understand when tests are run. --- build.gradle | 15 +++++++++++---- .../groovy/airbyte-integration-test-java.gradle | 3 ++- 2 files changed, 13 insertions(+), 5 deletions(-) diff --git a/build.gradle b/build.gradle index e0a1bc11ce2f5..040e23fc34a43 100644 --- a/build.gradle +++ b/build.gradle @@ -356,8 +356,9 @@ subprojects { subproj -> excludeTags(integrationTagName, slowIntegrationTagName, cloudStorageTestTagName) } testLogging() { - events 'failed' + events "passed", "skipped", "failed" exceptionFormat 'full' + // uncomment to get the full log output // showStandardStreams = true } finalizedBy jacocoTestReport @@ -368,8 +369,10 @@ subprojects { subproj -> includeTags integrationTagName } testLogging() { - events 'failed' + events "passed", "skipped", "failed" exceptionFormat 'full' + // uncomment to get the full log output + // showStandardStreams = true } finalizedBy jacocoTestReport } @@ -379,8 +382,10 @@ subprojects { subproj -> includeTags slowIntegrationTagName } testLogging() { - events 'failed' + events "passed", "skipped", "failed" exceptionFormat 'full' + // uncomment to get the full log output + // showStandardStreams = true } finalizedBy jacocoTestReport } @@ -388,8 +393,10 @@ subprojects { subproj -> task allTests(type: Test) { useJUnitPlatform() testLogging() { - events 'failed' + events "passed", "skipped", "failed" exceptionFormat 'full' + // uncomment to get the full log output + // showStandardStreams = true } finalizedBy jacocoTestReport } diff --git a/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle b/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle index fde263668ce93..078c358f4da8b 100644 --- a/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle +++ b/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle @@ -31,8 +31,9 @@ class AirbyteIntegrationTestJavaPlugin implements Plugin { } testLogging() { - events "passed", "failed" + events "passed", "skipped", "failed" exceptionFormat "full" + // uncomment to get the full log output showStandardStreams = true } From f8d2a95add29f1ed4267f44ace985ba88b7eeda6 Mon Sep 17 00:00:00 2001 From: Davin Chia Date: Wed, 2 Nov 2022 21:35:46 -0700 Subject: [PATCH 2/4] Better test setup to reduce logging output and understand when tests are run. --- build.gradle | 6 +++--- .../src/main/groovy/airbyte-integration-test-java.gradle | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/build.gradle b/build.gradle index 040e23fc34a43..1ddcf34e53a6f 100644 --- a/build.gradle +++ b/build.gradle @@ -369,7 +369,7 @@ subprojects { subproj -> includeTags integrationTagName } testLogging() { - events "passed", "skipped", "failed" + events "passed", "skipped", "failed", "started" exceptionFormat 'full' // uncomment to get the full log output // showStandardStreams = true @@ -382,7 +382,7 @@ subprojects { subproj -> includeTags slowIntegrationTagName } testLogging() { - events "passed", "skipped", "failed" + events "passed", "skipped", "failed", "started" exceptionFormat 'full' // uncomment to get the full log output // showStandardStreams = true @@ -393,7 +393,7 @@ subprojects { subproj -> task allTests(type: Test) { useJUnitPlatform() testLogging() { - events "passed", "skipped", "failed" + events "passed", "skipped", "failed", "started" exceptionFormat 'full' // uncomment to get the full log output // showStandardStreams = true diff --git a/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle b/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle index 078c358f4da8b..8f469413e672e 100644 --- a/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle +++ b/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle @@ -31,7 +31,7 @@ class AirbyteIntegrationTestJavaPlugin implements Plugin { } testLogging() { - events "passed", "skipped", "failed" + events "passed", "skipped", "failed", "started" exceptionFormat "full" // uncomment to get the full log output showStandardStreams = true From 4932719e1a9503e447a5f5f480470347b332f819 Mon Sep 17 00:00:00 2001 From: Davin Chia Date: Wed, 2 Nov 2022 21:40:46 -0700 Subject: [PATCH 3/4] Better test setup to reduce logging output and understand when tests are run. --- build.gradle | 6 +++--- .../src/main/groovy/airbyte-integration-test-java.gradle | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/build.gradle b/build.gradle index 1ddcf34e53a6f..2b5f8de1c9cc3 100644 --- a/build.gradle +++ b/build.gradle @@ -369,7 +369,7 @@ subprojects { subproj -> includeTags integrationTagName } testLogging() { - events "passed", "skipped", "failed", "started" + events "passed", "failed", "started" exceptionFormat 'full' // uncomment to get the full log output // showStandardStreams = true @@ -382,7 +382,7 @@ subprojects { subproj -> includeTags slowIntegrationTagName } testLogging() { - events "passed", "skipped", "failed", "started" + events "passed", "failed", "started" exceptionFormat 'full' // uncomment to get the full log output // showStandardStreams = true @@ -393,7 +393,7 @@ subprojects { subproj -> task allTests(type: Test) { useJUnitPlatform() testLogging() { - events "passed", "skipped", "failed", "started" + events "passed", "failed", "started" exceptionFormat 'full' // uncomment to get the full log output // showStandardStreams = true diff --git a/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle b/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle index 8f469413e672e..84f8071ee7b34 100644 --- a/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle +++ b/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle @@ -31,7 +31,7 @@ class AirbyteIntegrationTestJavaPlugin implements Plugin { } testLogging() { - events "passed", "skipped", "failed", "started" + events "passed", "failed", "started" exceptionFormat "full" // uncomment to get the full log output showStandardStreams = true From f043c44bf849671e02731425e43b210a786a709d Mon Sep 17 00:00:00 2001 From: Davin Chia Date: Wed, 2 Nov 2022 21:41:47 -0700 Subject: [PATCH 4/4] Make sure to comment out showStandardStreams. --- buildSrc/src/main/groovy/airbyte-integration-test-java.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle b/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle index 84f8071ee7b34..767f309d3a0d9 100644 --- a/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle +++ b/buildSrc/src/main/groovy/airbyte-integration-test-java.gradle @@ -34,7 +34,7 @@ class AirbyteIntegrationTestJavaPlugin implements Plugin { events "passed", "failed", "started" exceptionFormat "full" // uncomment to get the full log output - showStandardStreams = true + // showStandardStreams = true } outputs.upToDateWhen { false }