Task #6295 (closed)
Opened 8 years ago
Closed 7 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 8 years ago by jmoore
- Sprint 2011-08-04 (2) deleted
comment:2 Changed 8 years ago by jburel
Referencing ticket #3532 has changed sprint.
comment:3 Changed 8 years ago by jburel
- Milestone changed from OMERO-Beta4.3.2 to OME-5.0
comment:4 Changed 7 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 7 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.