improving test coverage

haraka · Feb 21, 2016 · 1cae27e · 1cae27e
1 parent e661608
commit 1cae27e
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 16 deletions.
diff --git a/config.js b/config.js
@@ -3,7 +3,6 @@
 var path         = require('path');
 
 var cfreader     = require('./configfile');
-// var logger       = require('./logger');
 
 module.exports = new Config();
 
@@ -123,4 +122,4 @@ Config.prototype.arrange_args = function (args) {
     }
 
     return [fs_name, fs_type, cb, options];
-};
+};
diff --git a/readers/ini.js b/readers/ini.js
@@ -118,17 +118,6 @@ exports.init_booleans = function (options, result) {
 };
 
 exports.logger = function (msg, level) {
-    if (!level) level = 'logwarn';
-    if (!this.haLogger) {
-        try {
-            // even inside try, loading logger makes bad things happen.
-            // this.haLogger = require('../logger');
-        }
-        catch (ignore) {}
-    }
-
-    if (!this.haLogger) { return console.log(msg); }
-
-    try { this.haLogger[level](msg); }
-    catch (e) { console.log(e); console.log(msg); }
+    // if (!level) level = 'logwarn';
+    console.log(msg);
 };
diff --git a/test/config/test.ini b/test/config/test.ini
@@ -1,3 +1,4 @@
+; comment
 
 bool_true=true
 bool_false=false

diff --git a/test/readers/flat.js b/test/readers/flat.js
@@ -44,7 +44,7 @@ exports.load = {
         test.deepEqual(result, 'line1');
         test.done();
     },
-    'returns hostname for "me"': function(test) {
+    'returns hostname for empty "me"': function(test) {
         test.expect(1);
         var result = this.flat.load( 'test/config/me', null, null, regex);
         console.log(result);