{"action":"created","issue":{"url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/4669","repository_url":"https://api.github.com/repos/openmicroscopy/openmicroscopy","labels_url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/4669/labels{/name}","comments_url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/4669/comments","events_url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/4669/events","html_url":"https://github.com/openmicroscopy/openmicroscopy/pull/4669","id":155569990,"number":4669,"title":"Chown tests","user":{"login":"pwalczysko","id":2478303,"avatar_url":"https://avatars.githubusercontent.com/u/2478303?v=3","gravatar_id":"","url":"https://api.github.com/users/pwalczysko","html_url":"https://github.com/pwalczysko","followers_url":"https://api.github.com/users/pwalczysko/followers","following_url":"https://api.github.com/users/pwalczysko/following{/other_user}","gists_url":"https://api.github.com/users/pwalczysko/gists{/gist_id}","starred_url":"https://api.github.com/users/pwalczysko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pwalczysko/subscriptions","organizations_url":"https://api.github.com/users/pwalczysko/orgs","repos_url":"https://api.github.com/users/pwalczysko/repos","events_url":"https://api.github.com/users/pwalczysko/events{/privacy}","received_events_url":"https://api.github.com/users/pwalczysko/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/labels/develop","name":"develop","color":"02e10c"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":11,"created_at":"2016-05-18T18:27:24Z","updated_at":"2016-06-14T08:48:55Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/pulls/4669","html_url":"https://github.com/openmicroscopy/openmicroscopy/pull/4669","diff_url":"https://github.com/openmicroscopy/openmicroscopy/pull/4669.diff","patch_url":"https://github.com/openmicroscopy/openmicroscopy/pull/4669.patch"},"body":"See [chown trello card](https://trello.com/c/RpdAOmKp/37-permissions-system-problems-chown#comment-5734d47e92374a1dec3292d5).\r\nThis PR should be a suggestion to cpmplement the tests and changes of existing tests in [chown trello card](https://trello.com/c/RpdAOmKp/37-permissions-system-problems-chown#comment-5734d47e92374a1dec3292d5). It deals with Python tests only. \r\n\r\nThe tests added here should deal with the comments [comment1 on trello](https://trello.com/c/RpdAOmKp/37-permissions-system-problems-chown#comment-5734d6a14a903d90c3e3b14a) and [comment2 on trello](https://trello.com/c/RpdAOmKp/37-permissions-system-problems-chown#comment-5734d47e92374a1dec3292d50) as well as other tests which bring the [chown test](https://github.com/openmicroscopy/openmicroscopy/blob/develop/components/tools/OmeroPy/test/integration/clitest/test_chown.py) in line with the [delete test](https://github.com/openmicroscopy/openmicroscopy/blob/develop/components/tools/OmeroPy/test/integration/clitest/test_delete.py). \r\n\r\nThe two parts which are not completely in line between delete and chown tests are \"default exclusion\" which is missing in chown logic (on CLI) and the Tag/Tagset workflow (which again, is intrinsically different for delete and chown). I have added just two tests representing these two paragraphs in [delete test](https://github.com/openmicroscopy/openmicroscopy/blob/develop/components/tools/OmeroPy/test/integration/clitest/test_delete.py), namely https://github.com/openmicroscopy/openmicroscopy/pull/4669/commits/7c59c652cdf502af71b0afbe1983f99de8c64900 and https://github.com/openmicroscopy/openmicroscopy/pull/4669/commits/68385640dfafdaeea6b17fe3f3d79c65c1d9c9f3 which check the default behaviour in the private group of attachments and tags/tagsets. I think the permissions here should be checked via  the java integration test. \r\n\r\nTo test:\r\nCheck that the integration test [chown test](https://github.com/openmicroscopy/openmicroscopy/blob/develop/components/tools/OmeroPy/test/integration/clitest/test_chown.py) is passing and makes sense.\r\n"},"comment":{"url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/comments/225818413","html_url":"https://github.com/openmicroscopy/openmicroscopy/pull/4669#issuecomment-225818413","issue_url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/4669","id":225818413,"user":{"login":"ximenesuk","id":1065145,"avatar_url":"https://avatars.githubusercontent.com/u/1065145?v=3","gravatar_id":"","url":"https://api.github.com/users/ximenesuk","html_url":"https://github.com/ximenesuk","followers_url":"https://api.github.com/users/ximenesuk/followers","following_url":"https://api.github.com/users/ximenesuk/following{/other_user}","gists_url":"https://api.github.com/users/ximenesuk/gists{/gist_id}","starred_url":"https://api.github.com/users/ximenesuk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ximenesuk/subscriptions","organizations_url":"https://api.github.com/users/ximenesuk/orgs","repos_url":"https://api.github.com/users/ximenesuk/repos","events_url":"https://api.github.com/users/ximenesuk/events{/privacy}","received_events_url":"https://api.github.com/users/ximenesuk/received_events","type":"User","site_admin":false},"created_at":"2016-06-14T08:48:55Z","updated_at":"2016-06-14T08:48:55Z","body":"https://ci.openmicroscopy.org/job/OMERO-DEV-merge-integration-python/lastCompletedBuild/testReport/OmeroPy.test.integration.clitest.test_chown/"}}
{"actor":{"display_login":"ximenesuk"}}