fix(test helpers): Split on newlines sooner so it has to happen less.

martynsmith · Apr 6, 2015 · 460bad2 · 460bad2
1 parent 21a76d2
commit 460bad2
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/test/helpers.js b/test/helpers.js
@@ -29,8 +29,8 @@ var MockIrcd = function(port, encoding, isSecure) {
 
     this.server = connectionClass.createServer(options, function(c) {
         c.on('data', function(data) {
-            var msg = data.toString(self.encoding);
-            self.incoming = self.incoming.concat(msg.split('\r\n'));
+            var msg = data.toString(self.encoding).split('\r\n').filter(function(m) { return m; });
+            self.incoming = self.incoming.concat(msg);
         });
 
         self.on('send', function(data) {
@@ -56,7 +56,7 @@ MockIrcd.prototype.close = function() {
 };
 
 MockIrcd.prototype.getIncomingMsgs = function() {
-    return this.incoming.filter(function(msg) { return msg; });
+    return this.incoming;
 };
 
 var fixtures = require('./data/fixtures');