Merge pull request #257 from dogriffiths/master

fix(utilities): skip invalid style attributes

Author: remarkablemark

lib/utilities.js

@@ -83,7 +83,11 @@ function setStyleProp(style, props) {
   if (style === null || style === undefined) {
     return;
   }
-  props.style = styleToJS(style, styleToJSOptions);
+  try {
+    props.style = styleToJS(style, styleToJSOptions);
+  } catch (err) {
+    props.style = {};
+  }
 }
 
 /**

test/index.test.js

@@ -153,3 +153,12 @@ describe('trim option', () => {
     );
   });
 });
+
+describe('invalid styles', () => {
+  it('copes with invalid styles', () => {
+    const html = '<p style="font - size: 1em">X</p>';
+    const options = {};
+    const reactElement = parse(html, options);
+    expect(render(reactElement)).toBe('<p>X</p>');
+  });
+});

test/utilities.test.js

@@ -117,4 +117,11 @@ describe('setStyleProp', () => {
     expect(setStyleProp(style, props)).toBe(undefined);
     expect(props).toMatchSnapshot();
   });
+
+  it('does not set props.style when style attribute corrupt', () => {
+    const style = `font - size: 1em`;
+    const props = {};
+    expect(setStyleProp(style, props)).toBe(undefined);
+    expect(props).toEqual({ style: {} });
+  });
 });