{"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":2,"created_at":"2016-05-18T18:27:24Z","updated_at":"2016-06-09T17:10:29Z","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 about starting on the list of tests and changes of existing tests listed in [card comment](https://trello.com/c/RpdAOmKp/37-permissions-system-problems-chown#comment-5734d6a14a903d90c3e3b14a).\r\n\r\nThe tests added here should deal with the \r\n - bullet points 1 and 2 of [the card](https://trello.com/c/RpdAOmKp/37-permissions-system-problems-chown#comment-5734d6a14a903d90c3e3b14a), which are 2 corrections of the existing tests, and\r\n - subbulletpoint 1 of the bullet point 3 of  [the card](https://trello.com/c/RpdAOmKp/37-permissions-system-problems-chown#comment-5734d6a14a903d90c3e3b14a) (admin chowning, tried to stick to the chgrp test example and the existing tests in the chown test suite, so hopefully minimal enough)\r\n\r\nNot ready for review yet, except for @ximenesuk and @mtbc to have a look and give some directions please. \r\n\r\n"},"comment":{"url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/comments/224962383","html_url":"https://github.com/openmicroscopy/openmicroscopy/pull/4669#issuecomment-224962383","issue_url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/4669","id":224962383,"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},"created_at":"2016-06-09T17:10:28Z","updated_at":"2016-06-09T17:10:28Z","body":"The last commit https://github.com/openmicroscopy/openmicroscopy/pull/4669/commits/7c59c652cdf502af71b0afbe1983f99de8c64900 is the only test which seems to fit for ``chown`` situation as opposed to ``delete`` situation. The other tests in this [group of delete tests](https://github.com/openmicroscopy/openmicroscopy/blob/develop/components/tools/OmeroPy/test/integration/clitest/test_delete.py#L348) deal with \"Default Exclusion\" which is not present for Chown. The only way how to include them in chown tests would be to use Read-annotate group (instead of the private used in all the tests here and in Delete test). @ximenesuk @mtbc - is it something worthwile ?"}}
{"actor":{"display_login":"pwalczysko"}}