Fixes the error with test/logglyAppender-test

Refs #172

I will add more relevant tests later

test/logglyAppender-test.js

@@ -1,33 +1,82 @@
 "use strict";
-var assert = require('assert')
+var vows = require('vows')
-, vows = require('vows')
+, assert = require('assert')
-, layouts = require('../lib/layouts')
+, log4js = require('../lib/log4js')
-, sandbox = require('sandboxed-module');
+, sandbox = require('sandboxed-module')
+;
 
-vows.describe('../lib/appenders/loggly').addBatch({
+function setupLogging(category, options) {
-  'appender': {
+  var msgs = [];
-    topic: function() {
+  
-      var messages = []
+  var fakeLoggly = {
-      , fakeLoggly = {
+    createClient: function (options) {
-        log: function(msg) { messages.push(msg); }
+      return {
-      }
+        config: options,
-      , appenderModule = sandbox.require(
+        log: function (msg, tags) {
-        '../lib/appenders/loggly',
+          msgs.push({
-        {
+            msg: msg,
-          globals: {
+            tags: tags
-            'loggly': fakeLoggly
+          });
-          }
         }
-      )
+      };
-      , appender = appenderModule.appender(layouts.messagePassThroughLayout);
+    }
+  };
 
-      appender({ data: ["blah"] });
+  var fakeLayouts = {
-      return messages;
+    layout: function(type, config) {
+      this.type = type;
+      this.config = config;
+      return log4js.layouts.messagePassThroughLayout;
     },
+    basicLayout: log4js.layouts.basicLayout,
+    messagePassThroughLayout: log4js.layouts.messagePassThroughLayout
+  };
 
-    'should output to loggly': function(messages) {
+  var fakeConsole = {
-      assert.equal(messages[0], 'blah');
+    errors: [],
+    error: function(msg, value) {
+      this.errors.push({ msg: msg, value: value });
+    }
+  };
+
+  var logglyModule = sandbox.require('../lib/appenders/loggly', {
+    requires: {
+      'loggly': fakeLoggly,
+      '../layouts': fakeLayouts
+    },
+    globals: {
+      console: fakeConsole
+    }
+  });
+
+  log4js.addAppender(logglyModule.configure(options), category);
+  
+  return {
+    logger: log4js.getLogger(category),
+    loggly: fakeLoggly,
+    layouts: fakeLayouts,
+    console: fakeConsole,
+    results: msgs
+  };
+}
+
+log4js.clearAppenders();
+vows.describe('log4js logglyAppender').addBatch({
+  'minimal config': {
+    topic: function() {
+      var setup = setupLogging('loggly', {
+        token: 'your-really-long-input-token',
+        subdomain: 'your-subdomain',
+        tags: ['loggly-tag1', 'loggly-tag2', 'loggly-tagn'] 
+      });
+      
+      setup.logger.log('Log event #1');
+      return setup;
+    },
+    'there should be one message only': function (topic) {
+      //console.log('topic', topic);
+      assert.equal(topic.results.length, 1);
     }
   }
-          
+
-}).exportTo(module);
+}).export(module);