fixed deploy mapping

pagopa · Nov 24, 2023 · 43d0379 · 43d0379
1 parent 24a7afb
commit 43d0379
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/...n/java/it/gov/pagopa/atmlayer/service/model/service/impl/WorkflowResourceServiceImpl.java b/...n/java/it/gov/pagopa/atmlayer/service/model/service/impl/WorkflowResourceServiceImpl.java
@@ -206,13 +206,11 @@ public Uni<WorkflowResource> setDeployInfo(UUID uuid, DeployResponseDto response
                         DeployedBPMNProcessDefinitionDto deployedProcessInfo = optionalDeployedProcessDefinition.get();
                         workflowResource.setDefinitionVersionCamunda(deployedProcessInfo.getVersion());
                         workflowResource.setDeploymentId(deployedProcessInfo.getDeploymentId());
-                        workflowResource.setCamundaDefinitionId(deployedProcessInfo.getId());
                         workflowResource.setDeployedFileName(deployedProcessInfo.getName());
                         workflowResource.setDescription(deployedProcessInfo.getDescription());
                         workflowResource.setResource(deployedProcessInfo.getResource());
                         workflowResource.setStatus(StatusEnum.DEPLOYED);
-                    }
-                    if (response.getDeployedDecisionDefinitions() != null) {
+                    } else if (response.getDeployedDecisionDefinitions() != null) {
                         Map<String, DeployedDMNDecisionDefinitionDto> deployedDecisionDefinitions = response.getDeployedDecisionDefinitions();
                         Optional<DeployedDMNDecisionDefinitionDto> optionalDeployedDecisionDefinition = deployedDecisionDefinitions.values()
                                 .stream().findFirst();
@@ -222,15 +220,14 @@ public Uni<WorkflowResource> setDeployInfo(UUID uuid, DeployResponseDto response
                         DeployedDMNDecisionDefinitionDto deployedDecisionDefinition = optionalDeployedDecisionDefinition.get();
                         workflowResource.setDefinitionVersionCamunda(deployedDecisionDefinition.getVersion());
                         workflowResource.setDeploymentId(deployedDecisionDefinition.getDeploymentId());
-                        workflowResource.setCamundaDefinitionId(deployedDecisionDefinition.getId());
                         workflowResource.setDeployedFileName(deployedDecisionDefinition.getName());
                         workflowResource.setResource(deployedDecisionDefinition.getResource());
                         workflowResource.setStatus(StatusEnum.DEPLOYED);
                     } else {
-                        workflowResource.setCamundaDefinitionId(response.getId());
                         workflowResource.setDeployedFileName(response.getName());
                         workflowResource.setStatus(StatusEnum.DEPLOYED);
                     }
+                    workflowResource.setCamundaDefinitionId(response.getId());
                     return this.workflowResourceRepository.persist(workflowResource);
                 }));
     }

diff --git a/...va/it/gov/pagopa/atmlayer/service/model/service/impl/WorkflowResourceServiceImplTest.java b/...va/it/gov/pagopa/atmlayer/service/model/service/impl/WorkflowResourceServiceImplTest.java
@@ -204,11 +204,8 @@ void setDeployInfoEmptyProcessDefinitions() {
     void setDeployInfoEmptyDecisionDefinitions() {
         UUID expectedId = UUID.randomUUID();
         WorkflowResource expectedWorkflowResource = new WorkflowResource();
-        Map<String, DeployedBPMNProcessDefinitionDto> deployedProcessDefinitions = new HashMap<>();
-        deployedProcessDefinitions.put("key", new DeployedBPMNProcessDefinitionDto());
         Map<String, DeployedDMNDecisionDefinitionDto> deployedDecisionDefinitions = new HashMap<>();
         DeployResponseDto deployResponseDto = new DeployResponseDto();
-        deployResponseDto.setDeployedProcessDefinitions(deployedProcessDefinitions);
         deployResponseDto.setDeployedDecisionDefinitions(deployedDecisionDefinitions);
         when(workflowResourceRepository.findById(any(UUID.class))).thenReturn(Uni.createFrom().item(expectedWorkflowResource));
         workflowResourceService.setDeployInfo(expectedId, deployResponseDto)