Close test-server after use. Fixes #22.

Not stopping FakeServer after use broke the unit-tests on OS X.
msiebuhr · Dec 11, 2012 · 6be275f · 6be275f
1 parent a9a0cca
commit 6be275f
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 0 deletions.
diff --git a/test/EphemeralSocket.js b/test/EphemeralSocket.js
@@ -17,6 +17,10 @@ describe('EphemeralSocket', function () {
         s.start(done);
     });
 
+    after(function () {
+        s.stop();
+    });
+
     it("Send 50 messages", function (done) {
         this.slow(500);
         for (var i = 0; i < 50; i += 1) {

diff --git a/test/FakeServer.js b/test/FakeServer.js
@@ -28,6 +28,13 @@ FakeServer.prototype.start = function (cb) {
     this._socket.bind(this.port);
 };
 
+/* For closing server down after use.
+ */
+FakeServer.prototype.stop = function () {
+    this._socket.close();
+    this._socket = undefined;
+};
+
 /* Expect `message` to arrive and call `cb` if/when it does.
  */
 FakeServer.prototype.expectMessage = function (message, cb) {

diff --git a/test/StatsDClient.js b/test/StatsDClient.js
@@ -23,6 +23,10 @@ describe('StatsDClient', function () {
         s.start(done);
     });
 
+    after(function () {
+        s.stop();
+    });
+
     describe("Counters", function () {
         it('.counter("abc", 1) → "abc:1|c', function (done) {
             c.counter('abc', 1);