fix: only root component gets its own state_handler

Author: kantord

src/parser/parser.js

@@ -10,7 +10,8 @@ const parse = (component_loader) => (input) => {
 
   const component = component_loader(input.component)
   const args = Object.assign({}, input.args)
-  args.state_handler = create_state_handler()
+  if (input.component === 'root')
+    args.state_handler = create_state_handler()
   const bind = component(args)
 
   return (selection) => {

src/parser/parser.test.js

@@ -55,24 +55,26 @@ describe('Parser', function() {
     component.should.be.a.Function()
   })
 
-  it('passes arguments to component', () => {
+  it('passes state handler to root', () => {
     const fake_component = sinon.stub().returns(sinon.stub().returns(sinon.spy()))
     const fake_component_loader = sinon.stub().returns(fake_component)
     const my_selection = null
-    const component = parse(fake_component_loader)({'component': 'foo', 'args': {'magic': 42}})
+    const component = parse(fake_component_loader)({'component': 'root', 'args': {'title': ''}})
     component(my_selection)
-    fake_component.should.be.calledWith({'magic': 42, 'state_handler': sinon.match.any})
+    fake_component.should.be.calledWith({
+      'title': '',
+      'state_handler': fake_state_handler
+    })
   })
 
-  it('passes state handler to root', () => {
+  it('only passes state handler to root', () => {
     const fake_component = sinon.stub().returns(sinon.stub().returns(sinon.spy()))
     const fake_component_loader = sinon.stub().returns(fake_component)
     const my_selection = null
-    const component = parse(fake_component_loader)({'component': 'root', 'args': {'title': ''}})
+    const component = parse(fake_component_loader)({'component': 'text', 'args': {'tagName': 'p'}})
     component(my_selection)
     fake_component.should.be.calledWith({
-      'title': '',
-      'state_handler': fake_state_handler
+      'tagName': 'p',
     })
   })