New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
assist #9610: adjust rules for moving link children #3327
Conversation
@@ -96,7 +96,9 @@ private void moveImageWithNonSharedAnnotation(String src, String dest, | |||
assertEquals(iQuery.findAllByQuery(sb.toString(), param).size(), | |||
annotationIdsUser1.size()); | |||
n = 0; | |||
if (src.equals("rwrw--")) n = annotationIdsUser2.size(); | |||
if (src.equals("rwrw--") && !dest.equals("rw----")) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do not expect to see another user's annotations once we have moved the image into a private group.
Tried to move entities tagged with owners tags (in ro) and other's tags (in ra) to another group. Works as expected. |
Waiting for integration tests builds to finish. |
I don't think anything in the current crop of failures points to chgrp. The tests referenced in this PR pass. |
*-integration-java is green https://ci.openmicroscopy.org/view/5.1/job/OMERO-5.1-merge-integration-java/ |
*-python tests are yellow with four failed tests, I have to rely on @mtbc when he says they have nothing to do with this PR. https://ci.openmicroscopy.org/view/5.1/job/OMERO-5.1-merge-integration-python/340/ All seems fine here. |
assist #9610: adjust rules for moving link children
Marks three integration tests as no longer broken. Integration tests should still pass. Could look for regressions regarding moving images that have owner's and others' comments and tags between groups.
--no-rebase