addGroupEntity: Block - add missing Group functions

Author: mpsonntag

+nix/Block.m

@@ -33,7 +33,12 @@
         function hasGroup = has_group(obj, id_or_name)
             hasGroup = nix_mx('Block::hasGroup', obj.nix_handle, id_or_name);
         end;
-        
+
+        function retObj = get_group(obj, id_or_name)
+            retObj = nix.Utils.open_entity(obj, ...
+                'Block::getGroup', id_or_name, @nix.Group);
+        end;
+
         function delCheck = delete_group(obj, del)
             [delCheck, obj.groupsCache] = nix.Utils.delete_entity(obj, ...
                 del, 'nix.Group', 'Block::deleteGroup');

nix_mx.cc

@@ -117,6 +117,7 @@ void mexFunction(int            nlhs,
             .reg("hasTag", GETBYSTR(bool, nix::Block, hasTag))
             .reg("hasMultiTag", GETBYSTR(bool, nix::Block, hasMultiTag))
             .reg("hasGroup", GETBYSTR(bool, nix::Block, hasGroup))
+            .reg("getGroup", GETBYSTR(nix::Group, nix::Block, getGroup))
             .reg("openDataArray", GETBYSTR(nix::DataArray, nix::Block, getDataArray))
             .reg("openSource", GETBYSTR(nix::Source, nix::Block, getSource))
             .reg("openTag", GETBYSTR(nix::Tag, nix::Block, getTag))

tests/TestBlock.m

@@ -26,8 +26,9 @@
     funcs{end+1} = @test_set_metadata;
     funcs{end+1} = @test_open_metadata;
     funcs{end+1} = @test_create_group;
-    funcs{end+1} = @test_delete_group;
     funcs{end+1} = @test_has_group;
+    funcs{end+1} = @test_get_group;
+    funcs{end+1} = @test_delete_group;
 end
 
 function [] = test_attrs( varargin )
@@ -375,7 +376,7 @@
     assert(strcmp(b.open_metadata.name, 'testSection'));
 end
 
-%% Test: Create Group
+%% Test: Create nix.Group
 function [] = test_create_group( varargin )
     fileName = 'testRW.h5';
     groupName = 'testGroup';
@@ -397,7 +398,37 @@
     assert(strcmp(f.blocks{1}.groups{1}.name, groupName));
 end
 
-%% Test: delete group by entity and id
+%% Test: nix.Block has nix.Group by name or id
+function [] = test_has_group( varargin )
+    groupName = 'testGroup';
+    f = nix.File(fullfile(pwd, 'tests', 'testRW.h5'), nix.FileMode.Overwrite);
+    b = f.createBlock('grouptest', 'nixBlock');
+
+    assert(~b.has_group('I do not exist'));
+    
+    g = b.create_group(groupName, 'nixGroup');
+    assert(b.has_group(b.groups{1}.id));
+    assert(b.has_group(groupName));
+
+    b.delete_group(b.groups{1});
+    assert(~b.has_group(g.id));
+end
+
+%% Test: Get nix.Group by name or id
+function [] = test_get_group( varargin )
+    groupName = 'testGroup';
+    f = nix.File(fullfile(pwd, 'tests', 'testRW.h5'), nix.FileMode.Overwrite);
+    b = f.createBlock('grouptest', 'nixBlock');
+    g = b.create_group(groupName, 'nixGroup');
+    gID = g.id;
+
+    clear g b f;
+    f = nix.File(fullfile(pwd, 'tests', 'testRW.h5'), nix.FileMode.ReadOnly);
+    assert(strcmp(f.blocks{1}.get_group(gID).name, groupName));
+    assert(strcmp(f.blocks{1}.get_group(groupName).name, groupName));
+end
+
+%% Test: Delete nix.Group by entity and id
 function [] = test_delete_group( varargin )
     fileName = 'testRW.h5';
     groupType = 'nixGroup';
@@ -419,19 +450,3 @@
     f = nix.File(fullfile(pwd, 'tests', fileName), nix.FileMode.ReadOnly);
     assert(isempty(f.blocks{1}.groups));
 end
-
-%% Test: Block has Group by name or id
-function [] = test_has_group( varargin )
-    groupName = 'testGroup';
-    f = nix.File(fullfile(pwd, 'tests', 'testRW.h5'), nix.FileMode.Overwrite);
-    b = f.createBlock('grouptest', 'nixBlock');
-
-    assert(~b.has_group('I do not exist'));
-    
-    g = b.create_group(groupName, 'nixGroup');
-    assert(b.has_group(b.groups{1}.id));
-    assert(b.has_group(groupName));
-
-    b.delete_group(b.groups{1});
-    assert(~b.has_group(g.id));
-end