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

Opened 13 years ago

Closed 13 years ago

Bug: DeleteServiceFilesTest failures

Reported by: cblackburn Owned by: cblackburn
Priority: minor Milestone: OMERO-Beta4.3
Component: Bin-Services Version: n.a.
Keywords: n.a. Cc: jamoore, jburel, cxallan
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2011-06-02 (13)

Description

Five tests in DeleteServiceFilesTest? fail. The reason seems to be that the helper method:

    private Image makeImageWithPixelsFile() throws ServerError, Exception {
        Image img = (Image) iUpdate.saveAndReturnObject(
                mmFactory.createImage());
        Pixels pix = img.getPrimaryPixels();
        IRenderingSettingsPrx rsPrx = factory.getRenderingSettingsService();
        List<Long> ids = new ArrayList<Long>();
        ids.add(pix.getId().getValue());
        rsPrx.resetDefaultsInSet(Pixels.class.getName(), ids);
        return img;
    }

no longer creates Pixels files.

These failures pre-date the changes in #5524 and #5572.

Since the deletion of _pyramid , _pryramid.pyr_lock and _pryramid*.tmp also needs to be tested a new method to create files in the Pixels directory is needed.

Change History (7)

comment:1 Changed 13 years ago by cblackburn

  • Owner set to cblackburn
  • Remaining Time set to 0.25
  • Status changed from new to accepted

comment:2 Changed 13 years ago by cblackburn

  • Milestone changed from OMERO-Beta4.3.1 to OMERO-Beta4.3
  • Sprint set to 2011-06-02 (13)

comment:3 Changed 13 years ago by cblackburn

  • Owner cblackburn deleted
  • Status changed from accepted to new

comment:4 Changed 13 years ago by cblackburn

  • Owner set to cblackburn
  • Status changed from new to accepted

comment:5 Changed 13 years ago by Colin Blackburn <C.Blackburn@…>

(In [6a7ad0f1118666057716664e8a246d0569ca7a46/ome.git] on branch develop) Delete tests now pass using repository create file method. See #5593

comment:6 Changed 13 years ago by Colin Blackburn <C.Blackburn@…>

(In [430564016b7d2313fce69306345d2d36190e7d58/ome.git] on branch develop) Additional delete tests added for pyramid files plus refactoring. See #5593

comment:7 Changed 13 years ago by cblackburn

  • Remaining Time changed from 0.25 to 0
  • Resolution set to fixed
  • Status changed from accepted to closed
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.66133 sec.)

We're Hiring!