Merge pull request #16 from brianc/conform-to-readable-stream-spec

Conform to readable stream spec

Author: brianc

Diff for: package.json

@@ -32,6 +32,6 @@
   },
   "dependencies": {
     "pg-cursor": "1.0.0",
-    "readable-stream": "^1.0.27-1"
+    "readable-stream": "^2.0.4"
   }
 }

Diff for: test/fast-reader.js

@@ -10,7 +10,13 @@ helper('fast reader', function(client) {
     var count = 0
     stream.on('readable', function() {
       var res = stream.read()
-      assert(res, 'should not return null on evented reader')
+      if (result.length !== 201) {
+        assert(res, 'should not return null on evented reader')
+      } else {
+        //a readable stream will emit a null datum when it finishes being readable
+        //https://nodejs.org/api/stream.html#stream_event_readable
+        assert.equal(res, null)
+      }
       if(res) {
         result.push(res.num)
       }