json-lines - npm Package Compare versions

Comparing version 0.1.0 to 0.2.0

lib/route/Client.js

@@ -26,2 +26,7 @@ 'use strict';
 
+Client.prototype.close = function () {
+  this.res.end();
+  this.res.removeAllListeners();
+};
+
 module.exports = Client;

package.json

 {
   "name": "json-lines",
-  "version": "0.1.0",
+  "version": "0.2.0",
   "description": "json-lines streams JSON Lines.",
@@ -5,0 +5,0 @@ "contributors": [

README.md

@@ -39,2 +39,8 @@ # json-lines
 
+If you want to close the connection, call the `close` function. This will emit the `close` event and clean up any event listeners.
+
+```javascript
+client.close();
+```
+
 ## Running the build
@@ -41,0 +47,0 @@

test/routeTests.js

@@ -59,2 +59,40 @@ 'use strict';
 
+  test('is able to close a client.', function (done) {
+    app.get('/', route(function (client) {
+      client.once('open', function () {
+        client.send({ foo: 'bar' });
+        client.close();
+      });
+    }));
+
+    http.get('http://localhost:' + port, function (res) {
+      res.once('data', function (data) {
+        assert.that(JSON.parse(data.toString())).is.equalTo({ foo: 'bar' });
+      });
+
+      res.once('end', function () {
+        done();
+      });
+    });
+  });
+
+  test('emits a close event when the client is closed from the server.', function (done) {
+    app.get('/', route(function (client) {
+      client.once('open', function () {
+        client.close();
+      });
+
+      client.once('close', function () {
+        done();
+      });
+    }));
+
+    http.get('http://localhost:' + port, function (res) {
+      setTimeout(function () {
+        res.socket.end();
+        res.removeAllListeners();
+      }, 0.5 * 1000);
+    });
+  });
+
   test('cleans up when the client disconnects.', function (done) {
@@ -61,0 +99,0 @@ app.get('/', route(function (client) {

No alert changes

Improved metrics

Total package byte prevSize

10910

increased by12.54%

Lines of code

270

increased by15.38%

Number of lines in readme file

61

increased by10.91%

No dependency changes