TheThing/nconf-lite
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Prefer this fix for #65 to 6045618

Browse source
This commit is contained in:
Michael Hart 2012-12-21 09:35:39 +11:00 committed by indexzero
parent aec2b4eb46
commit bb57c497d3
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/nconf/stores/memory.js
View file

@ -42,7 +42,7 @@ Memory.prototype.get = function (key) {
// //
while (path.length > 0) { while (path.length > 0) {
key = path.shift(); key = path.shift();
if (target && typeof target !== 'string' && key in target) { if (target && target.hasOwnProperty(key)) {
target = target[key]; target = target[key];
continue; continue;
} }

4
test/stores/memory-store-test.js
View file

@ -58,7 +58,7 @@ vows.describe('nconf/stores/memory').addBatch({
assert.equal(store.get('falsy:string:empty:not:exist'), undefined); assert.equal(store.get('falsy:string:empty:not:exist'), undefined);
}, TypeError); }, TypeError);
}, },
"within empty strings": function (store) { "within non-empty strings": function (store) {
assert.doesNotThrow(function() { assert.doesNotThrow(function() {
assert.equal(store.get('falsy:string:value:not:exist'), undefined); assert.equal(store.get('falsy:string:value:not:exist'), undefined);
}, TypeError); }, TypeError);
@ -105,4 +105,4 @@ vows.describe('nconf/stores/memory').addBatch({
} }
} }
} }
}).export(module); }).export(module);