expected artifacts from hashmap to map

spinnaker · Jul 22, 2019 · a746d98 · a746d98
1 parent 6f9408f
commit a746d98
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/...e/src/main/java/com/netflix/spinnaker/orca/pipelinetemplate/TemplatedPipelineRequest.java b/...e/src/main/java/com/netflix/spinnaker/orca/pipelinetemplate/TemplatedPipelineRequest.java
@@ -26,7 +26,7 @@ public class TemplatedPipelineRequest {
   String id;
   String schema;
   String type;
-  List<HashMap<String, Object>> expectedArtifacts;
+  List<Map<String, Object>> expectedArtifacts;
   Map<String, Object> trigger = new HashMap<>();
   Map<String, Object> config;
   Map<String, Object> template;
@@ -129,11 +129,11 @@ public boolean isKeepWaitingPipelines() {
     return keepWaitingPipelines;
   }
 
-  public void setExpectedArtifacts(List<HashMap<String, Object>> expectedArtifacts) {
+  public void setExpectedArtifacts(List<Map<String, Object>> expectedArtifacts) {
     this.expectedArtifacts = expectedArtifacts;
   }
 
-  public List<HashMap<String, Object>> getExpectedArtifacts() {
+  public List<Map<String, Object>> getExpectedArtifacts() {
     return this.expectedArtifacts;
   }
 }
diff --git a/...java/com/netflix/spinnaker/orca/pipelinetemplate/v1schema/V1SchemaExecutionGenerator.java b/...java/com/netflix/spinnaker/orca/pipelinetemplate/v1schema/V1SchemaExecutionGenerator.java
@@ -160,7 +160,7 @@ private void addExpectedArtifacts(
       TemplateConfiguration configuration,
       TemplatedPipelineRequest request) {
 
-    List<HashMap<String, Object>> mergedExpectedArtifacts =
+    List<Map<String, Object>> mergedExpectedArtifacts =
         mergeExpectedArtifact(
             emptyListIfNull(configuration.getConfiguration().getExpectedArtifacts()),
             emptyListIfNull(request.getExpectedArtifacts()));
@@ -175,13 +175,13 @@ private void addExpectedArtifacts(
     pipeline.put("expectedArtifacts", mergedExpectedArtifacts);
   }
 
-  private List<HashMap<String, Object>> emptyListIfNull(List<HashMap<String, Object>> eas) {
+  private List<? extends Map<String, Object>> emptyListIfNull(List<? extends Map<String, Object>> eas) {
     return Optional.ofNullable(eas).orElse(Collections.emptyList());
   }
 
-  private List<HashMap<String, Object>> mergeExpectedArtifact(
-      List<HashMap<String, Object>> eas1, List<HashMap<String, Object>> eas2) {
-    HashMap<String, HashMap<String, Object>> mergedByName = new HashMap<>();
+  private List<Map<String, Object>> mergeExpectedArtifact(
+      List<? extends Map<String, Object>> eas1, List<? extends Map<String, Object>> eas2) {
+    HashMap<String, Map<String, Object>> mergedByName = new HashMap<>();
 
     eas1.forEach(artifact -> mergedByName.put(String.valueOf(artifact.get("id")), artifact));
     eas2.forEach(artifact -> mergedByName.put(String.valueOf(artifact.get("id")), artifact));