feat: updated saveTokenAttachment

pagopa · Nov 27, 2024 · a2290ba · a2290ba
1 parent 982ce4f
commit a2290ba
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 22 deletions.
diff --git a/...ding-functions/src/main/java/it/pagopa/selfcare/onboarding/service/OnboardingService.java b/...ding-functions/src/main/java/it/pagopa/selfcare/onboarding/service/OnboardingService.java
@@ -146,8 +146,6 @@ public void saveTokenWithAttachment(OnboardingAttachment onboardingAttachment) {
 
     Onboarding onboarding = onboardingAttachment.getOnboarding();
 
-    //  if (checkTokenExist(onboarding)) return;
-
     Product product = productService.getProductIsValid(onboarding.getProductId());
 
     File contract = contractService.retrieveAttachment(onboardingAttachment, product.getTitle());
@@ -173,6 +171,7 @@ private void saveToken(OnboardingWorkflow onboardingWorkflow, Product product, S
 
     // Persist token entity
     Token token = buildBaseToken(onboarding, digest);
+    token.setId(onboarding.getId());
     token.setContractTemplate(onboardingWorkflow.getContractTemplatePath(product));
     token.setContractVersion(onboardingWorkflow.getContractTemplateVersion(product));
     token.setContractFilename(
@@ -191,6 +190,7 @@ private void saveTokenAttachment(
 
     // Persist token entity
     Token token = buildBaseToken(onboarding, digest);
+    token.setId(UUID.randomUUID().toString());
     token.setContractTemplate(attachmentTemplate.getTemplatePath());
     token.setContractVersion(attachmentTemplate.getTemplateVersion());
     token.setContractFilename(
@@ -204,7 +204,6 @@ private void saveTokenAttachment(
   private Token buildBaseToken(Onboarding onboarding, String digest) {
     log.debug("creating Token for onboarding {} ...", onboarding.getId());
     Token token = new Token();
-    token.setId(UUID.randomUUID().toString());
     token.setOnboardingId(onboarding.getId());
     token.setCreatedAt(LocalDateTime.now());
     token.setUpdatedAt(LocalDateTime.now());

diff --git a/...-functions/src/test/java/it/pagopa/selfcare/onboarding/service/OnboardingServiceTest.java b/...-functions/src/test/java/it/pagopa/selfcare/onboarding/service/OnboardingServiceTest.java
@@ -343,25 +343,6 @@ void saveTokenAttachment() {
     assertEquals(digestExpected, tokenArgumentCaptor.getValue().getChecksum());
   }
 
-  /*
-    @Test
-    void saveTokenAttachment_shouldSkipIfTokenExists() {
-      Onboarding onboarding = createOnboarding();
-      AttachmentTemplate attachmentTemplate = createDummyAttachmentTemplate();
-      OnboardingAttachment onboardingAttachment = new OnboardingAttachment();
-      onboardingAttachment.setOnboarding(onboarding);
-      onboardingAttachment.setAttachment(attachmentTemplate);
-      Token token = createDummyToken();
-
-      when(tokenRepository.findByOnboardingId(onboarding.getId())).thenReturn(Optional.of(token));
-
-      onboardingService.saveTokenWithAttachment(onboardingAttachment);
-
-      Mockito.verify(tokenRepository, Mockito.times(1)).findByOnboardingId(onboarding.getId());
-      Mockito.verifyNoMoreInteractions(tokenRepository);
-    }
-  */
-
   @Test
   void loadContract() {