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

Opened 12 years ago

Closed 8 years ago

BUG: omero web unittest ignores ICE_CONFIG env var

Reported by: cneves Owned by: cneves
Priority: major Milestone: Testing2
Component: General Version: OMERO-5.2.0
Keywords: n.a. Cc: python-team@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

When running unittests using omero web unittest --test=webgateway and similar, ICE_CONFIG defaults to $DISTDIR/etc/ice.config and the ICE_CONFIG env variable is simply ignored.

Change History (3)

comment:1 Changed 12 years ago by jmoore

Used this to trace down the issue with ICE_CONFIG:

@tickets/9643 ~/git/dist $ git diff --cached
diff --git a/components/tools/OmeroPy/src/omero/gateway/scripts/dbhelpers.py b/components/tools/OmeroPy/src/omero/gateway/scripts/dbhelpers.py
index 8a9fea1..98a0f2c 100644
--- a/components/tools/OmeroPy/src/omero/gateway/scripts/dbhelpers.py
+++ b/components/tools/OmeroPy/src/omero/gateway/scripts/dbhelpers.py
@@ -34,7 +34,12 @@ def refreshConfig ():
         ru = bg.c.ic.getProperties().getProperty('omero.rootuser')
         rp = bg.c.ic.getProperties().getProperty('omero.rootpass')
     finally:
-        bg.seppuku()
+        try:
+            bg.seppuku()
+        except:
+            print "ERROR ON seppuku DURING FINALLY!"
+            import traceback
+            traceback.print_exc()
 
     if ru:
         ROOT.name = ru
@@ -474,7 +479,8 @@ def bootstrap (onlyUsers=False):
             i = i.create()
             i._conn.seppuku()
     finally:
-        client.seppuku()
+        if client:
+            client.seppuku()
 
 def cleanup ():
     for k, p in PROJECTS.items():

comment:2 Changed 8 years ago by jamoore

  • Milestone changed from 5.x to Testing2
  • Version set to OMERO-5.2.0

comment:3 Changed 8 years ago by sbesson

  • Resolution set to wontfix
  • Status changed from new to closed

As omero web unittest has been removed in favor of pytest, marking this as wontfix - see https://github.com/openmicroscopy/openmicroscopy/pull/1460.

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

We're Hiring!