Task #6295 (closed)
Opened 13 years ago
Closed 12 years ago
Copy/refactor delete tests for chgrp
| Reported by: | jamoore | Owned by: | jamoore |
|---|---|---|---|
| Priority: | critical | Milestone: | OMERO-4.4.7 |
| Component: | Services | Version: | n.a. |
| Keywords: | n.a. | Cc: | jburel, cxallan |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | 0.0d |
| Sprint: | n.a. |
Description
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 13 years ago by jmoore
- Sprint 2011-08-04 (2) deleted
comment:2 Changed 13 years ago by jburel
Referencing ticket #3532 has changed sprint.
comment:3 Changed 13 years ago by jburel
- Milestone changed from OMERO-Beta4.3.2 to OME-5.0
comment:4 Changed 12 years ago by jmoore
- Milestone changed from OMERO-Beta4.4 to OMERO-Beta4.4.1
Hopefully we can spend some time on this after release.
comment:5 Changed 12 years ago by jburel
- Remaining Time changed from 2 to 0
- Resolution set to duplicate
- Status changed from new to closed
This should actually be a story. see #9057
Note: See
TracTickets for help on using
tickets.
You may also have a look at Agilo extensions to the ticket.
Not starting on this until J-M returns.