From 439aa01ee618ac4e4cb9d52c4fb3a0874bcd4335 Mon Sep 17 00:00:00 2001
From: Michael Muller <mmuller@enduden.com>
Date: Fri, 30 Aug 2019 10:43:07 -0400
Subject: [PATCH] Remove the "showAllOutput" property from the build (#247)

* Remove the "showAllOutput" property from the build

It doesn't work very well and has been superseded by "verboseTestOutput",
which does the same thing and more.
---
 build.gradle      | 9 ---------
 core/build.gradle | 3 ---
 2 files changed, 12 deletions(-)

diff --git a/build.gradle b/build.gradle
index 9c76bb3cf..1e4cc2a95 100644
--- a/build.gradle
+++ b/build.gradle
@@ -95,11 +95,6 @@ tasks.build.dependsOn(tasks.checkLicense)
 
 // Provide defaults for all of the project properties.
 
-// showAllOutput: boolean.  If true, dump all test output during the build.
-if (!project.hasProperty('showAllOutput')) {
-  ext.showAllOutput = 'false'
-}
-
 // Only do linting if the build is successful.
 gradleLint.autoLintAfterFailure = false
 
@@ -243,10 +238,6 @@ subprojects {
 
   if (['util', 'proxy', 'core', 'prober'].contains(project.name)) return
 
-  test {
-    testLogging.showStandardStreams = Boolean.parseBoolean(showAllOutput)
-  }
-
   ext.relativePath = "google/registry/${project.name}"
 
   sourceSets.each {
diff --git a/core/build.gradle b/core/build.gradle
index d77c1186d..380322a3b 100644
--- a/core/build.gradle
+++ b/core/build.gradle
@@ -568,9 +568,6 @@ task fragileTest(type: Test) {
 
   // Run every test class in a freshly started process.
   forkEvery 1
-
-  // Uncomment to see test outputs in stdout.
-  //testLogging.showStandardStreams = true
 }
 
 task outcastTest(type: Test) {