TESTS/DOC: added test and TODO for #1155

Author: gdementen

larray/core/group.py

@@ -752,6 +752,13 @@ def __init__(self, key, name=None, axis=None):
         if isinstance(key, tuple):
             key = list(key)
         if isinstance(key, Group):
+            # FIXME: this is wrong for slices. Triggering a problem with
+            #        this needs axes with labels in different order.
+            #        We should call .eval() instead, but we might want
+            #        to keep the slice as is (for performance reasons) when
+            #        possible (when slice evaluates to the same labels on
+            #        both axes). See #1155.
+            # key = key.eval()
             key = key.to_label()
         self.key = remove_nested_groups(key)
 

larray/tests/test_axis.py

@@ -240,6 +240,15 @@ def test_getitem_group_keys():
     assert res.key == slice('a1', 'a2')
     assert res.axis is alt_a
 
+    alt2_a = Axis('a=a1,a0,a2')
+    res = a[alt2_a.i[0:3]]
+    assert isinstance(res, LGroup)
+    # FIXME: we SHOULD have this result, but it is not what we currently
+    #        return (see #1155)
+    # assert res.key == ['a1', 'a0', 'a2']
+    assert res.key == slice('a1', 'a2')
+    assert res.axis is a
+
     key = a.i[[1, 2]]
 
     res = a[key]