damccorm commented on code in PR #40147:
URL: https://github.com/apache/beam/pull/40147#discussion_r4037504367


##########
buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy:
##########
@@ -1263,6 +1263,30 @@ class BeamModulePlugin implements Plugin<Project> {
         if (System.getProperty("os.name").toLowerCase().contains("windows")) {
           systemProperty 'org.xerial.snappy.tempdir', 
System.getProperty('org.xerial.snappy.tempdir') ?: 
"${project.rootDir.absolutePath}/build/snappy_bin"
         }
+        // Report each test as it runs. Gradle logs only failures by default.
+        //   -PshowTests        one line per test, plus a per-task summary
+        //   -PshowTests=out    also forward test stdout/stderr
+        // A cached or up-to-date test task forks no JVM and prints nothing;
+        // add --no-build-cache --rerun to force real execution.
+        if (project.hasProperty('showTests')) {
+          testLogging {
+            events 'passed', 'skipped', 'failed'
+            exceptionFormat = 'full'
+            showExceptions = true
+            showCauses = true
+            showStackTraces = true
+            showStandardStreams = (project.property('showTests') == 'out')
+            displayGranularity = 0
+          }

Review Comment:
   Should these just be the default flags we use? (at least it would be nice to 
show that the test ran by default, even if at lower granularity)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to