Merge pull request #34335 from jprinet/fix/disable-test-build-scans-p…

…ublication

Disable build scan publication for test builds

.mvn/gradle-enterprise-custom-user-data.groovy

@@ -1,4 +1,16 @@
 
+// Configure build scan publication
+boolean publish = true
+if(session?.getRequest()?.getBaseDirectory() != null) {
+    def testBuildPaths = [ '/target/codestart-test/', '/target/it/', '/target/test-classes/', '/target/test-project/']
+    publish = testBuildPaths.every {testBuildPath -> !session.getRequest().getBaseDirectory().contains(testBuildPath) }
+    if(!publish) {
+        // do not publish a build scan for test builds
+        log.debug("Disabling build scan publication for " + session.getRequest().getBaseDirectory())
+    }
+}
+buildScan.publishAlwaysIf(publish)
+buildScan.publishIfAuthenticated()
 
 // Add mvn command line
 def mvnCommand = ''

.mvn/gradle-enterprise.xml

@@ -5,15 +5,12 @@
      </server>
     <buildScan>
         <!-- adjust conditions ?
-        mvn gradle-enterprise:provision-access-key 
+        mvn gradle-enterprise:provision-access-key
         https://docs.gradle.com/enterprise/maven-extension/#publishing_based_on_criteria
          -->
-        <!-- only publish test runs when failure (no option to completely avoid it)-->
-        <publish>#{basedir.contains('test-classes') ? 'ON_FAILURE' : 'ALWAYS'}</publish>
-        <!-- tag which builds are tests vs real -->
-        <tags>
-          <tag>#{basedir.contains('test-classes') ? 'TEST-BUILD' : 'MAIN-BUILD'}</tag>
-        </tags>
+        <!-- build scan publication is configured in gradle-enterprise-custom-user-data.groovy
+        to leverage options to disable build scan publication for test builds
+        -->
         <obfuscation>
           <!-- Don't share ip addresses-->
           <ipAddresses>#{{'0.0.0.0'}}</ipAddresses>
@@ -23,7 +20,6 @@
         </capture>
         <!-- https://docs.gradle.com/enterprise/maven-extension/#manual_access_key_configuration -->
         <backgroundBuildScanUpload>#{env['CI'] == null}</backgroundBuildScanUpload>
-        <publishIfAuthenticated>true</publishIfAuthenticated>
     </buildScan>
     <buildCache>
         <local><enabled>false</enabled></local>