[test fix] Don't error out on Loggly tests with no config

joshkehn · Nov 18, 2011 · f89de80 · f89de80
1 parent 184c3bd
commit f89de80
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/test/helpers.js b/test/helpers.js
@@ -28,8 +28,8 @@ helpers.loadConfig = function (dir) {
     return config;
   }
   catch (ex) {
-    util.puts('test/fixtures/test-config.json must be created with valid data before running tests');
-    process.exit(0);
+    console.error('test/fixtures/test-config.json must be created with valid data before running tests');
+    return false;
   }
 };
 

diff --git a/test/transports/loggly-test.js b/test/transports/loggly-test.js
@@ -12,8 +12,13 @@ var path = require('path'),
     winston = require('../../lib/winston'),
     helpers = require('../helpers');
 
-var config = helpers.loadConfig(),
-    tokenTransport = new (winston.transports.Loggly)({ 
+var config = helpers.loadConfig();
+
+if (!config) {
+  return;
+}
+
+var tokenTransport = new (winston.transports.Loggly)({
       subdomain: config.transports.loggly.subdomain,
       inputToken: config.transports.loggly.inputToken
     }),
@@ -52,4 +57,5 @@ vows.describe('winston/transports/loggly').addBatch({
       })
     }
   }
-}).export(module);
+}).export(module);
+