Merge pull request #6250 from mtbc/test-other-group-duplication

Add integration test for other-group duplication.

components/tools/OmeroJava/test/integration/DuplicationTest.java

@@ -52,6 +52,7 @@
 import omero.model.DoubleAnnotationI;
 import omero.model.Ellipse;
 import omero.model.EllipseI;
+import omero.model.ExperimenterGroup;
 import omero.model.FileAnnotation;
 import omero.model.FileAnnotationI;
 import omero.model.Folder;
@@ -1287,6 +1288,61 @@ public void testDuplicateImageViaSkipHead() throws Exception {
         Assert.assertNotEquals(originalImageId, reportedImageId);
     }
 
+    /**
+     * Test duplication of an image that is not in the current group.
+     * @param isExplicitGroupContext if the group context should be set for the duplication
+     * @throws Exception unexpected
+     */
+    @Test(dataProvider = "group context test cases", groups = "broken")
+    public void testDuplicateImageDifferentGroup(boolean isExplicitGroupContext) throws Exception {
+        final EventContext ec = newUserAndGroup("rwra--");
+        final ExperimenterGroup otherGroup = newGroupAddUser("rwra--", ec.userId);
+        final long otherGroupId = otherGroup.getId().getValue();
+        loginUser(ec);
+
+        /* save an image then put it in another group */
+
+        final Image originalImage = (Image) iUpdate.saveAndReturnObject(mmFactory.simpleImage());
+        final long originalImageId = originalImage.getId().getValue();
+        testImages.add(originalImageId);
+
+        doChange(Requests.chgrp().target(originalImage).toGroup(otherGroup).build());
+
+        /* duplicate the image even though it is not in the current group */
+
+        final long currentGroupId = iAdmin.getEventContext().groupId;
+        Assert.assertNotEquals(currentGroupId, otherGroupId);
+
+        final Duplicate dup = Requests.duplicate().target(originalImage).build();
+        final DuplicateResponse response;
+
+        if (isExplicitGroupContext) {
+            response = (DuplicateResponse) doChange(dup, otherGroupId);
+        } else {
+            response = (DuplicateResponse) doChange(dup);
+        }
+
+        /* check that the response includes duplication of the image */
+
+        final Set<Long> reportedImageIds = new HashSet<Long>(response.duplicates.get("ome.model.core.Image"));
+
+        Assert.assertEquals(reportedImageIds.size(), 1);
+
+        /* check that the reported image has a new ID */
+
+        final long reportedImageId = reportedImageIds.iterator().next();
+        testImages.add(reportedImageId);
+
+        Assert.assertNotEquals(originalImageId, reportedImageId);
+
+        /* check that the reported image is in the other group */
+
+        loginUser(otherGroup);
+        final Image reportedImage = (Image) iQuery.get("Image", reportedImageId);
+        final long reportedImageGroup = reportedImage.getDetails().getGroup().getId().getValue();
+        Assert.assertEquals(reportedImageGroup, otherGroupId);
+    }
+
     /**
      * Test duplication of links depending on ownership and group permissions.
      * @param groupPerms the permissions on the group in which to test
@@ -1443,6 +1499,14 @@ public void testDuplicateWithContainerPermissions(String groupPerms, boolean myC
         }
     }
 
+    /**
+     * @return context for group context test cases
+     */
+    @DataProvider(name = "group context test cases")
+    public Object[][] provideGroupContextCases() {
+        return new Boolean[][] { new Boolean[] {false}, new Boolean[] {true} };
+    }
+
     /**
      * @return group permissions for container permissions test cases
      */