Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #2946 (closed)

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

Check that SOFT deletes don't accidentally cause files to be deleted.

Reported by: jamoore Owned by: cblackburn
Priority: critical Milestone: OMERO-Beta4.2.1
Component: Services Version: n.a.
Keywords: n.a. Cc:
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2010-10-28 (18)

Description

Basically, do we have tests to know that there is no way for ids of binary files to be passed back when the transaction (or the savepoint) which tried to delete them were rolled back due to a SOFT flag or similar?

Change History (7)

comment:1 Changed 14 years ago by jburel

  • Sprint changed from 2010-09-09 (16) to 2010-09-30 (17)

comment:2 Changed 14 years ago by cblackburn

(In [8279]) Helper method added to get the DeleteReport? back from a delete. Check on undeletedFiles added to existing tests. This should also help in testing soft deletes, see #2946

comment:3 Changed 14 years ago by jburel

  • Sprint changed from 2010-09-30 (17) to 2010-10-28 (18)

Moved from sprint 2010-09-30 (17)

comment:4 Changed 14 years ago by cblackburn

  • Status changed from new to assigned

comment:5 Changed 14 years ago by cblackburn

  • Remaining Time set to 0.5

comment:6 Changed 14 years ago by cblackburn

  • Remaining Time changed from 0.5 to 0
  • Resolution set to fixed
  • Status changed from assigned to closed

(In [8375]) Test added that might test this.
With delete refactoring this test may not be as important.
Closing for 4.2.1

comment:7 Changed 14 years ago by cblackburn

(In [8388]) Test added. Refactored to use helper method. See #2945

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.69738 sec.)

We're Hiring!