refactor(attributes-to-props): remove string type check in cssToJs

Since it does not provide any value as style is either undefined,
null, or string.

Author: remarkablemark

lib/attributes-to-props.js

@@ -68,26 +68,24 @@ function attributesToProps(attributes) {
 }
 
 /**
- * Converts CSS style string to JS style object.
+ * Converts inline CSS style to POJO (Plain Old JavaScript Object).
  *
- * @param  {String} style - The CSS style.
- * @return {Object}       - The JS style object.
+ * @param  {String} style - The inline CSS style.
+ * @return {Object}       - The style object.
  */
 function cssToJs(style) {
-  if (typeof style !== 'string') {
-    throw new TypeError('First argument must be a string.');
+  var styleObject = {};
+
+  if (style) {
+    styleToObject(style, function(property, value) {
+      // skip CSS comment
+      if (property && value) {
+        styleObject[camelCase(property)] = value;
+      }
+    });
   }
 
-  var styleObj = {};
-
-  styleToObject(style, function(property, value) {
-    // skip if it's a comment node
-    if (property && value) {
-      styleObj[camelCase(property)] = value;
-    }
-  });
-
-  return styleObj;
+  return styleObject;
 }
 
 module.exports = attributesToProps;

test/attributes-to-props.js

@@ -234,20 +234,6 @@ describe('attributes-to-props', () => {
         }
       );
     });
-
-    [Object, Array, Number, Date, Function].forEach(type => {
-      it(`throws an error when attributes.style=${type.name}`, () => {
-        assert.throws(
-          () => {
-            attributesToProps({ style: type });
-          },
-          {
-            name: 'TypeError',
-            message: 'First argument must be a string.'
-          }
-        );
-      });
-    });
   });
 
   describe('custom', () => {