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 #2880 (closed)

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

Delete and binary data

Reported by: jburel Owned by: cblackburn
Priority: critical Milestone: OMERO-Beta4.2.1
Component: General Version: n.a.
Keywords: n.a. Cc: wmoore, jamoore, cxallan
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2010-09-09 (16)

Description

Add tests to deleteServiceTest to make sure that original file and binary data are deleted.

Change History (14)

comment:1 Changed 9 years ago by atarkowska

  • Sprint set to 2010-09-09 (16)

comment:2 Changed 9 years ago by cblackburn

  • Owner set to cblackburn
  • Remaining Time set to 1
  • Status changed from new to assigned

comment:3 Changed 9 years ago by cblackburn

(In [8144]) Tests added for file delete. See #2880.

comment:4 Changed 9 years ago by cblackburn

  • Remaining Time changed from 1 to 0.5

The tests are fairly crude at the moment. A neater test using import might be better.

comment:5 Changed 9 years ago by jburel

(In [8146]) added test for thumbnail (see #2880)

comment:6 Changed 9 years ago by cblackburn

(In [8161]) fileExists method added to ICE API to support delete testing. See #2880

comment:7 Changed 9 years ago by cblackburn

(In [8162]) Delete files tests separated out. Delete tests now use repository service. Changes to other files to reflect this. Still room for refactoring. See #2880

comment:8 Changed 9 years ago by jburel

(In [8163]) Refactor code to delete objects (more classes). Add delete tests and permissions (see #2879 and see #2880)

comment:9 Changed 9 years ago by cblackburn

(In [8168]) Changing way config value is obtained so tests pass on Hudson. See #2880

comment:10 Changed 9 years ago by cblackburn

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

Tests now pass on Hudson as well as locally.

comment:11 Changed 9 years ago by cblackburn

(In [8170]) Adding more information to exception message to test on Hudson. See #2880

comment:12 Changed 9 years ago by cblackburn

(In [8173]) Checking path with and without trailing searator to test on Hudson. See #2880

comment:13 Changed 9 years ago by cblackburn

(In [8174]) Checking path with and without trailing searator to test on Hudson. See #2880

comment:14 Changed 9 years ago by cblackburn

(In [8175]) Replaces last accidental commit. Using FilenameUtils? to address trailing separator issues. See #2880

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.84041 sec.)

We're Hiring!