on 2nd thought, only coerce numbers to strings

etpinard · etpinard · commit ad8db59d9a2a · 2016-07-12T14:33:36.000-04:00
diff --git a/src/lib/coerce.js b/src/lib/coerce.js
@@ -100,11 +100,7 @@ exports.valObjects = {
         otherOpts: ['dflt', 'noBlank', 'strict', 'arrayOk', 'values'],
         coerceFunction: function(v, propOut, dflt, opts) {
             if(typeof v !== 'string') {
-                var okToCoerce = (
-                    typeof v === 'number' ||
-                    v instanceof Date ||
-                    typeof v === 'boolean'
-                );
+                var okToCoerce = (typeof v === 'number');
 
                 if(opts.strict === true || !okToCoerce) propOut.set(dflt);
                 else propOut.set(String(v));
diff --git a/test/jasmine/tests/lib_test.js b/test/jasmine/tests/lib_test.js
@@ -529,7 +529,7 @@ describe('Test lib.js:', function() {
                     .toEqual(dflt);
 
                 expect(coerce({s: true}, {}, stringAttrs, 's'))
-                    .toEqual('true');
+                    .toEqual(dflt);
 
                 expect(coerce({s: {1: 2}}, {}, stringAttrs, 's'))
                     .toEqual(dflt);
@@ -878,12 +878,12 @@ describe('Test lib.js:', function() {
         it('should work for valType \'string\' where', function() {
             var date = new Date(2016, 1, 1);
 
-            assert(['3', '4', 'a', 3, 1.2113, '', date, false], [undefined, {}, [], null], {
+            assert(['3', '4', 'a', 3, 1.2113, ''], [undefined, {}, [], null, date, false], {
                 valType: 'string',
                 dflt: 'a'
             });
 
-            assert(['3', '4', 'a', 3, 1.2113, date, true], ['', undefined, {}, [], null], {
+            assert(['3', '4', 'a', 3, 1.2113], ['', undefined, {}, [], null, date, true], {
                 valType: 'string',
                 dflt: 'a',
                 noBlank: true
