See http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=20092

Jc: Seems they were multiple people editing at the same time .. re-adding the note

Julien: Partially implemented in 20093

See http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=20093

"test_remove" function has been updated, there is now a "testRemove_data" slot that could serve as example.

See http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=20096
Note that the commit message incorrectly reference issue 2064

Fixed in http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=20104

Notes (for non self-explanatory lines):
qMRMLNodeAttributeTableViewTest.cxx:124: vtkMRMLNode::SetAttribute asserts name to be not null, this is why the test failed
qMRMLNodeAttributeTableView.cxx:348: Causes test to fail if no item is found, which is a valid case
vtkMRMLNode.cxx:356: To meet the expected result in test (row "empty empty" and "empty valid" with setOnNode turned on in testSetAttribute)

Please close it if the tests pass on your OS too.

The idea is to have the following methods in the test:

void testPopulate();
void testPopulate_data();

void testSelect();
void testSelect_data();

void testSetAttribute();
void testSetAttribute_data();

void testAdd();
void testAdd_data();

void testRemove();
void testRemove_data();

whereas currently the methods are the following:

void testPopulate();

void testSelect();

void testSetAttribute();
void testSetAttribute_data();

void testAdd();

void testRemove();
void testRemove_data();

I changed all the tests to use the "_data" concept in this revision:
http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=20140

Please close if satisfactory. Thanks!