style & className attrs should work together

Author: lukebayes

src/builder.js

@@ -122,9 +122,11 @@ function processAttrs(attrs) {
       /* eslint-disable no-param-reassign */
       attrs.className = processClassName(attrs.className);
       /* eslint-enable no-param-reassign */
-    } else if (attrs.style && typeof attrs.style === 'object') {
+    }
+    if (attrs.style && typeof attrs.style === 'object') {
       attrs.style = processStyleObject(attrs.style);
-    } else if (attrs.onenter && attrs.onkeyup || attrs.onEnter && attrs.onKeyUp) {
+    }
+    if (attrs.onenter && attrs.onkeyup || attrs.onEnter && attrs.onKeyUp) {
       throw new Error('onenter and onkeyup are mutually exclusive, consider adding a switch statement to your onkeyup handler for the enter case.');
     }
 

test/builder_test.js

@@ -59,6 +59,18 @@ describe('Nomplate ElementBuilder', () => {
     assert.equal(root.firstChild.textContent, '');
   });
 
+  it('handles className and style attrs', () => {
+    const root = builder('div', () => {
+      builder('div', {className: 'abcd', style: {color: 'blue'}}, 'hello');
+    });
+
+    assert.equal(root.nodeName, 'div');
+    const child = root.firstChild;
+    assert.equal(child.nodeName, 'div');
+    assert.equal(child.className, 'abcd');
+    assert.equal(child.attrs.style, 'color:blue;');
+  });
+
   it('resets the stack, even on error', () => {
     try {
       builder('div', () => {