test(pipelinetriggers): demonstrate behavior of SpEL expression evalu…

…ation (#1434)

that uses a non-public constructor/method, and so requires special treatment under java 17.

build.gradle

@@ -51,6 +51,12 @@ subprojects {
         jvmArgs '-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=8189'
       }
     }
+
+    // This is required for some SpEL expressions to evaluate properly with java
+    // 17.  It works with java 11 as well, but isn't required there.
+    tasks.withType(Test).configureEach {
+      jvmArgs += '--add-opens=java.base/java.util=ALL-UNNAMED'
+    }
   }
 
   if (name != "echo-bom" && name != "echo-api") {

echo-pipelinetriggers/src/test/groovy/com/netflix/spinnaker/echo/pipelinetriggers/postprocessors/ExpectedArtifactExpressionEvaluationPostProcessorSpec.groovy

@@ -76,4 +76,29 @@ class ExpectedArtifactExpressionEvaluationPostProcessorSpec extends Specificatio
     then:
     noExceptionThrown()
   }
+
+  def 'allows calling toString on an UnmodifiableMap'() {
+    given:
+    def artifact = ExpectedArtifact.builder()
+      .matchArtifact(
+        Artifact.builder()
+          // This fails under java 17 without something like
+          // --add-opens=java.base/java.util=ALL-UNNAMED as an argument to the jvm.
+          .name('${ {"foo": "bar"}.toString() }')
+          .version('77')
+          .type('maven/file')
+          .build())
+      .id('goodId')
+      .build()
+
+    def inputPipeline = createPipelineWith([artifact], trigger).withTrigger(trigger)
+
+    when:
+    def outputPipeline = artifactPostProcessor.processPipeline(inputPipeline)
+    def evaluatedArtifact = outputPipeline.expectedArtifacts[0].matchArtifact
+
+    then:
+    evaluatedArtifact.name == '{foo=bar}'
+    evaluatedArtifact.version == '77'
+  }
 }