[CI] Disable all external modules tests for non-snapshot builds

Author: brianseeders

test/external-modules/apm-integration/build.gradle

@@ -24,10 +24,6 @@ tasks.named("test").configure {
   enabled = false
 }
 
-tasks.named('javaRestTest').configure {
-  it.onlyIf("snapshot build") { buildParams.snapshotBuild }
-}
-
 dependencies {
   clusterModules project(':modules:apm')
   implementation project(':libs:logging')

test/external-modules/build.gradle

@@ -15,4 +15,8 @@ subprojects {
     licenseFile = layout.settingsDirectory.file('licenses/AGPL-3.0+SSPL-1.0+ELASTIC-LICENSE-2.0.txt').asFile
     noticeFile = layout.settingsDirectory.file('NOTICE.txt').asFile
   }
+
+  tasks.named { it == "javaRestTest" || it == "yamlRestTest" }.configureEach {
+    it.onlyIf("snapshot build") { buildParams.snapshotBuild }
+  }
 }

test/external-modules/delayed-aggs/build.gradle

@@ -9,10 +9,6 @@
 
 apply plugin: 'elasticsearch.legacy-yaml-rest-test'
 
-tasks.named('yamlRestTest').configure {
-  it.onlyIf("snapshot build") { buildParams.snapshotBuild }
-}
-
 esplugin {
   description = 'A test module that allows to delay aggregations on shards with a configurable time'
   classname ='org.elasticsearch.test.delayedshard.DelayedShardAggregationPlugin'

test/external-modules/die-with-dignity/build.gradle

@@ -23,7 +23,3 @@ GradleUtils.extendSourceSet(project, "main", "javaRestTest", tasks.named("javaRe
 tasks.named("test").configure {
   enabled = false
 }
-
-tasks.named('javaRestTest').configure {
-  it.onlyIf("snapshot build") { buildParams.snapshotBuild }
-}

test/external-modules/error-query/build.gradle

@@ -12,11 +12,6 @@ apply plugin: 'elasticsearch.internal-java-rest-test'
 
 tasks.named('javaRestTest') {
   usesDefaultDistribution()
-  it.onlyIf("snapshot build") { buildParams.snapshotBuild }
-}
-
-tasks.named('yamlRestTest').configure {
-  it.onlyIf("snapshot build") { buildParams.snapshotBuild }
 }
 
 esplugin {

test/external-modules/esql-heap-attack/build.gradle

@@ -20,5 +20,4 @@ esplugin {
 
 tasks.named('javaRestTest') {
   usesDefaultDistribution()
-  it.onlyIf("snapshot build") { buildParams.snapshotBuild }
 }

test/external-modules/jvm-crash/build.gradle

@@ -20,5 +20,4 @@ esplugin {
 
 tasks.named('javaRestTest') {
   usesDefaultDistribution()
-  it.onlyIf("snapshot build") { buildParams.snapshotBuild }
 }

test/external-modules/multi-project/build.gradle

@@ -17,3 +17,7 @@ dependencies {
 tasks.withType(StandaloneRestIntegTestTask).configureEach {
   usesDefaultDistribution()
 }
+
+tasks.named("javaRestTest").configure {
+  enabled = buildParams.snapshotBuild
+}