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

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

Bug: Test failing: gateway/chgrp.testTwoDatasetsChgrpToProject

Reported by: cblackburn Owned by:
Priority: blocker Milestone: 5.0.0-beta1
Component: OmeroPy Version: n.a.
Keywords: n.a. Cc: fs@…, wmoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: FS demo 4.1

Description

This test passes on dev_4_4 but fails on develop. See #10777

FAIL: testTwoDatasetsChgrpToProject (__main__.ChgrpTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "gatewaytest/chgrp.py", line 210, in testTwoDatasetsChgrpToProject
    self.assertNotEqual(None, img, "Image should still be available in original group")
AssertionError: Image should still be available in original group

Change History (4)

comment:1 Changed 8 years ago by wmoore

I think this behaviour shouldn't change with new MIF chgrp logic in 'develop', so the test is still valid.

comment:2 Changed 8 years ago by jamoore

Agreed. Most likely while enabling the moving of filesets some restriction was loosened. Mark, this might be an interesting intermediate thing to try to fix, before we break more.

comment:3 Changed 8 years ago by jmoore <josh@…>

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

(In [56ba561949968602fbd1eff1ebd5d6ca2ed23865/ome.git] on branch develop) Fix NPE in OMRI.java (Fix #10820)

A later check tests for the fileset to be null,
but the returned array can still be empty requiring
an earlier check.

comment:4 Changed 8 years ago by Josh Moore <josh@…>

(In [bf0bb69b139fe431cc7894dd8ddc96e52c20d840/ome.git] on branch develop) Merge pull request #1118 from joshmoore/10820-npe-ormi

Fix NPE in OMRI.java (Fix #10820)

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

We're Hiring!