implement multiSet()

Author: Raynos

config-wrapper.js

@@ -21,7 +21,11 @@ function ConfigWrapper(configObject) {
     }
 
     function setKey(keyPath, value) {
-        if (typeof keyPath !== 'string' && !Array.isArray(keyPath)) {
+        if (arguments.length === 1) {
+            return multiSet(keyPath);
+        }
+
+        if (!isValidKeyPath(keyPath)) {
             throw errors.InvalidKeyPath({
                 keyPath: keyPath
             });
@@ -37,4 +41,24 @@ function ConfigWrapper(configObject) {
 
         return putPath(configObject, keyPath, v);
     }
+
+    function multiSet(obj) {
+        if (obj === null || typeof obj !== 'object') {
+            throw errors.InvalidMultiSetArgument({
+                objStr: JSON.stringify(obj),
+                obj: obj
+            });
+        }
+
+        Object.keys(obj).forEach(setEachKey);
+
+        function setEachKey(key) {
+            setKey([key], obj[key]);
+        }
+    }
+}
+
+function isValidKeyPath(keyPath) {
+    return typeof keyPath === 'string' ||
+        Array.isArray(keyPath);
 }

errors.js

@@ -40,10 +40,21 @@ var InvalidKeyPath = TypedError({
         'SUGGESTED FIX: update the `config.set()` callsite.\n'
 });
 
+var InvalidMultiSetArgument = TypedError({
+    type: 'invalid.multi.set',
+    message: 'Invalid `config.set(obj)` argument.\n' +
+        'expected an object but instead got {objStr}.\n' +
+        'SUGGESTED FIX: update the `config.set()` callsite to ' +
+            'be a valid object.\n',
+    objStr: null,
+    obj: null
+});
+
 module.exports = {
     InvalidDirname: InvalidDirname,
     MissingDatacenter: MissingDatacenter,
     DatacenterRequired: DatacenterRequired,
     DatacenterFileRequired: DatacenterFileRequired,
-    InvalidKeyPath: InvalidKeyPath
+    InvalidKeyPath: InvalidKeyPath,
+    InvalidMultiSetArgument: InvalidMultiSetArgument
 };

test/index.js

@@ -295,7 +295,7 @@ test('config.set(undefined) throws', function (assert) {
     var config = fetchConfig(__dirname);
 
     assert.throws(function () {
-        config.set(undefined);
+        config.set(undefined, 42);
     }, /invalid keypath/);
 
     assert.end();
@@ -326,3 +326,35 @@ test('config({ seed: seed })', function (assert) {
 
     assert.end();
 });
+
+test('config.set(entireObj)', function t(assert) {
+    var config = fetchConfig(__dirname);
+
+    config.set('foo', 'bar');
+    config.set('baz', { 42: 42 });
+
+    assert.equal(config.get('foo'), 'bar');
+    assert.deepEqual(config.get('baz'), { '42': 42 });
+
+    config.set({
+        foo: 'new-key',
+        baz: { 50: 50 },
+        other: 'thing'
+    });
+
+    assert.equal(config.get('foo'), 'new-key');
+    assert.deepEqual(config.get('baz'), { '42': 42, '50': 50 });
+    assert.equal(config.get('other'), 'thing');
+
+    assert.end();
+});
+
+test('config.set(weirdValue)', function t(assert) {
+    var config = fetchConfig(__dirname);
+
+    assert.throws(function () {
+        config.set(42);
+    }, /Invalid `config\.set\(obj\)` argument/);
+
+    assert.end();
+});