[Matlab] Refactor MetadataMixIn function names

- Refactors all MetadataMixIn function names to CamelCase.
- Refactors all affected tests.

Author: mpsonntag

+nix/MetadataMixIn.m

@@ -17,11 +17,11 @@
     end
 
     methods
-        function r = open_metadata(obj)
+        function r = openMetadata(obj)
             r = nix.Utils.fetchObj(obj, 'openMetadataSection', @nix.Section);
         end
 
-        function [] = set_metadata(obj, val)
+        function [] = setMetadata(obj, val)
             if (isempty(val))
                 fname = strcat(obj.alias, '::setNoneMetadata');
                 nix_mx(fname, obj.nix_handle, val);

tests/TestBlock.m

@@ -530,34 +530,35 @@
     tmp = f.createSection(secName2, 'nixSection');
     b = f.createBlock('testBlock', 'nixBlock');
 
-    assert(isempty(b.open_metadata));
-    assert(isempty(f.blocks{1}.open_metadata));
+    assert(isempty(b.openMetadata));
+    assert(isempty(f.blocks{1}.openMetadata));
 
-    b.set_metadata(f.sections{1});
-    assert(strcmp(b.open_metadata.name, secName1));
-    assert(strcmp(f.blocks{1}.open_metadata.name, secName1));
+    b.setMetadata(f.sections{1});
+    assert(strcmp(b.openMetadata.name, secName1));
+    assert(strcmp(f.blocks{1}.openMetadata.name, secName1));
 
-    b.set_metadata(f.sections{2});
-    assert(strcmp(b.open_metadata.name, secName2));
-    assert(strcmp(f.blocks{1}.open_metadata.name, secName2));
-    b.set_metadata('');
-    assert(isempty(b.open_metadata));
-    assert(isempty(f.blocks{1}.open_metadata));
+    b.setMetadata(f.sections{2});
+    assert(strcmp(b.openMetadata.name, secName2));
+    assert(strcmp(f.blocks{1}.openMetadata.name, secName2));
+    b.setMetadata('');
+    assert(isempty(b.openMetadata));
+    assert(isempty(f.blocks{1}.openMetadata));
+
+    b.setMetadata(f.sections{2});
 
-    b.set_metadata(f.sections{2});
     clear tmp b f;
     f = nix.File(fileName, nix.FileMode.ReadOnly);
-	assert(strcmp(f.blocks{1}.open_metadata.name, secName2));
+	assert(strcmp(f.blocks{1}.openMetadata.name, secName2));
 end
 
 function [] = testOpenMetadata( varargin )
 %% Test: Open metadata
     f = nix.File(fullfile(pwd, 'tests', 'testRW.h5'), nix.FileMode.Overwrite);
     tmp = f.createSection('testSection', 'nixSection');
     b = f.createBlock('testBlock', 'nixBlock');
-    b.set_metadata(f.sections{1});
+    b.setMetadata(f.sections{1});
 
-    assert(strcmp(b.open_metadata.name, 'testSection'));
+    assert(strcmp(b.openMetadata.name, 'testSection'));
 end
 
 %% Test: nix.Block has nix.DataArray by ID or name
@@ -803,7 +804,7 @@
     mainSource = b.createSource(mainName, 'nixSource');
     subName = 'testSubSection1';
     s = f.createSection(subName, 'nixSection');
-    mainSource.set_metadata(s);
+    mainSource.setMetadata(s);
     subID = s.id;
 
     assert(isempty(f.blocks{1}.filterSources(nix.Filter.metadata, 'Do not exist')));
@@ -873,7 +874,7 @@
     mainEntity = b.createGroup(mainName, 'nixGroup');
     subName = 'testSubSection1';
     s = f.createSection(subName, 'nixSection');
-    mainEntity.set_metadata(s);
+    mainEntity.setMetadata(s);
     subID = s.id;
 
     assert(isempty(f.blocks{1}.filterGroups(nix.Filter.metadata, 'Do not exist')));
@@ -938,7 +939,7 @@
     mainEntity = b.createTag(mainName, 'nixTag', [1 8]);
     subName = 'testSubSection1';
     s = f.createSection(subName, 'nixSection');
-    mainEntity.set_metadata(s);
+    mainEntity.setMetadata(s);
     subID = s.id;
 
     assert(isempty(f.blocks{1}.filterTags(nix.Filter.metadata, 'Do not exist')));
@@ -1004,7 +1005,7 @@
     mainEntity = b.createMultiTag(mainName, 'nixMultiTag', d);
     subName = 'testSubSection1';
     s = f.createSection(subName, 'nixSection');
-    mainEntity.set_metadata(s);
+    mainEntity.setMetadata(s);
     subID = s.id;
 
     assert(isempty(f.blocks{1}.filterMultiTags(nix.Filter.metadata, 'Do not exist')));
@@ -1075,7 +1076,7 @@
     mainEntity = b.createDataArray(mainName, 'nixDataArray', nix.DataType.Bool, [2 9]);
     subName = 'testSubSection1';
     s = f.createSection(subName, 'nixSection');
-    mainEntity.set_metadata(s);
+    mainEntity.setMetadata(s);
     subID = s.id;
 
     assert(isempty(f.blocks{1}.filterMultiTags(nix.Filter.metadata, 'Do not exist')));
@@ -1193,7 +1194,7 @@
 
     % test nix.Filter.metadata
     sec = f.createSection('testSection', 'nixSection');
-    sl43.set_metadata(sec);
+    sl43.setMetadata(sec);
     filtered = b.filterFindSources(1, nix.Filter.metadata, sec.id);
     assert(isempty(filtered));
     filtered = b.filterFindSources(4, nix.Filter.metadata, sec.id);

tests/TestDataArray.m

@@ -128,25 +128,26 @@
     b = f.createBlock('testBlock', 'nixBlock');
     da = b.createDataArray('testDataArray', 'nixDataArray', nix.DataType.Double, [1 2]);
 
-    assert(isempty(da.open_metadata));
-    assert(isempty(f.blocks{1}.dataArrays{1}.open_metadata));
+    assert(isempty(da.openMetadata));
+    assert(isempty(f.blocks{1}.dataArrays{1}.openMetadata));
 
-    da.set_metadata(f.sections{1});
-    assert(strcmp(da.open_metadata.name, secName1));
-    assert(strcmp(f.blocks{1}.dataArrays{1}.open_metadata.name, secName1));
+    da.setMetadata(f.sections{1});
+    assert(strcmp(da.openMetadata.name, secName1));
+    assert(strcmp(f.blocks{1}.dataArrays{1}.openMetadata.name, secName1));
 
-    da.set_metadata(f.sections{2});
-    assert(strcmp(da.open_metadata.name, secName2));
-    assert(strcmp(f.blocks{1}.dataArrays{1}.open_metadata.name, secName2));
+    da.setMetadata(f.sections{2});
+    assert(strcmp(da.openMetadata.name, secName2));
+    assert(strcmp(f.blocks{1}.dataArrays{1}.openMetadata.name, secName2));
 
-    da.set_metadata('');
-    assert(isempty(da.open_metadata));
-    assert(isempty(f.blocks{1}.dataArrays{1}.open_metadata));
+    da.setMetadata('');
+    assert(isempty(da.openMetadata));
+    assert(isempty(f.blocks{1}.dataArrays{1}.openMetadata));
 
-    da.set_metadata(f.sections{2});
+    da.setMetadata(f.sections{2});
+
     clear tmp da b f;
     f = nix.File(fileName, nix.FileMode.ReadOnly);
-    assert(strcmp(f.blocks{1}.dataArrays{1}.open_metadata.name, secName2));    
+    assert(strcmp(f.blocks{1}.dataArrays{1}.openMetadata.name, secName2));    
 end
 
 %% Test: Open metadata
@@ -155,9 +156,9 @@
     tmp = f.createSection('testSection', 'nixSection');
     b = f.createBlock('testBlock', 'nixBlock');
     da = b.createDataArray('testDataArray', 'nixDataArray', nix.DataType.Double, [1 2]);
-    da.set_metadata(f.sections{1});
+    da.setMetadata(f.sections{1});
 
-    assert(strcmp(da.open_metadata.name, 'testSection'));
+    assert(strcmp(da.openMetadata.name, 'testSection'));
 end
 
 %% Test: List sources
@@ -645,7 +646,7 @@
     d.addSource(mainSource);
     subName = 'testSubSection1';
     s = f.createSection(subName, 'nixSection');
-    mainSource.set_metadata(s);
+    mainSource.setMetadata(s);
     subID = s.id;
 
     assert(isempty(f.blocks{1}.dataArrays{1}.filterSources(nix.Filter.metadata, 'Do not exist')));

tests/TestGroup.m

@@ -778,24 +778,25 @@
     tmp = f.createSection(secName2, 'nixSection');
     b = f.createBlock('testBlock', 'nixBlock');
     g = b.createGroup('testGroup', 'nixGroup');
-    assert(isempty(g.open_metadata));
-    assert(isempty(f.blocks{1}.groups{1}.open_metadata))
+    assert(isempty(g.openMetadata));
+    assert(isempty(f.blocks{1}.groups{1}.openMetadata))
 
-    g.set_metadata(f.sections{1});
-    assert(strcmp(g.open_metadata.name, secName1));
-    assert(strcmp(f.blocks{1}.groups{1}.open_metadata.name, secName1));
-
-    g.set_metadata(f.sections{2});
-    assert(strcmp(g.open_metadata.name, secName2));
-    assert(strcmp(f.blocks{1}.groups{1}.open_metadata.name, secName2));
-    g.set_metadata('');
-    assert(isempty(g.open_metadata));
-    assert(isempty(f.blocks{1}.groups{1}.open_metadata));
-
-    g.set_metadata(f.sections{2});
+    g.setMetadata(f.sections{1});
+    assert(strcmp(g.openMetadata.name, secName1));
+    assert(strcmp(f.blocks{1}.groups{1}.openMetadata.name, secName1));
+
+    g.setMetadata(f.sections{2});
+    assert(strcmp(g.openMetadata.name, secName2));
+    assert(strcmp(f.blocks{1}.groups{1}.openMetadata.name, secName2));
+    g.setMetadata('');
+    assert(isempty(g.openMetadata));
+    assert(isempty(f.blocks{1}.groups{1}.openMetadata));
+
+    g.setMetadata(f.sections{2});
+
     clear tmp g b f;
     f = nix.File(fileName, nix.FileMode.ReadOnly);
-	assert(strcmp(f.blocks{1}.groups{1}.open_metadata.name, secName2));
+	assert(strcmp(f.blocks{1}.groups{1}.openMetadata.name, secName2));
 end
 
 function [] = testOpenMetadata( varargin )
@@ -804,9 +805,9 @@
     tmp = f.createSection('testSection', 'nixSection');
     b = f.createBlock('testBlock', 'nixBlock');
     g = b.createGroup('testGroup', 'nixGroup');
-    g.set_metadata(f.sections{1});
+    g.setMetadata(f.sections{1});
 
-    assert(strcmp(g.open_metadata.name, 'testSection'));
+    assert(strcmp(g.openMetadata.name, 'testSection'));
 end
 
 function [] = testOpenDataArrayIdx( varargin )
@@ -941,7 +942,7 @@
     g.addSource(mainSource);
     subName = 'testSubSection1';
     s = f.createSection(subName, 'nixSection');
-    mainSource.set_metadata(s);
+    mainSource.setMetadata(s);
     subID = s.id;
 
     assert(isempty(f.blocks{1}.groups{1}.filterSources(nix.Filter.metadata, 'Do not exist')));
@@ -1016,7 +1017,7 @@
     g.addTag(main);
     subName = 'testSubSection1';
     s = f.createSection(subName, 'nixSection');
-    main.set_metadata(s);
+    main.setMetadata(s);
     subID = s.id;
 
     assert(isempty(f.blocks{1}.groups{1}.filterTags(nix.Filter.metadata, 'Do not exist')));
@@ -1091,7 +1092,7 @@
     g.addMultiTag(main);
     subName = 'testSubSection1';
     s = f.createSection(subName, 'nixSection');
-    main.set_metadata(s);
+    main.setMetadata(s);
     subID = s.id;
 
     assert(isempty(f.blocks{1}.groups{1}.filterMultiTags(nix.Filter.metadata, 'Do not exist')));
@@ -1165,7 +1166,7 @@
     g.addDataArray(main);
     subName = 'testSubSection1';
     s = f.createSection(subName, 'nixSection');
-    main.set_metadata(s);
+    main.setMetadata(s);
     subID = s.id;
 
     assert(isempty(f.blocks{1}.groups{1}.filterDataArrays(nix.Filter.metadata, 'Do not exist')));

tests/TestMultiTag.m

@@ -614,25 +614,26 @@
     tmp = b.createDataArray('testDataArray', 'nixDataArray', nix.DataType.Double, [1 2 3 4 5 6]);
     t = b.createMultiTag('metadataTest', 'nixMultiTag', b.dataArrays{1});
 
-    assert(isempty(t.open_metadata));
-    assert(isempty(f.blocks{1}.multiTags{1}.open_metadata));
+    assert(isempty(t.openMetadata));
+    assert(isempty(f.blocks{1}.multiTags{1}.openMetadata));
 
-    t.set_metadata(f.sections{1});
-    assert(strcmp(t.open_metadata.name, secName1));
-    assert(strcmp(f.blocks{1}.multiTags{1}.open_metadata.name, secName1));
+    t.setMetadata(f.sections{1});
+    assert(strcmp(t.openMetadata.name, secName1));
+    assert(strcmp(f.blocks{1}.multiTags{1}.openMetadata.name, secName1));
 
-    t.set_metadata(f.sections{2});
-    assert(strcmp(t.open_metadata.name, secName2));
-    assert(strcmp(f.blocks{1}.multiTags{1}.open_metadata.name, secName2));
+    t.setMetadata(f.sections{2});
+    assert(strcmp(t.openMetadata.name, secName2));
+    assert(strcmp(f.blocks{1}.multiTags{1}.openMetadata.name, secName2));
 
-    t.set_metadata('');
-    assert(isempty(t.open_metadata));
-    assert(isempty(f.blocks{1}.multiTags{1}.open_metadata));
+    t.setMetadata('');
+    assert(isempty(t.openMetadata));
+    assert(isempty(f.blocks{1}.multiTags{1}.openMetadata));
 
-    t.set_metadata(f.sections{2});
+    t.setMetadata(f.sections{2});
+
     clear tmp t b f;
     f = nix.File(fileName, nix.FileMode.ReadOnly);
-    assert(strcmp(f.blocks{1}.multiTags{1}.open_metadata.name, secName2));
+    assert(strcmp(f.blocks{1}.multiTags{1}.openMetadata.name, secName2));
 end
 
 %% Test: Open metadata
@@ -642,9 +643,9 @@
     b = f.createBlock('testBlock', 'nixBlock');
     tmp = b.createDataArray('testDataArray', 'nixDataArray', nix.DataType.Double, [1 2 3 4 5 6]);
     t = b.createMultiTag('metadataTest', 'nixMultiTag', b.dataArrays{1});
-    t.set_metadata(f.sections{1});
+    t.setMetadata(f.sections{1});
 
-    assert(strcmp(t.open_metadata.name, 'testSection'));
+    assert(strcmp(t.openMetadata.name, 'testSection'));
 end
 
 %% Test: Retrieve reference data by id or name
@@ -1119,7 +1120,7 @@
     t.addSource(mainSource);
     subName = 'testSubSection1';
     s = f.createSection(subName, 'nixSection');
-    mainSource.set_metadata(s);
+    mainSource.setMetadata(s);
     subID = s.id;
 
     assert(isempty(f.blocks{1}.multiTags{1}.filterSources(nix.Filter.metadata, 'Do not exist')));
@@ -1195,7 +1196,7 @@
     t.addReference(main);
     subName = 'testSubSection1';
     s = f.createSection(subName, 'nixSection');
-    main.set_metadata(s);
+    main.setMetadata(s);
     subID = s.id;
 
     assert(isempty(f.blocks{1}.multiTags{1}.filterReferences(nix.Filter.metadata, 'Do not exist')));