BUG: Exporter not working across groups

[cneves]

exporter.generateTiff is not honoring {'omero.group':'-1'}.

​https://github.com/cneves/openmicroscopy/commit/0cdeda8385528d6b3043d6caea91b89bb66cc1d9 extends a test (gatewaytest.image.ImageTest?.testExport) that shows this behavior.

[jmoore]

Carlos, even after applying the following patch, I can't get the test to work properly:

diff --git a/components/tools/OmeroPy/src/omero/gateway/scripts/dbhelpers.py b/components/tools/OmeroPy/src/omero/gateway/scripts/dbhelpers.py
index acd1395..ce679a4 100644
--- a/components/tools/OmeroPy/src/omero/gateway/scripts/dbhelpers.py
+++ b/components/tools/OmeroPy/src/omero/gateway/scripts/dbhelpers.py
@@ -243,10 +243,13 @@ class DatasetEntry (ObjectEntry):
                 project = self.project
         else:
             project = forceproj
-        for d in project.listChildren():
-            if d.getName() == self.name and ((self.description is None and d.getDescription() == '') or (self.description is not None and omero.gateway.omero_type(d.getDescription()) == omero.gateway.omero_type(self.description))):
-                d.__loadedHotSwap__()
-                return d
+
+        if project:
+            for d in project.listChildren():
+                if d.getName() == self.name and ((self.description is None and d.getDescription() == '') or (self.description is not None and omero.gateway.omero_type(d.getDescription()) == omero.gateway.omero_type(self.description))):
+                    d.__loadedHotSwap__()
+                    return d
+
         return None
 
     def create (self):
@@ -286,9 +289,12 @@ class ImageEntry (ObjectEntry):
             dataset = DATASETS[self.dataset].get(client)
         else:
             dataset = forceds
-        for i in dataset.listChildren():
-            if i.getName() == self.name:
-                return i
+
+        if dataset:
+            for i in dataset.listChildren():
+                if i.getName() == self.name:
+                    return i
+
         return None
 
     def create (self):
diff --git a/components/tools/OmeroPy/test/gatewaytest/image.py b/components/tools/OmeroPy/test/gatewaytest/image.py
index f87c18d..806bca0 100644
--- a/components/tools/OmeroPy/test/gatewaytest/image.py
+++ b/components/tools/OmeroPy/test/gatewaytest/image.py
@@ -173,7 +173,7 @@ class ImageTest (lib.GTest):
         self.assert_(len(self.image.exportOmeTiff()) > 0)
         # Now try the same using a different user, admin first
         self.loginAsAdmin()
-        sopts = dict(self.gateway.CONFIG['SERVICE_OPTS'] or {})
+        sopts = {} # dict(self.gateway.CONFIG['SERVICE_OPTS'] or {})
         sopts['omero.group'] = '-1'
         self.gateway.CONFIG['SERVICE_OPTS'] = sopts
         image = self.getTestImage()

I'm seeing:

@3529-callcontext ~/tmp/sprint9/components/tools/OmeroPy $ python test/gatewaytest/image.py ImageTest.testExport
E
======================================================================
ERROR: Test exporting the image to ometiff
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test/gatewaytest/image.py", line 180, in testExport
    self.assertEqual(image.getId(), self.image.getId())
AttributeError: 'NoneType' object has no attribute 'getId'

----------------------------------------------------------------------
Ran 1 test in 7.490s

FAILED (errors=1)

which I don't think is what you meant with this ticket. Passing back to you.

[jmoore]

Note: I merged e86e4d8 - (carlos/merging_gs) fixing gateway.image exportOmeTiff test (8 hours ago) <Carlos Neves> in to create the branch I'm testing from:

@3529-callcontext ~/tmp/sprint9/components/tools/OmeroPy $ git graph
*   940b0db - (HEAD, 3529-callcontext) Merge commit 'e86e4d8' into 3529-callcontext (9 minutes ago) <jmoore>
|\  
| * e86e4d8 - (carlos/merging_gs) fixing gateway.image exportOmeTiff test (8 hours ago) <Carlos Neves>
| * d97a84e - fixing gateway.services tables test (8 hours ago) <Carlos Neves>
| * 31ce979 - removing print statement (8 hours ago) <Carlos Neves>
| * 9b2ee70 - Trying once more to remove spurious logging (8 hours ago) <Carlos Neves>
* | e3f1a9b - (josh/3529-callcontext) Use group of pixels in PixelDataThread (23 hours ago) <jmoore>

[jmoore]

Commit got lost. Closing invalid.