get ReturnDatasetToAuthorCommandTest passing #4124 #4139

The relationship between locks and users is confusing. I left a TODO.

src/main/java/edu/harvard/iq/dataverse/engine/command/impl/ReturnDatasetToAuthorCommand.java

@@ -17,7 +17,6 @@
 import java.sql.Timestamp;
 import java.util.Date;
 import java.util.List;
-import java.util.Set;
 import java.util.concurrent.Future;
 
 @RequiredPermissions(Permission.PublishDataset)
@@ -63,16 +62,12 @@ public Dataset save(CommandContext ctxt) throws CommandException {
         for (DatasetLock lock : theDataset.getLocks()) {
             if (DatasetLock.Reason.InReview.equals(lock.getReason())) {
                 theDataset.removeLock(lock);
+                // TODO: Are we supposed to remove the dataset lock from the user? What's going on here?
                 authenticatedUser = lock.getUser();
-                authenticatedUser.getDatasetLocks().remove(lock);
             }
         }
         Dataset savedDataset = ctxt.em().merge(theDataset);
         ctxt.em().flush();
-        if (authenticatedUser != null) {
-            // Remove lock from user.
-            ctxt.em().merge(authenticatedUser);
-        }
 
         DatasetVersionUser ddu = ctxt.datasets().getDatasetVersionUser(theDataset.getLatestVersion(), this.getUser());