support dcValue feature

Author: Raynos

README.md

@@ -31,12 +31,13 @@ A zero configuration configuration loader
 
 ```js
 // server.js
+var fs = require('fs');
 var fetchConfig = require('playdoh-server/config')
 
-var NODE_ENV = process.env.NODE_ENV
 var config = fetchConfig(__dirname, {
-    dc: NODE_ENV === 'production' ?
-        '/etc/playdoh/datacenter' : null
+    dcValue: fs.existsSync('/etc/zero-config/datacenter') ?
+        fs.readFileSync('/etc/zero-config/datacenter', 'utf8') :
+        null
 })
 
 var port = config.get("port")

errors.js

@@ -18,10 +18,10 @@ var MissingDatacenter = TypedError({
 
 var DatacenterRequired = TypedError({
     type: 'datacenter.option.required',
-    message: 'expected `opts.dc` to be passed to fetchConfig.\n' +
-        'must call `fetchConfig(__dirname, { dc: "..." }).\n' +
+    message: 'expected `opts.dcValue` to be passed to fetchConfig.\n' +
+        'must call `fetchConfig(__dirname, { dcValue: "..." }).\n' +
         'instead I got opts: {strOpts}.\n' +
-        '`opts.dc` is not optional when NODE_ENV is "production".\n' +
+        '`opts.dcValue` is not optional when NODE_ENV is "production".\n' +
         'SUGGESTED FIX: update the `fetchConfig()` callsite.\n'
 });
 

read-datacenter.js

@@ -9,17 +9,21 @@ function readDatacenter(opts) {
     var env = opts.env || process.env;
     var NODE_ENV = env.NODE_ENV;
 
-    if (NODE_ENV === 'production' && !opts.dc) {
+    // specifying a datacenter is optional in dev but required
+    // in production.
+    if (NODE_ENV === 'production' && !opts.dc && !opts.dcValue) {
         throw errors.DatacenterRequired({
             strOpts: JSON.stringify(opts)
         });
     }
 
     var result;
 
-    // specifying a datacenter is optional in dev but required
-    // in production.
-    if (opts.dc) {
+    if (opts.dcValue) {
+        result = Result.Ok({
+            'datacenter': opts.dcValue.replace(/\s/g, '')
+        });
+    } else if (opts.dc) {
         var fileResult = readFileOrError(opts.dc);
         if (Result.isErr(fileResult)) {
             var err = Result.Err(fileResult);

test/index.js

@@ -137,6 +137,43 @@ test('config loads from datacenter file', withFixtures(__dirname, {
     assert.end();
 }));
 
+test('config loads from dcValue', withFixtures(__dirname, {
+    'config': {
+        'common.json': JSON.stringify({
+            a: 'a',
+            b: {
+                c: 'c',
+                d: 'd'
+            }
+        }),
+        'production.json': JSON.stringify({
+            b: {
+                c: 'c2'
+            }
+        }),
+        'production.peak1.json': JSON.stringify({
+            a: 'a3'
+        })
+    }
+}, function (assert) {
+    var env = {
+        'NODE_ENV': 'production'
+    };
+
+    var config = fetchConfig(__dirname, {
+        env: env,
+        dcValue: 'peak1'
+    });
+
+    assert.equal(config.get('datacenter'), 'peak1');
+    assert.equal(config.get('a'), 'a3');
+    assert.equal(config.get('b.c'), 'c2');
+    assert.equal(config.get('b.d'), 'd');
+    assert.deepEqual(config.get('b'), { c: 'c2', d: 'd' });
+
+    assert.end();
+}));
+
 test('config reads a datacenter file', withFixtures(__dirname, {
     datacenter: 'peak1'
 }, function (assert) {
@@ -201,7 +238,7 @@ test('will load from --config', withFixtures(__dirname, {
     assert.end();
 }));
 
-test('no opts.dc in production', function (assert) {
+test('no opts.dcValue in production', function (assert) {
     var err = catchFn(function () {
         fetchConfig(__dirname, {
             env: { NODE_ENV: 'production' }
@@ -210,8 +247,8 @@ test('no opts.dc in production', function (assert) {
 
     assert.ok(err);
     assert.equal(err.type, 'datacenter.option.required');
-    assert.ok(/expected `opts.dc`/.test(err.message));
-    assert.ok(/`opts.dc` is not optional/.test(err.message));
+    assert.ok(/expected `opts.dcValue`/.test(err.message));
+    assert.ok(/`opts.dcValue` is not optional/.test(err.message));
 
     assert.end();
 });