Skip to content

Commit

Permalink
fixing the tests
Browse files Browse the repository at this point in the history
  • Loading branch information
mansona committed May 27, 2014
1 parent 8105c76 commit f69e43a
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 11 deletions.
3 changes: 2 additions & 1 deletion test/fixtures/hierarchy/user.json
@@ -1,4 +1,5 @@
{
"title": "My specific title",
"color": "green"
"color": "green",
"weebls": "crap"
}
6 changes: 3 additions & 3 deletions test/hierarchy-test.js
Expand Up @@ -20,8 +20,8 @@ vows.describe('nconf/hierarchy').addBatch({
"When using nconf": {
"configured with two file stores": {
topic: function () {
nconf.add('user', { type: 'file', file: userConfig });
nconf.add('global', { type: 'file', file: globalConfig });
nconf.add('user', { type: 'file', file: userConfig });
nconf.load();
return nconf;
},
Expand Down Expand Up @@ -98,12 +98,12 @@ vows.describe('nconf/hierarchy').addBatch({
assert.deepEqual(JSON.parse(data), {
apples: true,
candy: {
something: 'foo',
something: 'file1',
something1: true,
something2: true,
something5: {
first: 1,
second: 'bar'
second: 2
}
}
});
Expand Down
16 changes: 9 additions & 7 deletions test/nconf-test.js
Expand Up @@ -107,22 +107,24 @@ vows.describe('nconf').addBatch({
"without a callback": {
"should respond with the merged store": function () {
assert.deepEqual(nconf.load(), {
title: 'My specific title',
color: 'green',
movie: 'Kill Bill'
title: 'My generic title',
color: 'red',
weebls: 'crap',
movie: 'Kill Bill'
});
}
},
"with a callback": {
topic: function () {
nconf.load(this.callback.bind(null, null));
nconf.load(this.callback.bind(null, null));
},
"should respond with the merged store": function (ign, err, store) {
assert.isNull(err);
assert.deepEqual(store, {
title: 'My specific title',
color: 'green',
movie: 'Kill Bill'
title: 'My generic title',
color: 'red',
weebls: 'crap',
movie: 'Kill Bill'
});
}
}
Expand Down

0 comments on commit f69e43a

Please sign in to comment.