check slash in Container names

Author: ajtritt

src/pynwb/form/container.py

@@ -9,7 +9,10 @@ class Container(with_metaclass(abc.ABCMeta, object)):
             {'name': 'parent', 'type': 'Container', 'doc': 'the Container that holds this Container', 'default': None},
             {'name': 'container_source', 'type': str, 'doc': 'the source of this container', 'default': None})
     def __init__(self, **kwargs):
-        self.__name = getargs('name', kwargs)
+        name = getargs('name', kwargs)
+        if '/' in name:
+            raise ValueError("name '" + name + "' cannot contain '/'")
+        self.__name = name
         self.__parent = getargs('parent', kwargs)
         self.__container_source = getargs('container_source', kwargs)
         self.__children = list()

tests/unit/form_tests/test_container.py

@@ -0,0 +1,52 @@
+import unittest2 as unittest
+
+from pynwb.form.container import Container
+
+
+class MyTestClass(Container):
+
+    def __init__(self, src, name, parent=None):
+        super(MyTestClass, self).__init__(src, name, parent=parent)
+
+    def basic_add(self, **kwargs):
+        return kwargs
+
+    def basic_add2(self, **kwargs):
+        return kwargs
+
+    def basic_add2_kw(self, **kwargs):
+        return kwargs
+
+
+class MyTestSubclass(MyTestClass):
+
+    def basic_add(self, **kwargs):
+        return kwargs
+
+    def basic_add2_kw(self, **kwargs):
+        return kwargs
+
+
+class TestNWBContainer(unittest.TestCase):
+
+    def test_constructor(self):
+        """Test that constructor properly sets parent
+        """
+        parent_obj = MyTestClass('test source', 'obj1')
+        child_obj = MyTestSubclass('test source', 'obj2', parent=parent_obj)
+        self.assertIs(child_obj.parent, parent_obj)
+
+    def test_set_parent_parent(self):
+        """Test that parent setter  properly sets parent
+        """
+        parent_obj = MyTestClass('test source', 'obj1')
+        child_obj = MyTestSubclass('test source', 'obj2')
+        child_obj.parent = parent_obj
+        self.assertIs(child_obj.parent, parent_obj)
+
+    def test_slash_restriction(self):
+        self.assertRaises(ValueError, Container, 'bad/name')
+
+
+if __name__ == '__main__':
+    unittest.main()