instead of passing config, overwrite in tests

smurfy · Jan 1, 2014 · 2bc88e7 · 2bc88e7
1 parent 36187f3
commit 2bc88e7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/core/candleStore.js b/core/candleStore.js
@@ -8,9 +8,10 @@ var fs = require('fs');
 var lodash = require('lodash');
 var async = require('async');
 var _ = require('lodash');
-
-var Store = function(config) {
-  config = config || require('./util').util.getConfig();
+
+var config = require('./util').getConfig();
+
+var Store = function() {
   this.directory = config.history.directory;
 
   //TODO(yin): Make this mockable, or mock the fs in tests.

diff --git a/test/csv.js b/test/csv.js
@@ -110,11 +110,8 @@ module.exports = {
     fs.existsSync(TMPDIR) || fs.mkdirSync(TMPDIR);
     async.compose(save, deflate)(CSVFILE, DATA, done);
     timer = new Timer();
-    csv = new CSVStore({
-      history: {
-        directory: TMPDIR
-      }
-    });
+    csv = new CSVStore();
+    csv.directory = TMPDIR;
   },
   // runs after each test method invocation
   tearDown: function (done) {