Include publication when getting submission

On deposit failed retry so repoCopy has publication if retry succeeds and repoCopy is created.
eclipse-pass · Oct 24, 2023 · 8b7d5e7 · 8b7d5e7
1 parent a2abbdc
commit 8b7d5e7
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/...vices/deposit-core/src/main/java/org/eclipse/pass/deposit/service/FailedDepositRetry.java b/...vices/deposit-core/src/main/java/org/eclipse/pass/deposit/service/FailedDepositRetry.java
@@ -61,7 +61,8 @@ public FailedDepositRetry(PassClient passClient, DepositTaskHelper depositTaskHe
     public void retryFailedDeposit(Deposit failedDeposit) {
         try {
             Deposit deposit = passClient.getObject(failedDeposit, "submission", "repository");
-            final Submission submission = deposit.getSubmission();
+            final Submission submission = passClient.getObject(deposit.getSubmission(),
+                "publication", "submitter");
             final Repository repository = deposit.getRepository();
 
             final Packager packager = packagerRegistry.get(repository.getRepositoryKey());

diff --git a/...ces/deposit-core/src/test/java/org/eclipse/pass/deposit/service/FailedDepositRetryIT.java b/...ces/deposit-core/src/test/java/org/eclipse/pass/deposit/service/FailedDepositRetryIT.java
@@ -15,6 +15,7 @@
  */
 package org.eclipse.pass.deposit.service;
 
+import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.ArgumentMatchers.any;
@@ -66,9 +67,13 @@ public void testFailedDepositRetry() throws Exception {
 
         Deposit actualDeposit = actualDeposits.getResult().iterator().next();
         Deposit updatedDeposit = passClient.getObject(actualDeposit, "repositoryCopy");
-        RepositoryCopy popRepoCopy = passClient.getObject(updatedDeposit.getRepositoryCopy(), "repository");
+        RepositoryCopy popRepoCopy = passClient.getObject(updatedDeposit.getRepositoryCopy(),
+            "repository", "publication");
         updatedDeposit.setRepositoryCopy(popRepoCopy);
         verify(passClient).updateObject(eq(updatedDeposit));
+        assertEquals(submission.getPublication().getId(), popRepoCopy.getPublication().getId());
+        assertEquals(1, submission.getRepositories().size());
+        assertEquals(submission.getRepositories().get(0).getId(), popRepoCopy.getRepository().getId());
     }
 
     private Submission initFailedSubmissionDeposit() throws Exception {