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

Opened 13 years ago

Closed 13 years ago

Bug: failing test testMultiProcessSession

Reported by: wmoore Owned by: cblackburn
Priority: major Milestone: OMERO-Beta4.3.2
Component: OmeroPy Version: n.a.
Keywords: n.a. Cc: cneves
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2011-09-15 (5)

Description

Hi Carlos - I seem to remember creating a ticket for this before, but can't find it now.

Did you look at this earlier?

======================================================================
FAIL [15.485s]: testMultiProcessSession (gatewaytest.connection.ConnectionMethodsTest?)


Traceback (most recent call last):

File "/Users/will/Desktop/OMERO/components/tools/OmeroPy/test/gatewaytest/connection.py", line 32, in testMultiProcessSession

self.assertEqual(c2._session, None)

AssertionError?: object #0 (::omero::model::Session)
{
...
} != None

Change History (8)

comment:1 Changed 13 years ago by jmoore

  • Cc cneves-x added
  • Owner changed from cneves-x to jmoore

Taking away from Carlos so someone else can pick up.

comment:2 Changed 13 years ago by jmoore

  • Owner jmoore deleted

comment:3 Changed 13 years ago by atarkowska

  • Sprint set to 2011-09-15 (5)

comment:4 Changed 13 years ago by atarkowska

  • Owner set to atarkowska

comment:5 Changed 13 years ago by atarkowska

  • Owner changed from atarkowska to cblackburn

comment:6 Changed 13 years ago by cblackburn

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

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

(In [b520503bbfed5be732bb971fad62599110425b81/ome.git] on branch develop) Reversing the assertion logic. See #6017

comment:8 Changed 13 years ago by cblackburn

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

This is similar to the problem in #6037. Reversing the logic of some assertions passes the tests but does it now properly test the code? In the original lines,

        c2 = self.gateway.clone()
        self.assert_(c2.connect(sUuid=self.gateway._sessionUuid))
        self.assertEqual(c2._session, None)
        a = c2.getAdminService()

can the assertEqual(c2._session, None) be correct if the call on the next line is expected to be valid?

This test should be reviewed along with that in #6037 to check that it fulfills its original purpose.

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

We're Hiring!