User Story #9057 (closed)
Opened 7 years ago
Closed 7 years ago
Permissions Testing
| Reported by: | jburel | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | OMERO-4.4.7 |
| Component: | General | Keywords: | n.a. |
| Cc: | omero-team@… | Story Points: | n.a. |
| Sprint: | n.a. | Importance: | n.a. |
| Total Remaining Time: | 0.0d | Estimated Remaining Time: | n.a. |
Description (last modified by jburel)
Write tests to cover as much as possible the
Testing 4.4. permissions g.doc
New classes should be created when possible to avoid merge conflicts. cf. issue when merging hierarchyMoveTest
All or most of the actions which should happen during a delete should also take place for a chgrp. In fact, for a member of the old group who is not in the new group, it should look as if delete had been used. Therefore, it would make sense to copy/rewrite all of the tests under OmeroJava/test/integration/delete to also cover chgrp. If we really just copy the files, then when modifying one set, we will remember to modify the other. Alternatively, we could refactor them to test all of the actions: delete, chgrp, chown, .... etc.
Change History (5)
comment:1 Changed 7 years ago by cblackburn
- Summary changed from Permissions Testinghttps://trac.openmicroscopy.org.uk/ome/newticket?type=story to Permissions Testing
comment:2 Changed 7 years ago by jburel
- Description modified (diff)
comment:3 Changed 7 years ago by agilo
- Status changed from new to accepted
comment:4 Changed 7 years ago by jburel
- Description modified (diff)
- Milestone changed from OMERO-Beta4.4 to OMERO-Beta4.4.x
comment:5 Changed 7 years ago by jmoore
- Resolution set to fixed
- Status changed from accepted to closed
Closing since #9060 has been marked invalid.
Updated status, related task in progress