Casing should ignore non objects

Author: glena

src/helper/object.js

@@ -1,5 +1,6 @@
 /* eslint-disable no-param-reassign */
 
+var assert = require('./assert');
 var objectAssign = require('./object-assign');
 
 function pick(object, keys) {
@@ -75,21 +76,30 @@ function snakeToCamel(str) {
 }
 
 function toSnakeCase(object, exceptions) {
+  if (typeof(object) !== 'object' || assert.isArray(object) || !object === null) {
+    return object;
+  }
+
   exceptions = exceptions || [];
 
   return Object.keys(object).reduce(function (p, key) {
     var newKey = exceptions.indexOf(key) === -1 ? camelToSnake(key) : key;
-    p[newKey] = typeof(object[key]) === 'object' ? toSnakeCase(object[key]) : object[key];
+    p[newKey] = toSnakeCase(object[key]);
     return p;
   }, {});
 }
 
 function toCamelCase(object, exceptions) {
+
+  if (typeof(object) !== 'object' || assert.isArray(object) || !object === null) {
+    return object;
+  }
+
   exceptions = exceptions || [];
 
   return Object.keys(object).reduce(function (p, key) {
     var newKey = exceptions.indexOf(key) === -1 ? snakeToCamel(key) : key;
-    p[newKey] = typeof(object[key]) === 'object' ? toCamelCase(object[key]) : object[key];
+    p[newKey] = toCamelCase(object[key]);
     return p;
   }, {});
 }

test/helper/object.test.js

@@ -336,6 +336,9 @@ describe('helpers', function () {
         }
       });
 
+      expect(newObject.array_att).to.be.an('array');
+      expect(newObject.some_obj.inner_array_att).to.be.an('array');
+
       expect(newObject).to.eql({
         attr_name_1: 'attribute_1',
         attr_name_22: 'attribute_2',
@@ -373,6 +376,7 @@ describe('helpers', function () {
   });
 
   describe('toCamelCase', function () {
+
     it('should change the casing to all the attributes', function () {
       var object = {
         attr_name_1: 'attribute_1',
@@ -400,6 +404,9 @@ describe('helpers', function () {
         }
       });
 
+      expect(newObject.arrAtt).to.be.an('array');
+      expect(newObject.someObj.innerArrayAtt).to.be.an('array');
+
       expect(newObject).to.eql({
         attrName1: 'attribute_1',
         attrName22: 'attribute_2',